[jboss-dev-forums] [JBoss AOP Development] - Re: Help needed to create a patch.

[jaikiran]

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#4254900

Reply to the post : http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&p=4254900