public inbox for newlib@sourceware.org
 help / color / mirror / Atom feed
From: Corinna Vinschen <vinschen@redhat.com>
To: newlib@sourceware.org
Subject: Re: newlib: require autoconf-2.69
Date: Mon, 10 Jan 2022 10:07:25 +0100	[thread overview]
Message-ID: <Ydv3TUne1nYlizYF@calimero.vinschen.de> (raw)
In-Reply-To: <YdiQIay4DJEZzMhv@vapier>

On Jan  7 14:10, Mike Frysinger wrote:
> On 07 Jan 2022 10:56, Corinna Vinschen wrote:
> > On Jan  5 20:46, Mike Frysinger wrote:
> > > On 05 Jan 2022 13:10, Corinna Vinschen wrote:
> > > > On Dec 30 13:39, Mike Frysinger wrote:
> > > > > The top-level newlib dir is already using autoconf-2.69, and all the
> > > > > subdirs are using autoconf-2.68.  Update them all to use 2.69 as a
> > > > > prereq and then regenerate them accordingly.
> > > > > 
> > > > > The README file already says to use autoconf-2.69.
> > > > 
> > > > Yeah, but...
> > > > 
> > > > The AC_PREREQ only checks for the minimum required autoconf version for
> > > > *this* specific autoconf file.  AC_PREREQ([2.52]) doesn't mean we have to
> > > > use 2.52, but any later version will suffice.  All current autoconf
> > > > files are compatible with 2.69.  Neither binutils-gdb, nor gcc changed
> > > > their files to require 2.69.  So it's note quite clear to me what this
> > > > patch accomplishes.
> > > 
> > > that's incorrect.  gcc & binutils-gdb have config/override.m4 which requires
> > > autoconf-2.69 exactly is used.  no older versions, no newer versions, only
> > > 2.69 is permitted.
> > 
> > You're right, but that's not what I was up to.  Requiring 2.69 doesn't
> > mean we have to (or even should) change AC_PREREQ in every single file.
> > But, anyway, it probably won't hurt either, so just go ahead.
> 
> that's true.  but then i'd question what's the point of AC_PREREQ in the first
> place.  if we move to override.m4, maybe we delete all the AC_PREREQ calls ?
> -mike

I think it's ok to leave the files alone, unless a change is really
required.  The fact that they claim to run even under, say, autoconf
2.50, is still true and doesn't hurt the build system.  If it hurts the
build system because they use an outdated autoconf expression, then a
fix should also bump AC_PREREQ.

Of course, if binutils-gdb decide differently, we can follow suit.


Corinna


  reply	other threads:[~2022-01-10  9:07 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-30 18:39 Mike Frysinger
2022-01-05 12:10 ` Corinna Vinschen
2022-01-05 13:04   ` R. Diez
2022-01-06  1:46   ` Mike Frysinger
2022-01-06  6:10     ` Mike Frysinger
2022-01-07  9:58       ` Corinna Vinschen
2022-01-07 19:09         ` Mike Frysinger
2022-01-10  9:02           ` Corinna Vinschen
2022-01-11  0:58             ` Mike Frysinger
2022-01-11 10:23               ` Corinna Vinschen
2022-01-12  0:52                 ` Mike Frysinger
2022-01-12 10:19                   ` Corinna Vinschen
2022-01-12 17:24                     ` Mike Frysinger
2022-01-07  9:56     ` Corinna Vinschen
2022-01-07 19:10       ` Mike Frysinger
2022-01-10  9:07         ` Corinna Vinschen [this message]
2022-01-12 18:24 ` require autoconf-2.69 exactly Mike Frysinger
2022-01-12 20:01   ` R. Diez
2022-01-12 21:37     ` Mike Frysinger
2022-01-14 10:12   ` Corinna Vinschen

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=Ydv3TUne1nYlizYF@calimero.vinschen.de \
    --to=vinschen@redhat.com \
    --cc=newlib@sourceware.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).