[gdm] Stop using LDFLAGS for libraries
- From: Ryan Lortie <desrt src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gdm] Stop using LDFLAGS for libraries
- Date: Tue, 25 Feb 2014 21:47:38 +0000 (UTC)
commit 9a23fcc8022526efdb1716ca987de1519e4093d6
Author: Ryan Lortie <desrt desrt ca>
Date: Tue Feb 25 16:37:51 2014 -0500
Stop using LDFLAGS for libraries
Originally, EXTRA_*_LIBS was introduced as a way to get certain
libraries to the front of the line when invoking the linker in order to
control which version of a library we got. See bug 85785 for the
history there.
This is probably no longer needed, so officially declare that the only
thing that the EXTRA_ variables mean is "not from pkg-config".
Move all of the EXTRA_*_LIBS from LDFLAGS to _LDADD, along with some
other things that were improperly in LDFLAGS.
https://bugzilla.gnome.org/show_bug.cgi?id=724283
daemon/Makefile.am | 19 ++++---------------
1 files changed, 4 insertions(+), 15 deletions(-)
---
diff --git a/daemon/Makefile.am b/daemon/Makefile.am
index 2fcac8b..87fcb98 100644
--- a/daemon/Makefile.am
+++ b/daemon/Makefile.am
@@ -181,10 +181,6 @@ nodist_gdm_simple_slave_SOURCES = \
gdm-slave-glue.h \
$(NULL)
-gdm_simple_slave_LDFLAGS = \
- $(PAM_LIBS) \
- $(NULL)
-
gdm_simple_slave_LDADD = \
$(top_builddir)/common/libgdmcommon.la \
$(XLIB_LIBS) \
@@ -192,6 +188,7 @@ gdm_simple_slave_LDADD = \
$(EXTRA_DAEMON_LIBS) \
$(SYSTEMD_LIBS) \
$(JOURNALD_LIBS) \
+ $(PAM_LIBS) \
$(NULL)
gdm_xdmcp_chooser_slave_SOURCES = \
@@ -278,11 +275,6 @@ gdm_session_worker_SOURCES += gdm-session-solaris-auditor.h \
gdm-session-solaris-auditor.c
endif
-gdm_session_worker_LDFLAGS = \
- $(XLIB_LIBS) \
- $(PAM_LIBS) \
- $(NULL)
-
gdm_session_worker_LDADD = \
$(top_builddir)/common/libgdmcommon.la \
$(DAEMON_LIBS) \
@@ -290,6 +282,8 @@ gdm_session_worker_LDADD = \
$(JOURNALD_LIBS) \
$(LIBSELINUX_LIBS) \
$(EXTRA_DAEMON_LIBS) \
+ $(XLIB_LIBS) \
+ $(PAM_LIBS) \
$(NULL)
sbin_PROGRAMS = \
@@ -365,12 +359,6 @@ CONSOLE_KIT_SOURCES = \
EXTRA_gdm_SOURCES += $(CONSOLE_KIT_SOURCES)
-# Note that these libs are in LDFLAGS because they should come before
-# everything else on the link line as they may override stuff
-gdm_LDFLAGS = \
- $(EXTRA_DAEMON_LIBS) \
- $(NULL)
-
gdm_LDADD = \
$(top_builddir)/common/libgdmcommon.la \
$(XLIB_LIBS) \
@@ -379,6 +367,7 @@ gdm_LDADD = \
$(LIBWRAP_LIBS) \
$(SYSTEMD_LIBS) \
$(JOURNALD_LIBS) \
+ $(EXTRA_DAEMON_LIBS) \
$(NULL)
if WITH_CONSOLE_KIT
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]