Date: Fri, 14 Sep 2001 01:02:40 -0400 From: Garance A Drosihn <drosih@rpi.edu> To: dan@langille.org, Josef Karthauser <joe@tao.org.uk> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: Proposed patch to log_accum.pl Message-ID: <p05101011b7c73e0d45be@[128.113.24.47]> In-Reply-To: <3BA133F0.17179.75B20696@localhost> References: <3BA133F0.17179.75B20696@localhost>
next in thread | previous in thread | raw e-mail | index | archive | help
At 10:32 PM -0400 9/13/01, Dan Langille wrote: >I like that idea, from a FreshPorts II point of view. One of the >technical problems is tracking which version of a file has been >deleted. Having the above helps greatly. > >A more important item, from a FreshPorts II point of view is the >path name is often incomplete. That is, the first directory is >often omitted. A few example from recent commits: > > Added files: > biology/tinker/files tview > > Removed files: > biology/tinker/files patch-ag > > Added files: > usr.sbin/kldxref Makefile ef.c ef.h fileformat kldxref.8 > kldxref.c > >The logistics of determining the actual directory prefix is a bit >beyond hard. It would be simplified by a log entry such ... If you're running into the same challenge I ran into with cvsmerge, you won't need a change to this part of the log entry. The pathname is only trimmed-down in the "first section" of the log entry. In the section that Josef is changing, which lists the revision numbers, you always get the full path name. Right now my script *has* to parse the above section, because there is no other way to know about added and removed files. After Josef's proposed change, it should be true that scripts can completely ignore this first section of the commit message, and only parse the later section (and that does have the full pathname). The first section is trying to be as compact as possible, to make it easy for humans to skim thru. My guess is that it won't need to be changed. -- Garance Alistair Drosehn = gad@eclipse.acs.rpi.edu Senior Systems Programmer or gad@freebsd.org Rensselaer Polytechnic Institute or drosih@rpi.edu To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?p05101011b7c73e0d45be>