Bug 76174

Summary:	Link order important for IRIX
Product:	[Unmaintained] kxkb	Reporter:	The Written Word <bugzilla-kde>
Component:	general	Assignee:	Andriy Rysin <arysin>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	IRIX
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Patch

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