Bug 281385 – media-sound/gejengel installs a dummy static archive with libtool file

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Samuli Suominen (RETIRED) 2009-08-13 20:48:40 UTC]

There's no reason to install .a and .la. There could be some point if it would build a shared library and then link against it. Now it's just code duplication.

-rwxr-xr-x 1 root root  829232 2009-08-13 23:44 /usr/bin/gejengel
-rw-r--r-- 1 root root 2700264 2009-08-13 23:44 /usr/lib64/libgejengel.a
-rw-r--r-- 1 root root    3742 2009-08-13 23:44 /usr/lib64/libgejengel.la

[Comment 1 Markos Chandras (RETIRED) 2009-08-13 21:49:10 UTC]

True. I will try to come up within the next few days

[Comment 2 Samuli Suominen (RETIRED) 2009-08-14 08:25:28 UTC]

Line:

lib_LTLIBRARIES = libgejengel.la

Should likely be:

noinst_lib_LTLIBRARIES = libgejengel.la

in Makefile.am

(Untested.)

[Comment 3 Markos Chandras (RETIRED) 2009-10-02 13:49:15 UTC]

Samuli did you make any progress on this? I am not able to test this right now ( nor in the near future )

[Comment 4 Samuli Suominen (RETIRED) 2009-10-02 13:50:34 UTC]

(In reply to comment #3)
> Samuli did you make any progress on this? I am not able to test this right now
> ( nor in the near future )
> 

I was waiting for upstream response (since he's in this bug). That's the only reason why I haven't committed the change yet. I will do so now.

[Comment 5 Dirk Vanden Boer 2009-10-02 13:52:39 UTC]

Lost track of this one, I wil fix it upstream.

[Comment 6 Diego Elio Pettenò (RETIRED) 2009-10-02 14:00:47 UTC]

That's noinst_LTLIBRARIES rather than noinst_lib_LTLIBRARIES, by the way ;)

[Comment 7 Samuli Suominen (RETIRED) 2009-10-02 14:02:39 UTC]

OK, the bug was in the ebuild, not in the Makefile.am now that I have looked at it again.

Changed --disable-shared to --disable-static and deleted .la file in src_install.

ssuominen@unique ~/gentoo-x86/media-sound/gejengel $ qlist gejengel
/usr/share/doc/gejengel-0.1.0/TODO.bz2
/usr/share/doc/gejengel-0.1.0/AUTHORS.bz2
/usr/share/doc/gejengel-0.1.0/ChangeLog.bz2
/usr/share/doc/gejengel-0.1.0/README.bz2
/usr/share/applications/gejengel.desktop
/usr/bin/gejengel
/usr/lib64/libgejengel.so.0.0.0
/usr/lib64/libgejengel.so
/usr/lib64/libgejengel.so.0
ssuominen@unique ~/gentoo-x86/media-sound/gejengel $ objdump -p /usr/bin/gejengel | grep NEEDED
  NEEDED               libgejengel.so.0
  NEEDED               libgtkmm-2.4.so.1
  NEEDED               libglibmm-2.4.so.1
  NEEDED               libpthread.so.0
  NEEDED               libstdc++.so.6
  NEEDED               libgcc_s.so.1
  NEEDED               libc.so.6

[Comment 8 Diego Elio Pettenò (RETIRED) 2009-10-02 14:05:38 UTC]

Uh but does it make sense to have a shared object for a single executable? It doesn't seem to install include files, so I guess it does not support plugin interfaces.

[Comment 9 Samuli Suominen (RETIRED) 2009-10-02 14:09:47 UTC]

(In reply to comment #8)
> Uh but does it make sense to have a shared object for a single executable? It
> doesn't seem to install include files, so I guess it does not support plugin
> interfaces.
> 

In that case it would be a upstream bug indeed, passing both --disable-static and --disable-shared still installs the .a and .la. They are both set by default to "yes". Passing only --disable-static gives the result I already posted.

I guess they should be both set to "no" by default and fix the build system in a way that they are not installed...

[Comment 10 Samuli Suominen (RETIRED) 2009-10-02 14:11:56 UTC]

That said. How do I force autotools.eclass to use automake 1.11? This pkg seems to be requiring it...

[Comment 11 Diego Elio Pettenò (RETIRED) 2009-10-02 14:28:54 UTC]

WANT_AUTOMAKE="1.11" before inheriting autotools.eclass should do.

[Comment 12 Samuli Suominen (RETIRED) 2009-10-02 14:40:05 UTC]

Bah. I'm just deleting the libgejengel.{a,la} in src_install for now. I except
Dirk to be fixed this properly for next release. ;-)