[1/1] libcap: fix static linking issue

Submitted by Romain Naour on Oct. 6, 2013, 8:10 p.m.

Details

Message ID 5251C3B0.7090105@openwide.fr
State Superseded
Headers show

Commit Message

Romain Naour Oct. 6, 2013, 8:10 p.m.
Add -fPIC flag

Fixes:
http://autobuild.buildroot.net/results/619bceb3491ecd2ed4e1ae552fdb237a0ed2fa47/

Signed-off-by: Romain Naour <romain.naour@openwide.fr>
---
 package/libcap/libcap.mk | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

Comments

Thomas Petazzoni Oct. 6, 2013, 9:29 p.m.
Dear Romain Naour,

On Sun, 06 Oct 2013 22:10:24 +0200, Romain Naour wrote:
> Add -fPIC flag
> 
> Fixes:
> http://autobuild.buildroot.net/results/619bceb3491ecd2ed4e1ae552fdb237a0ed2fa47/
> 
> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
> ---
>  package/libcap/libcap.mk | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/package/libcap/libcap.mk b/package/libcap/libcap.mk
> index 16afaef..6248fed 100644
> --- a/package/libcap/libcap.mk
> +++ b/package/libcap/libcap.mk
> @@ -25,10 +25,15 @@ endif
>  # we don't have host-attr
>  HOST_LIBCAP_DEPENDENCIES =
> 
> +ifeq ($(BR2_PREFER_STATIC_LIB),y)
> +LIBCAP_LDFLAGS=-fPIC
> +endif

This seems really weird, because normally, we would expect exactly the
opposite: position independent code is needed for shared libraries, not
for static libraries.

Thomas
Romain Naour Oct. 8, 2013, 8:11 a.m.
Hi Thomas,

----- Mail original -----
| De: "Thomas Petazzoni" <thomas.petazzoni@free-electrons.com>
| À: "Romain Naour" <romain.naour@openwide.fr>
| Cc: "buildroot" <buildroot@busybox.net>
| Envoyé: Dimanche 6 Octobre 2013 23:29:54
| Objet: Re: [Buildroot] [PATCH 1/1] libcap: fix static linking issue
| 
| Dear Romain Naour,
| 
| On Sun, 06 Oct 2013 22:10:24 +0200, Romain Naour wrote:
| > Add -fPIC flag
| > 
| > Fixes:
| > http://autobuild.buildroot.net/results/619bceb3491ecd2ed4e1ae552fdb237a0ed2fa47/
| > 
| > Signed-off-by: Romain Naour <romain.naour@openwide.fr>
| > ---
| >  package/libcap/libcap.mk | 7 ++++++-
| >  1 file changed, 6 insertions(+), 1 deletion(-)
| > 
| > diff --git a/package/libcap/libcap.mk b/package/libcap/libcap.mk
| > index 16afaef..6248fed 100644
| > --- a/package/libcap/libcap.mk
| > +++ b/package/libcap/libcap.mk
| > @@ -25,10 +25,15 @@ endif
| >  # we don't have host-attr
| >  HOST_LIBCAP_DEPENDENCIES =
| > 
| > +ifeq ($(BR2_PREFER_STATIC_LIB),y)
| > +LIBCAP_LDFLAGS=-fPIC
| > +endif
| 
| This seems really weird, because normally, we would expect exactly
| the
| opposite: position independent code is needed for shared libraries,
| not
| for static libraries.

Ok, I'll take the problem from the beginning.
Sorry for that mistake.

Best regards,
Romain
| 
| Thomas
| --
| Thomas Petazzoni, Free Electrons
| Embedded Linux, Kernel and Android engineering
| http://free-electrons.com
|
Thomas Petazzoni Oct. 8, 2013, 8:13 a.m.
Dear Romain Naour,

On Tue, 8 Oct 2013 10:11:53 +0200 (CEST), Romain Naour wrote:

> | > +ifeq ($(BR2_PREFER_STATIC_LIB),y)
> | > +LIBCAP_LDFLAGS=-fPIC
> | > +endif
> | 
> | This seems really weird, because normally, we would expect exactly
> | the
> | opposite: position independent code is needed for shared libraries,
> | not
> | for static libraries.
> 
> Ok, I'll take the problem from the beginning.
> Sorry for that mistake.

Note that I'm not sure about what I said, but having to build with
-fPIC in the static lib case seems really strange.

Thanks!

Thomas
Romain Naour Oct. 13, 2013, 7:44 p.m.
Hi,

