public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Tom de Vries <tdevries@suse.de>
To: Simon Marchi <simark@simark.ca>,
	Bernd Edlinger <bernd.edlinger@hotmail.de>,
	"gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
Cc: Tom Tromey <tom@tromey.com>, Alan Hayward <Alan.Hayward@arm.com>,
	Kevin Buettner <kevinb@redhat.com>, Eli Zaretskii <eliz@gnu.org>
Subject: Re: [PATCH] Fix build with GNU Make 3.81
Date: Wed, 15 Jul 2020 20:30:15 +0200	[thread overview]
Message-ID: <12ab0278-62ba-6a47-5227-cad0e97aa87e@suse.de> (raw)
In-Reply-To: <071c6225-0704-8e1f-df78-8a574bfff449@simark.ca>

On 7/15/20 3:44 PM, Simon Marchi wrote:
> On 2020-07-15 4:19 a.m., Tom de Vries wrote:
>> I don't follow this reasoning.
>>
>> There's a bug in make 3.81, which we run into with the gdb build.
>>
>> We then require make 3.82, such that gdb builds again for all supported
>> make versions.
>>
>> Subsequently (that is, 3 days later) we work around the bug in make 3.81
>> with a gdb commit.
>>
>> But we decide to still require make 3.82, even if the original reason to
>> do so is no longer valid, and I'm assuming we haven't started using make
>> 3.82 features in the 3 days inbetween.
>>
>> Are there other considerations at play here, that are not being made
>> explicit?
>>
>> Because if there are not, we should update NEWS to say we support 3.81
>> again.
>>
>> Thanks,
>> - Tom
>>
> 
> Hi Tom,
> 
> Requiring make 3.82 was based on the assumption that pretty much all supported
> versions of major distros had 3.82 bundled (which turned out false for SLES11).

Yeah.  And looking here (
https://en.wikipedia.org/wiki/SUSE_Linux_Enterprise_Server#End-of-support_schedule
), that's still supported till 31 March 2022.

> So the simplest thing to do when it was found that GDB wouldn't build with 3.81
> was just to say "don't use 3.81", rather than trying to work around the bug.
> 

Right, that made sense at that point.

> Even with the claim that we require make 3.82, if somebody provides a simple fix
> (like Bernd did) to make it work again with 3.81, then I'm happy to merge it.  It
> makes his life easier, and it doesn't affect mine.
> 

Agreed.

> If we claim in NEWS that we support 3.81 again, then we ideally have to test it
> regularly and take care of any breakage.

Test it, yes.  Take care of any breakage, if it's reasonable for
upstream sources.  If not, we always have the option of requiring 3.82
at that point.

> That's not something I want to do, but
> if it is something you need to support (because you need to support SLES11) and
> are ready to take care of that support, then I have nothing against it.

Sure, I can set make 3.81 as my default make for development builds, and
I should at least encounter any breakage that way.

Anyways, I understand now, the decision to move to 3.82 was triggered by
the 3.81 build breaker, but based on the assumption that 3.81 was not
provided by any of the major distros so nobody would need to test and
support it, and when the build breaker got fixed, that reasoning didn't
change.

It all makes sense to me now, thanks for elaborating.

Thanks,
- Tom

      reply	other threads:[~2020-07-15 18:30 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-14  8:15 Bernd Edlinger
2019-12-14 18:21 ` Simon Marchi
2019-12-19 17:29   ` Tom Tromey
2019-12-23  7:30     ` Joel Brobecker
2020-07-15  8:19   ` Tom de Vries
2020-07-15 13:44     ` Simon Marchi
2020-07-15 18:30       ` Tom de Vries [this message]

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=12ab0278-62ba-6a47-5227-cad0e97aa87e@suse.de \
    --to=tdevries@suse.de \
    --cc=Alan.Hayward@arm.com \
    --cc=bernd.edlinger@hotmail.de \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=kevinb@redhat.com \
    --cc=simark@simark.ca \
    --cc=tom@tromey.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.
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).