bugGNU arch -- a revision control system - Bugs: bug #7020, tla does not combine multiple...

 
 

bug #7020: tla does not combine multiple rejects

Submitted by:  James Blackwell <jblack>
Submitted on:  Tue 30 Dec 2003 05:16:58 AM UTC  
 
Category: tlaSeverity: 2 - Minor
Item Group: small feature ideaStatus: None
Privacy: PublicOpen/Closed: Open
Release: tla-1.2
Fixed Release: 
Merge Request?: None
Your Archive Name: 
Your Archive Location: 
Assigned to: None

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 09 Jan 2004 10:17:12 PM UTC, comment #2:

BTW, I didn't put those angle-brackets around the revision names in the example I gave!

Yet another case of wacky-ass savannah `email' address munging I guess... though in this case it doesn't even seem to have tried to spam-proof them, so I don't know what it was trying to do.

Miles Bader <miles>
Fri 09 Jan 2004 10:03:50 PM UTC, comment #1:

Is this really desirable though?

Knowing that different rejects came from different changesets is useful, though obviously the way arch implements currently can be hard to use.

Hmmm, since patch files can contain arbitrary text interspersed with the diffs, how about making tla somehow append to .rej files, e.g.:

Rejects from foo@bar--baz/ack--barf--0--patch-9:
<diff 1>

Rejects from foo@bar--baz/ack--barf--0--patch-10:
<diff 2>

etc.

Since it's `patch' that makes the .rej files (I think!) it would be quite as simple as that, but tla could -- maybe in addition to moving all .orig/.rej files into a separate directory -- append old .rej files onto the end of a similarly named .old-rejs file, with a format like the above.

Food for thought anyway...

I do find the current system a bit problematic, as I tend to not notice the +...old-rejs... directories that tla creates until much later.

Miles Bader <miles>
Tue 30 Dec 2003 05:16:58 AM UTC, original submission:

Currently, when updating or replaying, multiple rejects do not result in a cumulative reject file.

Tim Waugh has made a patchutils package at http://cyberelk.net/tim/patchutils/, which has a patch combiner. It may be possible to take Tim's code and add reject unification abilities to arch.

James Blackwell <jblack>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

No Changes Have Been Made to This Item

Back to the top


Powered by Savane 3.1-cleanup1