Le 08/10/2013 10:13, Thomas Petazzoni a écrit :
> Dear Romain Naour,
> 
> On Tue, 8 Oct 2013 10:11:53 +0200 (CEST), Romain Naour wrote:
> 
>> | > +ifeq ($(BR2_PREFER_STATIC_LIB),y)
>> | > +LIBCAP_LDFLAGS=-fPIC
>> | > +endif
>> | 
>> | This seems really weird, because normally, we would expect exactly
>> | the
>> | opposite: position independent code is needed for shared libraries,
>> | not
>> | for static libraries.
>>
>> Ok, I'll take the problem from the beginning.
>> Sorry for that mistake.
> 
> Note that I'm not sure about what I said, but having to build with
> -fPIC in the static lib case seems really strange.
> 
> Thanks!
> 
> Thomas
> 

You're right, the static lib isn’t built with -fPIC.
But the libcap build system build the static and shared libraries,
even if you don't want one of them.

The build failure [1] is caused by the missing -fPIC during the build of libcap.so.2.22

So, we may add -fPIC if BR2_PREFER_STATIC_LIB=y and the shared lib is build correctly
(ok it's weird)
Or add a patch for enable/disable separately the static/shared build.

Best regards,
Romain Naour

[1] http://autobuild.buildroot.net/results/619bceb3491ecd2ed4e1ae552fdb237a0ed2fa47/
Thomas Petazzoni Oct. 16, 2013, 4:09 p.m.
Dear Romain Naour,

On Sun, 13 Oct 2013 21:44:08 +0200, Romain Naour wrote:

> You're right, the static lib isn’t built with -fPIC.
> But the libcap build system build the static and shared libraries,
> even if you don't want one of them.
> 
> The build failure [1] is caused by the missing -fPIC during the build of libcap.so.2.22
> 
> So, we may add -fPIC if BR2_PREFER_STATIC_LIB=y and the shared lib is build correctly
> (ok it's weird)
> Or add a patch for enable/disable separately the static/shared build.

If BR2_PREFER_STATIC_LIB=y, then not shared library should be built at
all. Even though the naming of the option doesn't make it clear for the
moment.

And therefore, when BR2_PREFER_STATIC_LIB=y, passing -fPIC should not
be needed, as we're building only static libraries.

I had a quick look at libcap, and we can easily build only the static
or shared library by doing the $(MAKE) only in $(@D)/libcap, and using
the libcap.a and libcap.so targets. But the install target is not really
adapted to that, so we would have to install things manually.

Maybe a little patch to the libcap/Makefile which understands
BUILD_STATIC and BUILD_SHARED variables would be good.

Best regards,

Thomas
Thomas Petazzoni Nov. 1, 2013, 5:48 p.m.
Dear Romain Naour,

On Sun, 06 Oct 2013 22:10:24 +0200, Romain Naour wrote:
> Add -fPIC flag
> 
> Fixes:
> http://autobuild.buildroot.net/results/619bceb3491ecd2ed4e1ae552fdb237a0ed2fa47/
> 
> Signed-off-by: Romain Naour <romain.naour@openwide.fr>

I've just sent a patch which I believe is a better fix than this one.
The problem was that libcap was building the shared library
unconditionally. By adjusting the libcap build system, we can make it
build only the static library, and therefore make things work properly.

Best regards,

Thomas
Romain Naour Nov. 2, 2013, 3:52 p.m.
Le 01/11/2013 18:48, Thomas Petazzoni a écrit :
> Dear Romain Naour,
>
> On Sun, 06 Oct 2013 22:10:24 +0200, Romain Naour wrote:
>> Add -fPIC flag
>>
>> Fixes:
>> http://autobuild.buildroot.net/results/619bceb3491ecd2ed4e1ae552fdb237a0ed2fa47/
>>
>> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
> I've just sent a patch which I believe is a better fix than this one.
> The problem was that libcap was building the shared library
> unconditionally. By adjusting the libcap build system, we can make it
> build only the static library, and therefore make things work properly.
>
> Best regards,
>
> Thomas
Hi Thomas,

I haven't take time to rework on this path :-/
Thanks.

Best regards,
Romain

Patch hide | download patch | download mbox

diff --git a/package/libcap/libcap.mk b/package/libcap/libcap.mk
index 16afaef..6248fed 100644
--- a/package/libcap/libcap.mk
+++ b/package/libcap/libcap.mk
@@ -25,10 +25,15 @@  endif
 # we don't have host-attr
 HOST_LIBCAP_DEPENDENCIES =

+ifeq ($(BR2_PREFER_STATIC_LIB),y)
+LIBCAP_LDFLAGS=-fPIC
+endif
+
 define LIBCAP_BUILD_CMDS
 	$(TARGET_MAKE_ENV) $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D) \
 		LIBATTR=$(LIBCAP_HAVE_LIBATTR) BUILD_CC="$(HOSTCC)" \
-		BUILD_CFLAGS="$(HOST_CFLAGS)"
+		BUILD_CFLAGS="$(HOST_CFLAGS)" \
+		LDFLAGS="$(LIBCAP_LDFLAGS)"
 endef

 define LIBCAP_INSTALL_STAGING_CMDS