Re: [PATCH 0/4] Do not install shared objects under versioned names

[Carlos O'Donell <carlos@redhat.com>]

On 6/10/21 4:22 AM, Florian Weimer via Libc-alpha wrote:
> This is essentially a repost of the “Add --disable-major-minor-libraries
> configure option” series.  Joseph suggested that the configure option is
> not needed, so this version implements the change unconditionally.

This is absolutely the right direction.

I like that we simplify the layout.

I have commented on Siddhesh's thoughts in the other thread, and I think
we are never likely to implement a solution that installs parallel libcs
given the QE and container aspects.

Therefore I'd like to see this move forward.
 
> Tested on i686-linux-gnu and x86_64-linux-gnu.  I compared two
> build-many-glibcs.py trees with and without these patches, using this
> command to see if there are missing files besides the versioned DSOs or
> any dangling symbolic links.
> 
> cd /home/bmg/install/glibcs && find -printf '%P\n' \
>   | while read x ; do
>     test -r /home/bmg-install-glibcs/$x || echo $x
>   done \
>   | grep -v '\-2\.33\.9000\.so$' | grep -v '/libthread_db-1\.0\.so$'
> 
> /home/bmg/install/glibcs is the unpatched build,
> /home/bmg-install-glibcs is the build with the patches applied.  As
> expected, there was no output.
> 
> Thanks,
> Florian
> 
> Florian Weimer (4):
>   nptl_db: Install libthread_db under a regular implementation name
>   Makerules: Remove lib-version, $(subdir-version)
>   elf: Generalize name-based DSO recognition in ldconfig
>   Install shared objects under their ABI names
> 
>  INSTALL             | 10 +++++++
>  Makefile            |  6 ----
>  Makerules           | 67 +++++----------------------------------------
>  NEWS                |  8 ++++++
>  elf/Makefile        | 12 ++------
>  elf/dl-is_dso.h     | 33 ++++++++++++++++++++++
>  elf/ldconfig.c      |  5 ++--
>  elf/tst-dl-is_dso.c | 35 +++++++++++++++++++++++
>  nptl_db/Makefile    |  2 --
>  9 files changed, 98 insertions(+), 80 deletions(-)
>  create mode 100644 elf/dl-is_dso.h
>  create mode 100644 elf/tst-dl-is_dso.c
> 


-- 
Cheers,
Carlos.