anonymous wrote : I do need to create a diff now if I'm correct. | So my next question would be of what? Going by your first post, the file that needs change is org.jboss.aop.proxy.ClassProxyFactory.java. So the steps would be (now that you are done with building a clean JBoss AOP code): 1) Do your change (fix) in the org.jboss.aop.proxy.ClassProxyFactory.java 2) Create a simple testcase which ensures that this fix works. You can check some of the existing testcases to see how you can write a simple new one. 3) Run the tests with your fix included to ensure there are no regressions 4) Create the svn diff as explained here http://jboss.org/help/submittingpatches.html That's it :) And if you have any more questions, feel free to post them in this forum. View the original post : http://www.jboss.org/index.html?module=bb&op=viewtopic&p=4254900#... Reply to the post : http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&a...