| Message ID | 20230223175634.1073050-1-richard.purdie@linuxfoundation.org |
|---|---|
| State | Accepted, archived |
| Commit | ccd28c418ab8390118d738fbe914395b5c2a1f75 |
| Headers | show |
| Series | binutils: Fix nativesdk ld.so search | expand |
Thanks for the patch it seems ok to me At least it’s not poking at build machine for ld.so.conf anymore On Thu, Feb 23, 2023 at 9:56 AM Richard Purdie < richard.purdie@linuxfoundation.org> wrote: > Currently binutils in buildtools is searching for /etc/etc/ld.so.conf > which makes no sense. ld_sysconfdir already contains /etc so we need to > drop the /etc from the fixed string. > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> > --- > ...3-binutils-nativesdk-Search-for-alternative-ld.so.conf.patch | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git > a/meta/recipes-devtools/binutils/binutils/0003-binutils-nativesdk-Search-for-alternative-ld.so.conf.patch > b/meta/recipes-devtools/binutils/binutils/0003-binutils-nativesdk-Search-for-alternative-ld.so.conf.patch > index 8e5e2be81dd..54967b9362f 100644 > --- > a/meta/recipes-devtools/binutils/binutils/0003-binutils-nativesdk-Search-for-alternative-ld.so.conf.patch > +++ > b/meta/recipes-devtools/binutils/binutils/0003-binutils-nativesdk-Search-for-alternative-ld.so.conf.patch > @@ -65,7 +65,7 @@ index eff6693e052..95787012b89 100644 > info.path = NULL; > info.len = info.alloc = 0; > - tmppath = concat (ld_sysroot, prefix, "/etc/ld.so.conf", > -+ tmppath = concat (ld_sysconfdir, "/etc/ld.so.conf", > ++ tmppath = concat (ld_sysconfdir, "/ld.so.conf", > (const char *) NULL); > if (!ldelf_parse_ld_so_conf (&info, tmppath)) > { > -- > 2.37.2 > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#177618): > https://lists.openembedded.org/g/openembedded-core/message/177618 > Mute This Topic: https://lists.openembedded.org/mt/97188650/1997914 > Group Owner: openembedded-core+owner@lists.openembedded.org > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [ > raj.khem@gmail.com] > -=-=-=-=-=-=-=-=-=-=-=- > >
diff --git a/meta/recipes-devtools/binutils/binutils/0003-binutils-nativesdk-Search-for-alternative-ld.so.conf.patch b/meta/recipes-devtools/binutils/binutils/0003-binutils-nativesdk-Search-for-alternative-ld.so.conf.patch index 8e5e2be81dd..54967b9362f 100644 --- a/meta/recipes-devtools/binutils/binutils/0003-binutils-nativesdk-Search-for-alternative-ld.so.conf.patch +++ b/meta/recipes-devtools/binutils/binutils/0003-binutils-nativesdk-Search-for-alternative-ld.so.conf.patch @@ -65,7 +65,7 @@ index eff6693e052..95787012b89 100644 info.path = NULL; info.len = info.alloc = 0; - tmppath = concat (ld_sysroot, prefix, "/etc/ld.so.conf", -+ tmppath = concat (ld_sysconfdir, "/etc/ld.so.conf", ++ tmppath = concat (ld_sysconfdir, "/ld.so.conf", (const char *) NULL); if (!ldelf_parse_ld_so_conf (&info, tmppath)) {
Currently binutils in buildtools is searching for /etc/etc/ld.so.conf which makes no sense. ld_sysconfdir already contains /etc so we need to drop the /etc from the fixed string. Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> --- ...3-binutils-nativesdk-Search-for-alternative-ld.so.conf.patch | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)