[PATCH] Filter out -l parameters when setting dependencies
Alan Coopersmith
alan.coopersmith at oracle.com
Thu May 9 00:04:30 PDT 2013
On 05/ 8/13 03:57 PM, Gustavo Noronha Silva wrote:
> Newer make (Fedora 19) gets confused when it finds a -l parameter in a
> dependency, and tries to make it as a target, causing the build to fail.
>
> Signed-off-by: Gustavo Noronha Silva <gustavo.noronha at collabora.com>
> ---
These files seem to be an unrelated change relating to pixman:
> configure.ac | 2 +-
> fb/fb.h | 3 +
> fb/fbpict.c | 149 +++++++++++++++++++++++++++++++++++-
> fb/fbscreen.c | 1 +
NAK on these, as they use GNU make extensions and will break on BSD & Solaris make:
> diff --git a/hw/dmx/Makefile.am b/hw/dmx/Makefile.am
> index fb727e6..62a6bbe 100644
> --- a/hw/dmx/Makefile.am
> +++ b/hw/dmx/Makefile.am
> @@ -82,7 +82,7 @@ XDMX_LIBS = \
> config/libdmxconfig.a
>
> Xdmx_LDFLAGS = $(LD_EXPORT_SYMBOLS_FLAG)
> -Xdmx_DEPENDENCIES= $(XDMX_LIBS)
> +Xdmx_DEPENDENCIES= $(filter-out -l%,$(XDMX_LIBS))
> Xdmx_LDADD = $(XDMX_LIBS) $(XDMX_SYS_LIBS) $(XSERVER_SYS_LIBS)
>
> relink:
> diff --git a/hw/vfb/Makefile.am b/hw/vfb/Makefile.am
> index 9f4992c..06830ae 100644
> --- a/hw/vfb/Makefile.am
> +++ b/hw/vfb/Makefile.am
> @@ -25,7 +25,7 @@ XVFB_LIBS = \
> $(XSERVER_LIBS)
>
> Xvfb_LDADD = $(XVFB_LIBS) $(XVFB_SYS_LIBS) $(XSERVER_SYS_LIBS)
> -Xvfb_DEPENDENCIES = $(XVFB_LIBS)
> +Xvfb_DEPENDENCIES = $(filter-out -l%,$(XVFB_LIBS))
> Xvfb_LDFLAGS = $(LD_EXPORT_SYMBOLS_FLAG)
>
> relink:
> diff --git a/hw/xfree86/Makefile.am b/hw/xfree86/Makefile.am
> index c3899b5..82bac1d 100644
> --- a/hw/xfree86/Makefile.am
> +++ b/hw/xfree86/Makefile.am
> @@ -65,7 +65,7 @@ Xorg_LDADD = \
> $(LOCAL_LIBS) \
> $(XORG_SYS_LIBS) \
> $(XSERVER_SYS_LIBS)
> -Xorg_DEPENDENCIES = $(LOCAL_LIBS)
> +Xorg_DEPENDENCIES = $(filter-out -l%,$(LOCAL_LIBS))
>
> Xorg_LDFLAGS = $(LD_EXPORT_SYMBOLS_FLAG)
>
> diff --git a/hw/xnest/Makefile.am b/hw/xnest/Makefile.am
> index 3c099cd..180d62c 100644
> --- a/hw/xnest/Makefile.am
> +++ b/hw/xnest/Makefile.am
> @@ -55,7 +55,7 @@ XNEST_LIBS = \
>
> Xnest_SOURCES = $(SRCS)
>
> -Xnest_DEPENDENCIES = $(XNEST_LIBS)
> +Xnest_DEPENDENCIES = $(filter-out -l%,$(XNEST_LIBS))
> Xnest_LDADD = $(XNEST_LIBS) $(XNEST_SYS_LIBS) $(XSERVER_SYS_LIBS)
> Xnest_LDFLAGS = $(LD_EXPORT_SYMBOLS_FLAG)
>
> diff --git a/hw/xquartz/mach-startup/Makefile.am b/hw/xquartz/mach-startup/Makefile.am
> index 77962b1..c8626cc 100644
> --- a/hw/xquartz/mach-startup/Makefile.am
> +++ b/hw/xquartz/mach-startup/Makefile.am
> @@ -23,17 +23,18 @@ nodist_X11_bin_SOURCES = \
> mach_startupServer.c \
> mach_startupUser.c
>
> -X11_bin_DEPENDENCIES = \
> +# $(XSERVER_SYS_LIBS) is placed here in order to set command line ordering
> +# to work around build issues on Tiger.
> +X11_bin_LDADD =\
> $(top_builddir)/hw/xquartz/libXquartz.la \
> $(top_builddir)/hw/xquartz/xpr/libXquartzXpr.la \
> $(top_builddir)/dix/dixfonts.lo \
> $(top_builddir)/miext/rootless/librootless.la \
> $(top_builddir)/hw/xquartz/pbproxy/libxpbproxy.la \
> - $(XQUARTZ_LIBS) $(XSERVER_LIBS)
> + $(XQUARTZ_LIBS) $(XSERVER_LIBS) \
> + $(XSERVER_SYS_LIBS)
>
> -# $(XSERVER_SYS_LIBS) is placed here in order to set command line ordering
> -# to work around build issues on Tiger.
> -X11_bin_LDADD = $(X11_bin_DEPENDENCIES) $(XSERVER_SYS_LIBS)
> +X11_bin_DEPENDENCIES = $(subst $(XSERVER_SYS_LIBS),, $(filter-out -l%,$(X11_bin_LDADD)))
>
> X11_bin_LDFLAGS = \
> -lXplugin \
> diff --git a/hw/xwin/Makefile.am b/hw/xwin/Makefile.am
> index 93ce570..a33c3f1 100644
> --- a/hw/xwin/Makefile.am
> +++ b/hw/xwin/Makefile.am
> @@ -153,7 +153,7 @@ INCLUDES = -I$(top_srcdir)/miext/rootless
>
> XWIN_SYS_LIBS += -ldxguid
>
> -XWin_DEPENDENCIES = $(MULTIWINDOWEXTWM_LIBS) $(XWIN_GLX_LIBS) $(XWIN_LIBS) $(MAIN_LIB) $(XSERVER_LIBS)
> +XWin_DEPENDENCIES = $(filter-out -l%,$(MULTIWINDOWEXTWM_LIBS) $(XWIN_GLX_LIBS) $(XWIN_LIBS) $(MAIN_LIB) $(XSERVER_LIBS))
> XWin_LDADD = $(MULTIWINDOW_LIBS) $(MULTIWINDOWEXTWM_LIBS) $(XWIN_GLX_LIBS) $(XWIN_GLX_LINK_FLAGS) $(XWIN_LIBS) $(MAIN_LIB) $(XSERVER_LIBS) $(XSERVER_SYS_LIBS) $(XWIN_SYS_LIBS)
> XWin_LDFLAGS = -mwindows -static
>
> diff --git a/test/Makefile.am b/test/Makefile.am
> index 34f53fc..3509306 100644
> --- a/test/Makefile.am
> +++ b/test/Makefile.am
> @@ -117,7 +117,7 @@ libxservertest_la_LIBADD += \
> endif
> endif
>
> -libxservertest_la_DEPENDENCIES = $(libxservertest_la_LIBADD)
> +libxservertest_la_DEPENDENCIES = $(filter-out -l%,$(libxservertest_la_LIBADD))
> endif
>
> EXTRA_DIST = ddxstubs.c
>
--
-Alan Coopersmith- alan.coopersmith at oracle.com
Oracle Solaris Engineering - http://blogs.oracle.com/alanc
More information about the xorg-devel
mailing list