public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Yann Droneaud <ydroneaud@opteya.com>
To: Florian Weimer <fweimer@redhat.com>
Cc: libc-alpha@sourceware.org
Subject: Re: Compatibility .so linker scripts for merged libraries
Date: Wed, 17 Nov 2021 22:32:56 +0100	[thread overview]
Message-ID: <8fd56fb7-4379-77c7-43f3-76950e77390e@opteya.com> (raw)
In-Reply-To: <87ee7f845c.fsf@oldenburg.str.redhat.com>

Hi,

Le 17/11/2021 à 11:31, Florian Weimer a écrit :
> * Yann Droneaud:
>
>> Was it considered dangerous to introduce "compatibility" .so link
>> scripts for the libraries that was merged into libc.so (libpthread,
>> librt, libdl, etc.) ?
>>
>>
>>
>> For example:
>> $ cat librt.so
>> /* GNU ld script
>>     Use the static library */
>> OUTPUT_FORMAT(elf64-x86-64)
>> GROUP ( /lib/x86_64-linux-gnu/librt.a )
>>
>>
>> Because I'm having some bad times fixing issues in a build system that
>> try to locate now missing .so with gcc -print-file-name= then uses the
>> paths to the libraries instead of -l name them.
> Which build system is that?  This seems to be rather uncommon.

It's surely is uncommon, and need to be fixed.

Adding fake .so would have, in my mind, buying some time to upgrade. But 
this is pointless if no other projects are having the same issue.

> Adding non-loadable .so files tends to break other things.

I was afraid you said that, but I expected it :)

With is piece of information, I have enough motivation to request a fix 
on the build system the project I work for is using.

> We have seen some breakage as well due to the merging, but they have
> been clear bugs in the build systems.  Like testing whether linking
> against -lpthread is needed to make pthread_create available, and then
> using that flag (the need for -lpthread) as an indicator whether
> threading is available and pthread_create should be used.  But the
> -print-file-name= stuff is new to me.


Regards.

-- 

Yann Droneaud

OPTEYA



      reply	other threads:[~2021-11-17 21:33 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-11-16 13:27 Yann Droneaud
2021-11-16 13:47 ` Andreas Schwab
2021-11-16 13:54   ` Yann Droneaud
2021-11-17 10:31 ` Florian Weimer
2021-11-17 21:32   ` Yann Droneaud [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=8fd56fb7-4379-77c7-43f3-76950e77390e@opteya.com \
    --to=ydroneaud@opteya.com \
    --cc=fweimer@redhat.com \
    --cc=libc-alpha@sourceware.org \
    /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).