Re: [PATCH v5 0/4] Update gnulib and drop some patches

[Daniel Kiper]

On Tue, Feb 22, 2022 at 12:35:00PM -0500, Robbie Harwood wrote:
> Daniel Kiper <dkiper@net-space.pl> writes:
>
> > Robbie Harwood wrote:
> >
> >> I don't get that, actually.  What OS/gcc?  It's fine with the gcc-11
> >> I'm using here.
> >
> > Does gcc-11 work with the configure mentioned above?
> >
> > Anyway, Debian 9, arm-linux-gnueabihf-gcc (Debian 6.3.0-18) 6.3.0 20170516.
>
> Yes, with arm-linux-gnueabihf-gcc (Debian 11.2.0-16) 11.2.0.
>
> > The GRUB INSTALL file says:
> >
> >   The Requirements
> >   ================
> >
> >   GRUB depends on some software packages installed into your system. If
> >   you don't have any of them, please obtain and install them before
> >   configuring the GRUB.
> >
> >   * GCC 5.1.0 or later
> >
> > So, even if it is old it is perfectly valid compiler according to the
> > GRUB docs.
> >
> > Hmmm... What are the latest gnulib minimal compiler, linker,
> > etc. requirements?
>
> It doesn't look formally specified anywhere, beyond what the autotools
> requirements are.  I suspect that means in practice it's what the devs
> are using.  Also, gnulib's position on sign comparison issues is to
> ignore them, so the "fix" from our end would probably be to pragma them

Huh! Where did you see that? Could you point me a source where they say
that thing?

> away... but that assumes this is the only issue, which hasn't been
> checked yet.

I am not sure what you mean here.

Anyway, I do not fully understand why this works with newer GCC version
and does not work with older ones.

Daniel