[Bug 123477] New - CFLAGS/CPPFLAGS/LDFLAGS in gar.conf.mk don't have desired effect

[bugzilla-daemon widget gnome org]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=123477

Changed by jtalkington users sourceforge net 

--- shadow/123477	Mon Sep 29 11:16:22 2003
+++ shadow/123477.tmp.24323	Mon Sep 29 11:16:22 2003
@@ -0,0 +1,47 @@
+Bug#: 123477
+Product: GARNOME
+Version: unspecified
+OS: All
+OS Details: all
+Status: NEW   
+Resolution: 
+Severity: normal
+Priority: Normal
+Component: general
+AssignedTo: jdub perkypants org                            
+ReportedBy: jtalkington users sourceforge net               
+QAContact: garnome-list gnome org
+TargetMilestone: ---
+URL: 
+Summary: CFLAGS/CPPFLAGS/LDFLAGS in gar.conf.mk don't have desired effect
+
+The following lines in gar.conf.mk don't have the desired effect:
+CPPFLAGS += -I$(DESTDIR)$(includedir) -I$(DESTDIR)/usr/X11R6/include
+CFLAGS += -I$(DESTDIR)$(includedir) -I$(DESTDIR)/usr/X11R6/include
+-L$(DESTDIR)$
+(libdir) -L$(DESTDIR)/usr/X11R6/lib
+LDFLAGS += -L$(DESTDIR)$(libdir) -L$(DESTDIR)/usr/X11R6/lib
+
+There is a big fat warning about 5 lines before that say DESTDIR isn't
+defined until install time, which is correct.  The result is that
+-I/usr/X11R6/include and -L/usr/X11R6/lib get added to the search order,
+when they shouldn't be, which will screw up some software (most notably
+expat on darwin) that adds the current build directory after the LDFLAGS or
+CFLAGS included search paths.
+
+This also effects the PKG_CONFIG_PATH and the PATH/LD_LIBRARY_PATH settings.
+
+Here are more sane settings that don't effect the search path adversely,
+but have the intended results:
+
+# allow us to link to libraries we installed
+CPPFLAGS += -I$(includedir)
+CFLAGS += -I$(includedir) -L$(libdir)
+LDFLAGS += -L$(libdir)
+
+# allow us to use programs we just built
+PATH := $(bindir):$(sbindir):$(PATH)
+LD_LIBRARY_PATH := $(libdir):$(LD_LIBRARY_PATH)
+
+# This is for foo-config chaos
+PKG_CONFIG_PATH=$(libdir)/pkgconfig:/usr/lib/pkgconfig