unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Paul Eggert <eggert@cs.ucla.edu>
To: esr@thyrsus.com
Cc: emacs-devel@gnu.org
Subject: Re: Converted git repository available for review
Date: Fri, 21 Mar 2014 20:37:33 -0700	[thread overview]
Message-ID: <532D057D.9050305@cs.ucla.edu> (raw)
In-Reply-To: <20140322030017.GA29313@thyrsus.com>

Eric S. Raymond wrote:
> What I've
> done is treat the Bazaar ignore files as authoritative for those
> revisions during which Bazaar was in use, ignoring .gitignores during
> that period.  The other major possibility would be to simply remove
> .bzrignores where .gitignore files exist.

Sorry, I'm not really following all that, as I haven't read the lift 
script.  Still, it puzzles me that the result is a .gitignore that has 
the wrong data in it, in the sense that it has an arch tag, something 
that is clearly wrong since we went through and removed those.  If we're 
generating .gitignore files from .bzrignore files, and if the .bzrignore 
file just before the bzr-to-git transition lacks an arch tag, why would 
the .gitignore file after the transition have it?  And if we're using 
some other process to generate the .gitignore file, then how did it 
manage to keep the arch tags even though we removed them?

How about if we fix the .bzrignore and .gitignore files to agree now, by 
hand, so that this part of the bzr-to-git transition is a no-op?

> But how much do we care about fewer than 108 overlong lines, really?
> I'm not even sure your small effort would be really justified, let
> alone my larger one.

I know some people care about readable ChangeLog entries.  We edit them 
even years after they were written to fix typos, for example.  It's 
worth the small effort, though not the larger one.

I wrote earlier that these changes could be made to the bzr trunk now, 
but since they're so small and niggling we might as well make them to 
the emacs-24 branch.



  reply	other threads:[~2014-03-22  3:37 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-21 20:00 Converted git repository available for review Eric S. Raymond
2014-03-21 22:00 ` Stefan
2014-03-21 22:12   ` Eric S. Raymond
2014-03-21 22:06 ` Juanma Barranquero
2014-03-22  1:00 ` Paul Eggert
2014-03-22  3:00   ` Eric S. Raymond
2014-03-22  3:37     ` Paul Eggert [this message]
2014-03-22  3:42       ` Juanma Barranquero
2014-03-22  4:29       ` Eric S. Raymond
2014-03-22 14:56         ` Paul Eggert
2014-03-22 18:47           ` Eric S. Raymond
2014-03-24  9:24             ` .gitignore and .bzrignore files Nicolas Richard
2014-03-24 10:38               ` Eric S. Raymond
2014-03-24 11:53                 ` Stephen J. Turnbull
2014-03-24 15:54                 ` Nicolas Richard
2014-03-24 22:44               ` Achim Gratz
2014-03-22  4:41 ` Converted git repository available for review Juanma Barranquero
2014-03-22  7:32   ` Stephen J. Turnbull
2014-03-22  7:38     ` David Kastrup
2014-03-22  7:35   ` Eli Zaretskii
2014-03-22 11:48     ` Juanma Barranquero
2014-03-22  9:12   ` Eric S. Raymond
2014-03-22 11:47     ` Juanma Barranquero
2014-03-22  9:40   ` Andreas Schwab
2014-03-22  8:02 ` Andreas Schwab
2014-03-22  9:20   ` Eric S. Raymond
2014-03-22  9:44     ` Andreas Schwab
2014-03-22 11:03       ` Eric S. Raymond
2014-03-24 22:33         ` James Cloos
2014-03-24 22:39           ` Eric S. Raymond
2014-03-25  7:57             ` David Kastrup
2014-03-25 10:54               ` Eric S. Raymond
2014-03-25 11:04                 ` David Kastrup
2014-03-25 11:32                   ` Eric S. Raymond
2014-03-25 11:39                     ` David Kastrup
2014-03-25 11:50                       ` Eric S. Raymond
2014-03-25 12:47                   ` Phillip Lord
2014-03-25 13:04                     ` David Kastrup

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=532D057D.9050305@cs.ucla.edu \
    --to=eggert@cs.ucla.edu \
    --cc=emacs-devel@gnu.org \
    --cc=esr@thyrsus.com \
    /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).