Re: rpath problems

[Noah Levitt <nlevitt columbia edu>]

I think I've discovered a solution to my problem. Some of
the Makefile.am's set LDFLAGS, but omit @LDFLAGS   The
./configure-time contents of $LDFLAGS are lost. I don't
think this is ideal behavior, and I think an appropriate
solution is to use AM_LDFLAGS instead of LDFLAGS. Then,
LDFLAGS will inherit $LDFLAGS, and all will be well.

I haven't actually tested this; I had problems getting the
configury to work with the cvs sources. I'm pretty confident
that it will work, though, so I submit it for your
consideration.

Noah


Index: gdk/Makefile.am
===================================================================
RCS file: /cvs/gnome/gtk+/gdk/Makefile.am,v
retrieving revision 1.88
diff -u -d -u -r1.88 Makefile.am
--- gdk/Makefile.am	20 Jun 2002 23:29:16 -0000	1.88
+++ gdk/Makefile.am	5 Jul 2002 05:25:19 -0000
@@ -68,7 +68,7 @@
 # libtool stuff: set version and export symbols for resolving
 # since automake doesn't support conditionalized libsomething_la_LDFLAGS
 # we use the general approach here
-LDFLAGS = @STRIP_BEGIN@ 						\
+AM_LDFLAGS = @STRIP_BEGIN@ 						\
 	-version-info $(LT_CURRENT):$(LT_REVISION):$(LT_AGE) 		\
 	-export-dynamic 						\
 	-rpath $(libdir) 						\
Index: gdk/linux-fb/Makefile.am
===================================================================
RCS file: /cvs/gnome/gtk+/gdk/linux-fb/Makefile.am,v
retrieving revision 1.19
diff -u -d -u -r1.19 Makefile.am
--- gdk/linux-fb/Makefile.am	25 Jun 2001 23:01:36 -0000	1.19
+++ gdk/linux-fb/Makefile.am	5 Jul 2002 05:25:19 -0000
@@ -19,7 +19,7 @@
 	@GTK_DEBUG_FLAGS@ 	\
 @STRIP_END@
 
-LDFLAGS = @STRIP_BEGIN@ \
+AM_LDFLAGS = @STRIP_BEGIN@ \
 	@GDK_DEP_LIBS@	\
 	-lm		\
 @STRIP_END@
Index: gtk/Makefile.am
===================================================================
RCS file: /cvs/gnome/gtk+/gtk/Makefile.am,v
retrieving revision 1.214
diff -u -d -u -r1.214 Makefile.am
--- gtk/Makefile.am	14 May 2002 22:04:45 -0000	1.214
+++ gtk/Makefile.am	5 Jul 2002 05:25:19 -0000
@@ -64,7 +64,7 @@
 # since automake doesn't support conditionalized libsomething_la_LDFLAGS
 # we use the general approach here
 libgtkincludedir = $(includedir)/gtk-2.0/gtk
-LDFLAGS = @STRIP_BEGIN@ \
+AM_LDFLAGS = @STRIP_BEGIN@ \
 	-version-info $(LT_CURRENT):$(LT_REVISION):$(LT_AGE)			\
 	-export-dynamic								\
 	-rpath $(libdir)							\