bug-gnulib@gnu.org mirror (unofficial)
 help / color / mirror / Atom feed
From: Zack Weinberg <zackw@panix.com>
To: Bruno Haible <bruno@clisp.org>
Cc: Autoconf <autoconf@gnu.org>, Gnulib bugs <bug-gnulib@gnu.org>
Subject: Re: gl_CACHE_VAL_SILENT won't work correctly with upcoming autoconf 2.70
Date: Fri, 13 Mar 2020 21:11:59 -0400	[thread overview]
Message-ID: <CAKCAbMjJUSuhv6LdiUZ+HFk=1s3CYKY891OyGKYmVFK4=8AJ3g@mail.gmail.com> (raw)
In-Reply-To: <1867344.WzrmtSNFgT@omega>

On Fri, Mar 13, 2020 at 5:54 PM Bruno Haible <bruno@clisp.org> wrote:
>
> [CCing autoconf@gnu.org]
>
> Hi Zack,
>
> > Abstractly, the best thing to do about this would be to remove the
> > macro and change all of its uses to be AC_CACHE_CHECK, with proper
> > messaging, instead.
>
> The point is that we don't want messaging for some tests. Most people
> find the 'configure' output already long enough.

 I disagree with this on principle.  If you're doing any test that's
complicated enough to be worth caching, then you should use
AC_CACHE_CHECK and provide messaging for it.  Yeah, most of the time
that text is just going to scroll by unread, but when something goes
wrong, the poor schmuck who has to debug it wants to know that it was
the test for `ceil` (for instance) that hung the build, not the
unrelated thing that happens to have been tested right before that.

> The underlying problem in Autoconf is the following: There is a macro
> AC_CACHE_CHECK that does messaging, and a macro AC_CACHE_VAL whose
> main purpose is to make a cache lookup, not messaging, but it
> nevertheless outputs '(cached)' strings occasionally.

So based on the above my attitude is that AC_CACHE_VAL (and
AC_MSG_CHECKING and AC_MSG_RESULT) exist only for backward
compatibility's sake and what you're supposed to be using is
AC_CACHE_CHECK, and I'm not inclined to spend any of my own hacking
time on "a macro that is like AC_CACHE_VAL but produces no spurious
output".

On the other hand, if you write a patch and Paul or Eric likes it, I
won't stand in its way, either.

zw


  parent reply	other threads:[~2020-03-14  1:12 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-13 20:57 gl_CACHE_VAL_SILENT won't work correctly with upcoming autoconf 2.70 Zack Weinberg
2020-03-13 21:54 ` Bruno Haible
2020-03-13 22:04   ` Nick Bowler
2020-03-13 23:30     ` Paul Eggert
2020-03-14  0:27       ` Bruno Haible
2020-03-14  1:27       ` Nick Bowler
2020-03-14  1:11   ` Zack Weinberg [this message]
2020-03-14  2:49     ` debugging autoconf tests (was: gl_CACHE_VAL_SILENT) Bruno Haible
2020-03-14 14:06       ` Zack Weinberg

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://lists.gnu.org/mailman/listinfo/bug-gnulib

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

  git send-email \
    --in-reply-to='CAKCAbMjJUSuhv6LdiUZ+HFk=1s3CYKY891OyGKYmVFK4=8AJ3g@mail.gmail.com' \
    --to=zackw@panix.com \
    --cc=autoconf@gnu.org \
    --cc=bruno@clisp.org \
    --cc=bug-gnulib@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.
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).