DragonFly kernel List (threaded) for 2005-05
Re: URLs in commit messages
:-On [20050522 05:32], Sascha Wildner (saw@xxxxxxxxx) wrote:
:>* Do we really want this? :) I do because it's annoying to not being
:>able to click on new files in order to see what's in them, but for
:>larger numbers of new/removed files, this would add a bit of data to the
:Somehow I doubt deleted is interesting. New might be debateable.
:But the emails will in the end consist of more lines of URLs than log
:content. I personally find it distracting, but hey.
:Jeroen Ruigrok van der Werven <asmodai(at)wxs.nl> / asmodai / kita no mono
:Free Tibet! http://www.savetibet.org/ | http://ashemedai.deviantart.com/
:http://www.tendra.org/ | http://www.in-nomine.org/
:Trying to fool myself with Dreams that never come true...
Yes, we really want it, even the ones that just generate '-' lines
indicating deleted files. The URL's are at the end of the commit mail
for a reason :-)
Even more interesting would be a *SINGLE* URL that generates the
entire patch set (including additions and deletions) all in one go.
This way when someone makes a large commit I would only have to click
on one URL to see the whole thing.
Would you like to undertake that job? I don't think it would be too
difficult but for obvious reasons we can't just generate one big
(multi-thousand-character-long) URL. Browser's have limits to what
they will allow. What we want to do is to somehow reference the
history file itself (e.g. /cvs/CVSROOT/history) by the transaction ID
and then have a program get the file list from the history file.
So, for example, one of Sascha's recent commits which includes some
file removals is in the history file as follows:
A URL that just references the history id, 427cfd19, should be
sufficient for a program to be able to locate the enties in the
history file and generate a diff.