Bug 1287 - feature request: patch generation tool
feature request: patch generation tool
Status: NEW
Product: ns-3
Classification: Unclassified
Component: general
pre-release
All All
: P5 enhancement
Assigned To: ns-bugs
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-11-03 11:50 EDT by Tom Henderson
Modified: 2012-02-29 22:41 EST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Henderson 2011-11-03 11:50:14 EDT
a recent post on the mailing list about generating patches for review got me thinking that it may be useful to include a program in utils/ that would generate a properly formatted patch against ns-3-dev, passing all the right --exclude flags to exclude build/ .hg/ bindings/ directories.

for instance, something like 
utils/generate-patch.sh

could do an hg clone of ns-3-dev in some temporary directory, run the diff properly, and clean up after itself, leaving a patch named such as diff-against-<changeset>.patch

it could be extended to allow users to pass other options such as -r <repository> (if not ns-3-dev), -n <name-of-patch>, etc.
Comment 1 Tommaso Pecorella 2012-02-29 18:18:49 EST
In my nightly buglist checks to see if I can fix something, I stumbled on this one.

I think it's safe to Downgrade it to enhancement (it's a feature request indeed).

On the other hand, out of curiosity: isn't upload.py already doing something similar ? 

T.
Comment 2 Tom Henderson 2012-02-29 22:41:37 EST
(In reply to comment #1)
> In my nightly buglist checks to see if I can fix something, I stumbled on this
> one.
> 
> I think it's safe to Downgrade it to enhancement (it's a feature request
> indeed).
> 
> On the other hand, out of curiosity: isn't upload.py already doing something
> similar ? 

Yes, if the target is to upload to Rietveld (which it not always is)