unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Miles Bader <miles@gnu.org>
Cc: emacs-devel@gnu.org, spiegel@gnu.org, miles@gnu.org
Subject: Re: deleting rcs keywords from emacs sources
Date: Tue, 23 Mar 2004 16:47:37 -0500	[thread overview]
Message-ID: <20040323214737.GA21737@fencepost> (raw)
In-Reply-To: <4242667.1080067569994.JavaMail.root@tintin.london.ongenie.net>

On Tue, Mar 23, 2004 at 06:46:09PM +0000, spiegel@genion.de wrote:
> > if they're actively tracking CVS, they can easily
> > enough get the cvs revision with `cvs status').
> 
> Is that really the only answer?  To find out what version of a file you are 
> looking at, you need to be hooked up to the version control system?

In general, the version of a single file is not what you want for bug
reports, you want the state of the entire _system_.

There are several reasons for this, e.g. (a) it doesn't rely on the the
bug-reporter knowing what the relevant file is (which they often get wrong,
even if they think they do), and (b) many problems which manifest in a
particular piece of code are actually due to changes outside it, or
interactions between changes in both places.  Really you want bug reports to
come from M-x report-emacs-bug, unless the submitter is very knowledgeable,
as that will make sure to include other info that will tell you the state
more exactly.

If someone is using a packaged distribution of emacs, then it should be set
up to generate a meaningful emacs version number, which will tell you what
snapshot of the source tree it is from.

The only other common case (AFAIK) is that someone is actively tracking emacs
CVS; in this case, you probably _still_ want to know the state of the whole
system for the reasons stated above, but the version reported by M-x
emacs-version is not very useful (it effectively just tells you "CVS sometime
after this release").  However, you also know that in this case both that the
user has CVS info easily available, and is probably more knowledgable than
average (so probably _knows_ they have CVS info available).

So given this, I think we ought to (1) encourage people to use M-x
report-emacs-bug, (2) find a way to make M-x emacs-version give more useful
info for CVS checkouts (easy for arch though).

> I often send files back and forth, and frequently I need to decide what
> version I'm looking at, with the file being isolated from its version
> control history.

Er, if you're making changes outside of CVS than the keyword info isn't
going to be very useful, is it?

> I find it completely natural, even essential that a file is stamped with the 
> version it contains, the stamp being a part of the actual contents of the 
> file.

I've seen others say this too, and I suspect that's a major reason people use
rcs keywords -- it gives them warm fuzzies.

> It should be the job of the version control system to ignore this kind of
> changes for operations where it clearly doesn't make sense.  CVS, as
> somebody pointed out, is capable of this for the headers it produces.

It's too much of a tar pit: it means that the V.C.S. is _changing_ the
contents of the file, which is _dangerous_ (see the CVS documentation for
details); arch uses the much more robust model that what you put in is
exactly what you get out.

These keywords are the sort of thing that may seem like a great idea at
first, but which come back to haunt you later (as we now see :-)

-Miles
-- 
Somebody has to do something, and it's just incredibly pathetic that it
has to be us.  -- Jerry Garcia

  reply	other threads:[~2004-03-23 21:47 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-03-23 18:46 deleting rcs keywords from emacs sources spiegel
2004-03-23 21:47 ` Miles Bader [this message]
2004-03-25  7:45   ` Andre Spiegel
2004-03-26 16:45     ` Richard Stallman
2004-03-26 18:10       ` Stefan Monnier
2004-03-26 18:46       ` David Kastrup
2004-03-28  1:36         ` Richard Stallman
2004-03-26 19:05       ` Robert J. Chassell
2004-03-26 19:37       ` Andre Spiegel
2004-03-28  1:36         ` Richard Stallman
2004-03-28 11:10           ` Andre Spiegel
2004-03-29 20:56             ` Richard Stallman
2004-03-30 14:22               ` Andre Spiegel
2004-03-31 15:05                 ` Richard Stallman
  -- strict thread matches above, loose matches on Subject: below --
2004-03-23  6:46 Miles Bader
2004-03-23  7:47 ` Eli Zaretskii
2004-03-23 10:56   ` Kim F. Storm
2004-03-23  9:51 ` spiegel
2004-03-23 10:16   ` Miles Bader
2004-03-23 11:41   ` Juanma Barranquero
2004-03-23 13:13     ` Miles Bader
2004-03-23 14:01       ` Juanma Barranquero
2004-03-23 14:35         ` Miles Bader
2004-03-23 14:58           ` Juanma Barranquero
2004-03-23 15:14             ` Stefan Monnier
2004-03-23 15:36               ` Juanma Barranquero
2004-03-24  5:34             ` Richard Stallman
2004-03-23 15:04         ` Stefan Monnier
2004-03-23 18:17       ` Nick Roberts
2004-03-23 18:07     ` Nick Roberts
2004-03-24  5:34 ` Richard Stallman
2004-03-25  8:17   ` Miles Bader

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20040323214737.GA21737@fencepost \
    --to=miles@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=spiegel@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 public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).