Spring AOP proxies broken

as discussed in GROOVY-3433 I close this one as "won't fix" too

Guys as I said said in GROOVY-3433 the fix for that change needed to be rolled back in order for this bug to be closed. I was very explicit about that in my dicussions with both Jochen and Guillaume and in my comment on the issue:

http://jira.codehaus.org/browse/GROOVY-3433?focusedCommentId=173642&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_173642

However, the change hasn't been rolled back at all and in fact there is nothing in the SVN logs that shows that it has been rolled back and Spring proxies as well as Grails test continue to be broken with 1.6.2

It seems not even the test attached to this issue was included in the Groovy build which I would have thought would have been a minimum requirement to avoid regression and would have caught the problem before a release was done. Anyway so it looks like we need Groovy 1.6.3 now which is a rather embarassing situation to be in.

you said the changes needs to be rolled back, I did that for 1.6.x. actually I accidentally mixed that into another commit, which caused the build to break and in the next commit I removed the test Alex added for GROOVY-3433, since that one is now no longer working... prove that the change has been rolled back. See http://svn.groovy.codehaus.org/browse/groovy/branches/GROOVY_1_6_X/src/main/org/codehaus/groovy/runtime/callsite/AbstractCallSite.java?r1=15902&r2=16093 for details. The change was http://svn.groovy.codehaus.org/browse/groovy/branches/GROOVY_1_6_X/src/main/org/codehaus/groovy/runtime/callsite/AbstractCallSite.java?r1=14182&r2=15768 or not?

But you are right, the test does not work. I should have looked after that.

After looking more into that I now know that you did mean http://svn.groovy.codehaus.org/changelog/groovy?cs=15848 I didn't find this before because this revision is not tied to GROOVY-3433