[jira] [Comment Edited] (OGNL-230) Update Javassist to 3.17.1

["Eduardo Macarron (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/OGNL-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13555940#comment-13555940 ] 

Eduardo Macarron edited comment on OGNL-230 at 1/17/13 7:10 AM:
----------------------------------------------------------------

Hi Lukasz.

You are right. My fault! 

<source>1.6</source>
<target>1.6</target>

It is quite difficult to follow which JDK is javassist compiled on, it changes quite frequently ;)

Note that 3.17.0 is JDK 1.7, but according to the issue you point, it was a mistake that was fixed in 3.17.1.

Note that next version will be 3.18.0 but it looks that from now over all javassist versions are going to require JDK 1.6  so I suppose that commons-ognl should stay in 3.16.1-GA then.

Thanks for looking into this!

                
      was (Author: emacarron):
    Hi Lukasz.

You are right. My fault! 

<source>1.6</source>
<target>1.6</target>

Is quite difficult to follow with JDK is javassist compiled on, it changes quite frequently ;)

Note that 3.17.0 is JDK 1.7, but according to the issue you point, it was a mistake that was fixed in 3.17.1.

Note that next version will be 3.18.0 but it looks that from now over all javassist versions are going to require JDK 1.6  so I suppose that commons-ognl should stay in 3.16.1-GA then.

Thanks for looking into this!

                  
> Update Javassist to 3.17.1
> --------------------------
>
>                 Key: OGNL-230
>                 URL: https://issues.apache.org/jira/browse/OGNL-230
>             Project: Commons OGNL
>          Issue Type: Improvement
>          Components: Core Runtime
>            Reporter: Eduardo Macarron
>            Priority: Trivial
>              Labels: javassist
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> Hello. 
> I have just tested latest MyBatis trunk with OGNL 4 and all test pass. Good work!
> I noticed OGNL 4 depends on a quite old Javassist version (3.12.1). I would like to suggest to upgrade to 3.17.1 (still jdk 5) before going GA.
> BTW, this version is ASL 2.0 (old versions are not)
> Note Javasisst maven groupid has changed:
> from:
>     <dependency>
>       <groupId>javassist</groupId>
>       <artifactId>javassist</artifactId>
>       <version>3.12.1-GA</version>
>     </dependency>
> to: 
>     <dependency>
>       <groupId>org.javassist</groupId>
>       <artifactId>javassist</artifactId>
>       <version>3.17.1-GA</version>
>     </dependency>

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira