From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Guillaume Smaha <guillaume.smaha@gmail.com>
Cc: buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH] package/glibc: fix usage of BR2_PACKAGE_GLIBC_KERNEL_COMPAT
Date: Fri, 2 Sep 2022 17:48:22 +0200 [thread overview]
Message-ID: <20220902154822.GN68979@scaer> (raw)
In-Reply-To: <CANf9tWYT3MZGq1UKDQN95X=L077Tuu8vRjOn1E5ek7ZLvjX1kA@mail.gmail.com>
Guillaume, All,
[Guillaume: please keep the list in Cc when you reply]
On 2022-09-02 14:20 +0200, Guillaume Smaha spake thusly:
> Thanks for the explanation !
> You are right, I got confused that glibc will enable all the backward
> compatibilitywithotu the flag.
No problem, thans for the feedback.
I've consequently marked the patch as rejected in our patchwork.
Regards,
Yann E. MORIN.
> On Fri, Sep 2, 2022 at 1:30 PM Yann E. MORIN < [1]yann.morin.1998@free.fr> wrote:
>
> Guillaume, All,
>
> On 2022-09-02 12:34 +0200, Guillaume Smaha spake thusly:
> > As explain in the commit 525ffc2bb17fa11fc128124d2d3a92d8698eaab8:
> > "Say 'y' here if you plan on running your system on a kernel older than
> > the version used for the toolchain headers."
> >
> > Signed-off-by: Guillaume Smaha < [2]guillaume.smaha@gmail.com>
> > ---
> > package/glibc/ [3]glibc.mk | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/package/glibc/ [4]glibc.mk b/package/glibc/ [5]glibc.mk
> > index 5af539ca62..4e4417b45a 100644
> > --- a/package/glibc/ [6]glibc.mk
> > +++ b/package/glibc/ [7]glibc.mk
> > @@ -98,7 +98,7 @@ endif
> > GLIBC_MAKE = $(BR2_MAKE)
> > GLIBC_CONF_ENV += ac_cv_prog_MAKE="$(BR2_MAKE)"
> >
> > -ifeq ($(BR2_PACKAGE_GLIBC_KERNEL_COMPAT),)
> > +ifeq ($(BR2_PACKAGE_GLIBC_KERNEL_COMPAT),y)
>
> You may have gotten a bit confused, but the curent situation is exactly
> correct.
>
> Indeed, when configured _without_ passing any --enable-kernel=X.Y.Z
> value, then glibc will enable *all* the backward compatibility wrappers.
>
> However, when configured with --enable-kernel=X.Y.Z, then only those
> wrappers needed for being compatible with that version of the kernel
> will be used.
>
> So, indeed, we do not want to pass --enable-kernel=X.Y.Z when
> BR2_PACKAGE_GLIBC_KERNEL_COMPAT is set.
>
> For more information, see:
> [8]https://www.gnu.org/software/libc/manual/html_mono/libc.html#Configuring-and-compiling
> [9]https://github.com/bminor/glibc/blob/master/configure.ac#L273
> [10]https://github.com/bminor/glibc/blob/master/sysdeps/unix/sysv/linux/configure.ac#L30
>
> Regards,
> Yann E. MORIN.
>
> > GLIBC_CONF_OPTS += --enable-kernel=$(call qstrip,$(BR2_TOOLCHAIN_HEADERS_AT_LEAST))
> > endif
> >
> > --
> > 2.34.1
> >
> > _______________________________________________
> > buildroot mailing list
> > [11]buildroot@buildroot.org
> > [12]https://lists.buildroot.org/mailman/listinfo/buildroot
>
> --
> .-----------------.--------------------.------------------.--------------------.
> | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___
> |
> | +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
> | [13]http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
> '------------------------------^-------^------------------^--------------------'
>
> --
> Guillaume Smaha
>
> Links:
> 1. mailto:yann.morin.1998@free.fr
> 2. mailto:guillaume.smaha@gmail.com
> 3. http://glibc.mk
> 4. http://glibc.mk
> 5. http://glibc.mk
> 6. http://glibc.mk
> 7. http://glibc.mk
> 8. https://www.gnu.org/software/libc/manual/html_mono/libc.html#Configuring-and-compiling
> 9. https://github.com/bminor/glibc/blob/master/configure.ac#L273
> 10. https://github.com/bminor/glibc/blob/master/sysdeps/unix/sysv/linux/configure.ac#L30
> 11. mailto:buildroot@buildroot.org
> 12. https://lists.buildroot.org/mailman/listinfo/buildroot
> 13. http://ymorin.is-a-geek.org/
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
prev parent reply other threads:[~2022-09-02 15:48 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-02 10:34 [Buildroot] [PATCH] package/glibc: fix usage of BR2_PACKAGE_GLIBC_KERNEL_COMPAT Guillaume Smaha
2022-09-02 11:30 ` Yann E. MORIN
[not found] ` <CANf9tWYT3MZGq1UKDQN95X=L077Tuu8vRjOn1E5ek7ZLvjX1kA@mail.gmail.com>
2022-09-02 15:48 ` Yann E. MORIN [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=20220902154822.GN68979@scaer \
--to=yann.morin.1998@free.fr \
--cc=buildroot@buildroot.org \
--cc=guillaume.smaha@gmail.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.