Bug 76174 - Link order important for IRIX
Summary: Link order important for IRIX
Status: RESOLVED FIXED
Alias: None
Product: kxkb
Classification: Miscellaneous
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources IRIX
: NOR normal
Target Milestone: ---
Assignee: Andriy Rysin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-02-26 10:00 UTC by The Written Word
Modified: 2004-04-12 20:48 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:

Attachments
Patch (815 bytes, patch)
2004-02-26 10:00 UTC, The Written Word 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description The Written Word 2004-02-26 10:00:05 UTC 
Version:            (using KDE KDE 3.2.0)
Installed from:    Compiled From Sources
Compiler:          CC 7.4.1 
OS:          Irix

1. [kxkb/Makefile.am]
   On IRIX, libraries must come after their search path (-l after -L).
Comment 1 The Written Word 2004-02-26 10:00:28 UTC 
Created attachment 4897 [details]
Patch
Comment 2 Waldo Bastian 2004-03-19 16:18:11 UTC 
Huh, how does this patch change that? It only changes the order of the libraries itself as far as I see.
Comment 3 The Written Word 2004-03-19 20:52:29 UTC 
Sorry, in this case, if library a is dependent on library b, on IRIX, then we need -la -lb, which the patch above fixes.
Comment 4 mcamen 2004-04-12 20:48:27 UTC 
CVS commit by mcamen: 

Another link order fixlet for IRIX. Harmless on Linux.

CCMAIL: 76174-done@bugs.kde.org


  M +2 -2      Makefile.am   1.36


--- kdebase/kxkb/Makefile.am  #1.35:1.36
@@ -13,5 +13,5 @@
 kcm_keyboard_la_SOURCES = rules.cpp kcmlayout.cpp pixmap.cpp kcmlayoutwidget.ui kcmmisc.cpp kcmmiscwidget.ui
 kcm_keyboard_la_LDFLAGS  = $(all_libraries) -module -avoid-version -no-undefined
-kcm_keyboard_la_LIBADD = $(XTESTLIB) $(LIB_KIO) -lxkbfile
+kcm_keyboard_la_LIBADD = -lxkbfile $(XTESTLIB) $(LIB_KIO)
 
 #kxkb_la_includedir = .
@@ -19,5 +19,5 @@
 kxkb_la_SOURCES = kxkb.cpp rules.cpp extension.cpp pixmap.cpp kxkb.skel
 kxkb_la_LDFLAGS = $(all_libraries) -module -avoid-version
-kxkb_la_LIBADD = -lX11 -lXext $(LIB_KDEUI) -lxkbfile
+kxkb_la_LIBADD = -lxkbfile -lX11 -lXext $(LIB_KDEUI)
 
 xdg_apps_DATA = keyboard.desktop keyboard_layout.desktop