Package: 3ddesktop
Version: 0.2.9-5
Severity: grave

The 3ddesktop package currently depends on libfreetype6, but it does not use
it.  This dependency is being pulled in via imlib2-config --libs, which
works as designed but is *not* a correct tool for getting a list of libs to
link against on a GNU system using dynamic linking.

Attached is a short patch which fixes the 3ddesktop package build to no
longer link against spurious libraries.  This patch changes the binary
package dependencies from this: 

 Depends: libc6 (>= 2.3.5-1), libfreetype6 (>= 2.1.5-1), libgcc1 (>= 1:4.0.1), 
libglu1-xorg | libglu1, libice6 | xlibs (>> 4.1.0), libimlib2, libsm6 | xlibs 
(>> 4.1.0), libstdc++6 (>= 4.0.1), libx11-6 | xlibs (>> 4.1.0), libxext6 | 
xlibs (>> 4.1.0), libxi6 | xlibs (>> 4.1.0), libxmu6 | xlibs (>> 4.1.0), libxt6 
| xlibs (>> 4.1.0), libxxf86vm1, xlibmesa-gl | libgl1, zlib1g (>= 1:1.2.1)

to this:

 Depends: libc6 (>= 2.3.5-1), libgcc1 (>= 1:4.0.2), libglu1-xorg | libglu1, 
libimlib2, libstdc++6 (>= 4.0.2-4), libx11-6 | xlibs (>> 4.1.0), libxxf86vm1, 
xlibmesa-gl | libgl1

There are a number of other ways to achieve this effect; you may wish to
edit the configure.ac, Makefile.am, and acinclude.m4 for this package
instead so that patch failures against future upstream versions will tell
you if the handling needs to change, or you may wish to use the (IMHO
kludgy) -Wl,--as-needed option to gcc.  By editing the autotools code, it's
also possible to optimize the build-dependencies in addition to the
dependencies, FWIW.

In any case, although libfreetype6 is currently available in unstable, due
to bug #314385 it is very likely that this package will have to go through a
library transition in the near future.  Since 3ddesktop doesn't use
freetype, I'm filing this bug at "grave" severity pre-emptively, to
encourage you to get your package free of the transition before it starts.

Cheers,
-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
[EMAIL PROTECTED]                                   http://www.debian.org/
diff -u 3ddesktop-0.2.9/debian/rules 3ddesktop-0.2.9/debian/rules
--- 3ddesktop-0.2.9/debian/rules
+++ 3ddesktop-0.2.9/debian/rules
@@ -39,7 +39,7 @@
 
 build-stamp: patch-stamp config.status
        dh_testdir
-       $(MAKE)
+       $(MAKE) 3ddeskd_LDFLAGS="-lImlib2 -lGLU -lGL -lXxf86vm -lX11" LIBS=""
        touch build-stamp
 
 clean: unpatch
diff -u 3ddesktop-0.2.9/debian/changelog 3ddesktop-0.2.9/debian/changelog
--- 3ddesktop-0.2.9/debian/changelog
+++ 3ddesktop-0.2.9/debian/changelog
@@ -1,3 +1,13 @@
+3ddesktop (0.2.9-5.1) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Hard-code the list of needed libraries in debian/rules; allowing
+    autodetection causes gratuitous linkage to a bunch of libraries not
+    needed on Debian, and the autotoolage is in bad enough shape that it's
+    not worth trying to fix this in the configure script.
+
+ -- Steve Langasek <[EMAIL PROTECTED]>  Mon, 21 Nov 2005 03:54:09 -0800
+
 3ddesktop (0.2.9-5) unstable; urgency=low
 
   [ Pierre Habouzit ]

Attachment: signature.asc
Description: Digital signature

Reply via email to