[Graph] how to test/submit patch

I would like to submit a patch for the Graph library. (This is the iterator splitting issue I raised some time ago.) So what is the procedure? 1) check out source from sourceforge SVN (done) 2) make changes 3) run tests??? (what is the best way to ensure the behavior has not changed?) 4) generate the patch (use svn diff?) 5) submit somewhere I just want to be sure I'm pulling the correct tree and there is some means to run regressions on my changes. Thanks. THK -- Timothy H. Keitt Associate Professor Section of Integrative Biology University of Texas at Austin http://www.keittlab.org/

I would like to submit a patch for the Graph library. (This is the iterator splitting issue I raised some time ago.)
So what is the procedure?
Same as before (file a ticket, discuss the patch). Again, I probably won't accept a patch that alters the default behavior of existing components. Andrew Sutton andrew.n.sutton@gmail.com

At 1:02 PM -0500 5/29/09, Tim Keitt wrote:
I would like to submit a patch for the Graph library. (This is the iterator splitting issue I raised some time ago.)
Same answer as on the -users list ;-)
So what is the procedure?
1) check out source from sourceforge SVN (done) 2) make changes 3) run tests??? (what is the best way to ensure the behavior has not changed?)
Run the tests. If you think that there should be more tests, write them. Ideally, you should write a test that fails on the unpatched version of the library, and succeeds after applying your patch.
4) generate the patch (use svn diff?)
Yep.
5) submit somewhere
In a Trac ticket. <http://svn.boost.org>
I just want to be sure I'm pulling the correct tree and there is some means to run regressions on my changes.
Sounds good to me. -- -- Marshall Marshall Clow Idio Software <mailto:marshall@idio.com> It is by caffeine alone I set my mind in motion. It is by the beans of Java that thoughts acquire speed, the hands acquire shaking, the shaking becomes a warning. It is by caffeine alone I set my mind in motion.
participants (3)
-
Andrew Sutton
-
Marshall Clow
-
Tim Keitt