Re: [Emacs-diffs] /srv/bzr/emacs/trunk r103740: Remove some files that autoreconf can supply.

Re: [Emacs-diffs] /srv/bzr/emacs/trunk r103740: Remove some files that autoreconf can supply.

[Stefan Monnier]

> removed:
>   compile
>   config.guess
>   config.sub
>   depcomp
>   install-sh
>   missing
> added:
>   autogen/compile
>   autogen/config.guess
>   autogen/config.sub
>   autogen/depcomp
>   autogen/install-sh
>   autogen/missing

The above is bad.  Never ever do "add+remove" instead of "bzr mv".
Please please please.  This is not CVS any more.  We've been through
this so many times already I'm surprised we still see this issue coming
back every time we perform such reorganizations.
Let's be more careful in the future.


        Stefan

Re: [Emacs-diffs] /srv/bzr/emacs/trunk r103740: Remove some files that autoreconf can supply.

[Glenn Morris]

Stefan Monnier wrote:

> The above is bad.  Never ever do "add+remove" instead of "bzr mv".
> Please please please.  This is not CVS any more.  We've been through
> this so many times already I'm surprised we still see this issue coming
> back every time we perform such reorganizations.
> Let's be more careful in the future.

I understand this point, but these are "generated" (sort of...) files.
The history is not important. Indeed, the history has been reset, since
they are now the versions that automake-1.11 supplies rather than the
versions that were previously in Emacs. In some sense, they aren't even
the same files. The "missing" that you use to build Emacs isn't in the
repo any more. Instead there is a fallback that you can copy into place
if you really need to. That is why I did not "move" them.

Re: [Emacs-diffs] /srv/bzr/emacs/trunk r103740: Remove some files that autoreconf can supply.

[Óscar Fuentes]

Glenn Morris <rgm@gnu.org> writes:

> Stefan Monnier wrote:
>
>> The above is bad.  Never ever do "add+remove" instead of "bzr mv".
>[snip]
>
> I understand this point, but these are "generated" (sort of...) files.
> The history is not important.
[snip]

Apart from preserving history, there is another reason for using `bzr
mv' instead of plain `mv': the latter requires storing again the file
contents into the database, while the former represents the operation in
a very efficient way, which saves storage and network traffic.

Re: [Emacs-diffs] /srv/bzr/emacs/trunk r103740: Remove some files that autoreconf can supply.

[Eli Zaretskii]

> From: Glenn Morris <rgm@gnu.org>
> Date: Fri, 25 Mar 2011 13:13:13 -0400
> Cc: emacs-devel@gnu.org
> 
> I understand this point, but these are "generated" (sort of...) files.
> The history is not important. Indeed, the history has been reset, since
> they are now the versions that automake-1.11 supplies rather than the
> versions that were previously in Emacs. In some sense, they aren't even
> the same files. The "missing" that you use to build Emacs isn't in the
> repo any more. Instead there is a fallback that you can copy into place
> if you really need to. That is why I did not "move" them.

I think at least the history of src/config.in and lib/Makefile.in
should be preserved, since they are affiliated with Emacs (unlike the
other files which are brought from outside).  Does anyone object to
restoring the history of these two files?

Re: [Emacs-diffs] /srv/bzr/emacs/trunk r103740: Remove some files that autoreconf can supply.

[Glenn Morris]

Eli Zaretskii wrote:

> I think at least the history of src/config.in and lib/Makefile.in
> should be preserved, since they are affiliated with Emacs (unlike the
> other files which are brought from outside).  Does anyone object to
> restoring the history of these two files?

Not at all. Please just do (or I can try to do) it so that we can get on
with other things. Maybe do the others at the same time, then it's done?

Re: [Emacs-diffs] /srv/bzr/emacs/trunk r103740: Remove some files that autoreconf can supply.

[Eli Zaretskii]

> From: Glenn Morris <rgm@gnu.org>
> Cc: monnier@iro.umontreal.ca,  emacs-devel@gnu.org
> Date: Fri, 25 Mar 2011 14:13:27 -0400
> 
> Eli Zaretskii wrote:
> 
> > I think at least the history of src/config.in and lib/Makefile.in
> > should be preserved, since they are affiliated with Emacs (unlike the
> > other files which are brought from outside).  Does anyone object to
> > restoring the history of these two files?
> 
> Not at all. Please just do (or I can try to do) it so that we can get on
> with other things. Maybe do the others at the same time, then it's done?

Done for all of the files.

Re: [Emacs-diffs] /srv/bzr/emacs/trunk r103740: Remove some files that autoreconf can supply.

[Glenn Morris]

Eli Zaretskii wrote:

> Done for all of the files.

Great! Thanks.

Re: [Emacs-diffs] /srv/bzr/emacs/trunk r103740: Remove some files that autoreconf can supply.

[Stefan Monnier]

>> The above is bad.  Never ever do "add+remove" instead of "bzr mv".
>> Please please please.  This is not CVS any more.  We've been through
>> this so many times already I'm surprised we still see this issue coming
>> back every time we perform such reorganizations.
>> Let's be more careful in the future.
> I understand this point, but these are "generated" (sort of...) files.

Yes, that history is not really important.  But we should simply always
use "bzr mv" when it can be used unless there's really a good reason
against it (never came across one, FWIW).


        Stefan

Re: [Emacs-diffs] /srv/bzr/emacs/trunk r103740: Remove some files that autoreconf can supply.

[Tom Tromey]

Stefan> Yes, that history is not really important.  But we should simply always
Stefan> use "bzr mv" when it can be used unless there's really a good reason
Stefan> against it (never came across one, FWIW).

It seems like bzr could recognize this case and ask if you intended to
`bzr mv' instead.

Otherwise, I think you just have to assume that people are going to make
this mistake from time to time.

Tom