Package: terminal.app
Version: 0.9.4+cvs20051125-6
Severity: important
Tags: patch
User: [email protected]
Usertags: origin-ubuntu ubuntu-patch oneiric

terminal.app fails to build with a linker that defaults to --as-needed,
as shown in this Ubuntu bug report:

  https://bugs.launchpad.net/ubuntu/+source/terminal.app/+bug/831221

This is because its link line order is wrong, putting libraries before
the objects that use them rather than after.  See:

  http://wiki.debian.org/ToolChain/DSOLinking#Only_link_with_needed_libraries

The following patch fixes this, although I'm never entirely sure about
the semantics of the various *_LIBS variables in gnustep-make so you
might be able to pick a different one.  *_LDFLAGS, though, is definitely
only for linker options other than -l*.

  * Move -lutil from Terminal_LDFLAGS to Terminal_TOOL_LIBS (LP: #831221).

--- terminal.app-0.9.4+cvs20051125.orig/GNUmakefile
+++ terminal.app-0.9.4+cvs20051125/GNUmakefile
@@ -44,7 +44,7 @@
        \
        Label.m
 
-Terminal_LDFLAGS = -lutil
+Terminal_TOOL_LIBS = -lutil
 
 Terminal_LOCALIZED_RESOURCE_FILES = Localizable.strings
 Terminal_LANGUAGES = English Swedish German French Spanish Hungarian Turkish \

Thanks,

-- 
Colin Watson                                       [[email protected]]



-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to