[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Drop gnulib fix-base64.patch
From: |
Daniel Kiper |
Subject: |
Re: [PATCH] Drop gnulib fix-base64.patch |
Date: |
Thu, 25 Nov 2021 18:09:04 +0100 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Wed, Nov 24, 2021 at 09:36:14AM -0500, Robbie Harwood wrote:
> Daniel Kiper <dkiper@net-space.pl> writes:
>
> > CC-ing Daren, Patrick and Vladimir...
> >
> > When I started looking at this issue I realized we have bigger problem
> > here than lack of _GL_ATTRIBUTE_CONST definition. In general all _GL_*
> > constants land in config-util.h.in and finally in config-util.h. It
> > does not make a lot of sense because config-util.h is not included
> > when you build GRUB core. Instead it is included when you build GRUB
> > utils. Huh! AFAICT this means the gnulib is deprived of all its
> > specific constants. I am not sure how it worked at all in so
> > far. Anyway, I think we have to fix it properly.
> >
> > Robbie, may I ask you to talk to gnulib folks and ask them how we
> > should solve this problem? Please CC all folks CC-ed in this email.
>
> I'm not opposed to getting gnulib's opinion, but don't we already know
> what they suggest from
> https://lists.gnu.org/archive/html/bug-gnulib/2021-11/msg00009.html
> (i.e., run configure twice)?
Yeah, but I think it would require major overhaul. Does not it? If yes
then maybe we should consider move to the Kconfig or something like that.
> (Though if other folks on CC want to provide more input on
> https://lists.gnu.org/archive/html/bug-gnulib/2021-11/msg00028.html that
> would be helpful.)
Yeah, that would be perfect.
Daniel