all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Paul Eggert <eggert@cs.ucla.edu>
To: Glenn Morris <rgm@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: master 38fb5f4: Make make-dist more automatic
Date: Sat, 2 Feb 2019 19:27:02 -0800	[thread overview]
Message-ID: <f9567472-b6fd-d9ac-04a7-e7ac405c5969@cs.ucla.edu> (raw)
In-Reply-To: <xfpnsaj8vc.fsf@fencepost.gnu.org>

Glenn Morris wrote:
> I like the simplification, but I have some issues with the MANIFEST file.

Yes, it's a bit of a wart.

> 1) I don't think it should be linked to --no-update. The MANIFEST file
> is fundamental to creating a tar file and it needs to be up-to-date.
> As it stands, if you use --no-update you can easily end up with a stale
> MANIFEST file and a broken tar file. If it does need to be optional,
> please can it have a separate eg --no-update-manifest option.

It would be easy to change make-dist to rebuild MANIFEST regardless of 
--no-update, and I don't see any problem with that.

> 2) Running make-dist leaves a top-level MANIFEST file that nothing ever
> deletes. Since in the vast majority of the time this file will be an
> automatically generated file, I think it would be better to use a
> temporary file. This also takes care of point 1.
> 
> Here's a possible patch.
> This excludes itself MANIFEST from the tar file.
> It could be re-added, but I don't know what the point would be.

The point of shipping MANIFEST is to handle the following situation:

1. User gets an Emacs tarball and extracts it.

2. User changes the source code.

2. User runs "make dist".

Without the MANIFEST file, step (3) would fail because no Git repository is 
available to tell make-dist the names of most of the source files.

If it's OK for us to say, "You must have a Git repository in order to make a 
distribution" then we can dispense with shipping a MANIFEST file; we might even 
be able to dispense from generate a manifest file at all, even a temporary one. 
But if we want to decouple having Git from the ability to make a distribution, 
we'll need something like a MANIFEST file (unless we want to go back to the old 
approach of listing files in make-dist itself, which was error-prone).



  reply	other threads:[~2019-02-03  3:27 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20190201234220.19088.3350@vcs0.savannah.gnu.org>
     [not found] ` <20190201234221.F285C205D8@vcs0.savannah.gnu.org>
2019-02-02 19:19   ` master 38fb5f4: Make make-dist more automatic Glenn Morris
2019-02-03  3:27     ` Paul Eggert [this message]
2019-02-05  3:54       ` Glenn Morris

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=f9567472-b6fd-d9ac-04a7-e7ac405c5969@cs.ucla.edu \
    --to=eggert@cs.ucla.edu \
    --cc=emacs-devel@gnu.org \
    --cc=rgm@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.