unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Alan Mackenzie <acm@muc.de>
To: Paul Eggert <eggert@cs.ucla.edu>
Cc: noloader@gmail.com, Eli Zaretskii <eliz@gnu.org>, emacs-devel@gnu.org
Subject: Re: make check fails due to missing test directory
Date: Sat, 4 May 2019 09:24:06 +0000	[thread overview]
Message-ID: <20190504092406.GA4139@ACM> (raw)
In-Reply-To: <e3851b52-4267-7189-a0bc-98baeb9a8c69@cs.ucla.edu>

Hello, Paul

On Fri, May 03, 2019 at 15:01:58 -0700, Paul Eggert wrote:
> On 5/3/19 12:13 PM, Eli Zaretskii wrote:
> > You need to download the test suite.  It is not part of the release
> > tarball.  You can find it in the Emacs development repository.

> Sure, but as his email suggests, it's better for 'make check' to succeed
> when the set of tests is empty. That way, standardized build procedures
> (like his) are more likely to do the right thing. I installed the patch
> 0001 (attached) into the master branch to do that.

> We've talked before about distributing tests in the Emacs tarball -
> which is what the GNU Codings Standards recommend and what pretty much
> every other GNU package does - and this email suggests that now's a good
> time to do that. Distributing tests adds about 2.5% to the size of the
> compressed tarball, and nowadays that is a small price to pay for making
> tests available to people building from a release tarball rather than a
> Git clone. Proposed patch attached as patch 0002, and comments welcome.

The tests in make check are development tests.  They're not build tests.
In fact, they're fairly useless for anybody who isn't developing Emacs.
Somebody who is developing Emacs will have the git repository.

Therefore, we shouldn't be encouraging users to run these tests, just as
we don't encourage users to byte compile the .el files.  Both these
things have been done as part of the release process, and both are
wastes of time for users.  I, personally, would certainly not run these
tests after downloading an Emacs release.

So I think, if we're going to include these tests in releases, there
should be a strong recommendation in the docs NOT to run them, that
they're only there for people with special needs.

-- 
Alan Mackenzie (Nuremberg, Germany).



  reply	other threads:[~2019-05-04  9:24 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-03 18:41 make check fails due to missing test directory Jeffrey Walton
2019-05-03 19:13 ` Eli Zaretskii
2019-05-03 22:01   ` Paul Eggert
2019-05-04  9:24     ` Alan Mackenzie [this message]
2019-05-04 15:56       ` Paul Eggert
2019-05-04 16:44       ` Michael Albinus
2019-05-16 17:54     ` Emacs distribution tarball now supports 'make check' Paul Eggert

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=20190504092406.GA4139@ACM \
    --to=acm@muc.de \
    --cc=eggert@cs.ucla.edu \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=noloader@gmail.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).