[systemd-devel] [PATCH] Makefile.am: reduce linked libraries

Koen Kooi koen at dominion.thruhere.net
Tue Oct 18 04:14:56 PDT 2011


Op 18 okt. 2011, om 12:51 heeft harald at redhat.com het volgende geschreven:

> From: Harald Hoyer <harald at redhat.com>
> 
> For libselinux and libcap the --as-needed linker flag does not seem to
> work. In fact, it seems to be turned off completly, if one of these
> libraries is used after --as-needed.
> 
> This patch reduces the amount of linked libraries.
> 
> before:
> $ ldd /lib/systemd/systemd-timestamp
> linux-vdso.so.1 (0x00007fffe95ff000)
> libselinux.so.1 => /lib64/libselinux.so.1 (0x000000356d000000)
> libcap.so.2 => /lib64/libcap.so.2 (0x000000356d400000)
> librt.so.1 => /lib64/librt.so.1 (0x000000356cc00000)
> libc.so.6 => /lib64/libc.so.6 (0x000000356b400000)
> /lib64/ld-linux-x86-64.so.2 (0x000000356ac00000)
> libdl.so.2 => /lib64/libdl.so.2 (0x000000356bc00000)
> libattr.so.1 => /lib64/libattr.so.1 (0x000000357b000000)
> libpthread.so.0 => /lib64/libpthread.so.0 (0x000000356c000000)
> 
> after:
> $ ldd systemd-timestamp
> linux-vdso.so.1 (0x00007fff4c333000)
> librt.so.1 => /lib64/librt.so.1 (0x000000356cc00000)
> libc.so.6 => /lib64/libc.so.6 (0x000000356b400000)
> /lib64/ld-linux-x86-64.so.2 (0x000000356ac00000)
> libpthread.so.0 => /lib64/libpthread.so.0 (0x000000356c000000)
> ---
> Makefile.am |   60 ++++++++++++++++++++++++++++++++++++++--------------------
> 1 files changed, 39 insertions(+), 21 deletions(-)
> 
> diff --git a/Makefile.am b/Makefile.am
> index dabe32a..76e5329 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> 
> -	$(CAP_LIBS)
> +        $(CAP_LIBS)

This replaces a tab with spaces and the rest of the patch is using a mixed mode of tab and space as well. What's preferred in the systemd makefiles, tabs or spaces?

regards,

Koen


More information about the systemd-devel mailing list