From: Miles Bader <miles@lsi.nec.co.jp>
Cc: gnu-arch-users@gnu.org, emacs-devel@gnu.org
Subject: Re: cvs <-> arch mirroring scripts
Date: 18 Aug 2003 13:10:37 +0900 [thread overview]
Message-ID: <buo65kvd2rm.fsf@mcspd15.ucom.lsi.nec.co.jp> (raw)
In-Reply-To: <E19oRLW-0007rQ-14@fencepost.gnu.org>
Richard Stallman <rms@gnu.org> writes:
> ----------------------------
> revision 1.43
> date: 2003/08/14 09:05:44; author: miles; state: Exp; lines: +2 -2
> Revision: emacs--cvs-trunk--0--patch-14
> Archive: miles@gnu.org--gnu-2003
> Creator: Miles Bader <miles@gnu.org>
> Summary: Avoid .arch-ids dirs when making autoloads
> Modified-files: ./lisp/Makefile.in
> New-patches: miles@gnu.org--gnu-2003/emacs--cvs-trunk--0--patch-14
>
> That is 6 lines longer than normal. The extra lines will make it
> harder to scan the CVS log for relevant information. Can you make
> that any more compact? Is all of it really necessary?
Hmmm, well lets see; here are the extra added lines:
Revision: Identifies the arch changeset the change came from;
I think this is very useful information and should be
kept.
Archive: This is really part of the changeset name; it could be
put into the Revision: header I think.
Creator: This is redundant in this case, but since CVS checkins
from arch probably will occur from a single `gateway user'
so that it's necessary to record the _actual_ author
somewhere.
This field might be eliminated for cases where both
names are the same, or if there were a way to make CVS
record a different author than the person who did the
checkin (I've heard of CVS patches to allow this, but I
think that implementation only works with a local CVS
server).
Summary: This is actually part of the description; maybe it
should be moved to be directly in front of the rest of
the description, or even made part of the description.
Modified-files: I think this is useful information, but it could be
elided in the case where only one file was part of the
changeset.
New-patches: This is mainly useful to arch, so I think it could be
removed (it can always be reconstructed by looking up
the arch changeset using the information in the
Revision: header).
So a typical one-file changeset might become something like this:
revision 1.43
date: 2003/08/14 09:05:44; author: arch; state: Exp; lines: +2 -2
Revision: miles@gnu.org--gnu-2003/emacs--cvs-trunk--0--patch-14
Creator: Miles Bader <miles@gnu.org>
Avoid .arch-ids dirs when making autoloads
A bigger changeset would add the Modified-files: header back in, but
since the DESCRIPTION would likely be much longer too, it's probably
less of an issue, and I think being able to see exactly which files
were modifies is useful. E.g.:
revision X.Y
date: 2003/08/14 09:05:44; author: arch; state: Exp; lines: +2 -2
Revision: miles@gnu.org--gnu-2003/emacs--cvs-trunk--0--patch-73
Creator: Miles Bader <miles@gnu.org>
Modified-files: lisp/file2.el src/file1.c src/file2.c
SUMMARY
LONG DESCRIPTION
...
This seems pretty readable to me; what do you think?
-Miles
--
`Suppose Korea goes to the World Cup final against Japan and wins,' Moon said.
`All the past could be forgiven.' [NYT]
_______________________________________________
Gnu-arch-users mailing list
Gnu-arch-users@gnu.org
http://mail.gnu.org/mailman/listinfo/gnu-arch-users
GNU arch home page:
http://savannah.gnu.org/projects/gnu-arch/
next prev parent reply other threads:[~2003-08-18 4:10 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1060744673.2522.27.camel@columbia>
[not found] ` <buo65l2facg.fsf@mcspd15.ucom.lsi.nec.co.jp>
[not found] ` <1060803518.28891.27.camel@columbia>
2003-08-14 9:35 ` [arch-users] Re: cvs <-> arch mirroring scripts Miles Bader
2003-08-14 9:46 ` Robert Collins
2003-08-14 9:59 ` Miles Bader
2003-08-17 17:29 ` Richard Stallman
2003-08-17 17:42 ` Rajesh Vaidheeswarran
2003-08-18 19:15 ` Richard Stallman
2003-08-19 14:38 ` [arch-users] " Rajesh Vaidheeswarran
2003-08-20 17:23 ` Richard Stallman
2003-08-22 8:17 ` dhruva
2003-08-18 4:10 ` Miles Bader [this message]
2003-08-18 19:16 ` Richard Stallman
2003-08-18 22:29 ` Miles Bader
2003-08-20 2:43 ` Richard Stallman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=buo65kvd2rm.fsf@mcspd15.ucom.lsi.nec.co.jp \
--to=miles@lsi.nec.co.jp \
--cc=emacs-devel@gnu.org \
--cc=gnu-arch-users@gnu.org \
--cc=miles@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.