Send plymouth mailing list submissions to plymouth@lists.freedesktop.org
To subscribe or unsubscribe via the World Wide Web, visit http://lists.freedesktop.org/mailman/listinfo/plymouth or, via email, send a message with subject or body 'help' to plymouth-requ...@lists.freedesktop.org You can reach the person managing the list at plymouth-ow...@lists.freedesktop.org When replying, please edit your Subject line so it is more specific than "Re: Contents of plymouth digest..." Today's Topics: 1. [PATCH] fix build, incorrect linking (Cristian Rodr?guez) 2. Re: [PATCH] fix build, incorrect linking (Ray Strode) ---------------------------------------------------------------------- Message: 1 Date: Sat, 14 Dec 2013 23:22:19 +0000 From: Cristian Rodr?guez <crrodrig...@opensuse.org> To: plymouth@lists.freedesktop.org Subject: [PATCH] fix build, incorrect linking Message-ID: <1387063339-36829-1-git-send-email-crrodrig...@opensuse.org> It is splashy-core that needs to be linked with udev, not plymouthd this causes linking to fail. --- src/Makefile.am | 3 +-- src/libply-splash-core/Makefile.am | 4 ++-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index 3803448..152cd43 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -15,13 +15,12 @@ plymouthdbindir = $(plymouthdaemondir) plymouthdbin_PROGRAMS = plymouthd plymouthd_CFLAGS = $(PLYMOUTH_CFLAGS) \ - $(UDEV_CFLAGS) \ -DPLYMOUTH_PLUGIN_PATH=\"$(PLYMOUTH_PLUGIN_PATH)\" \ -DPLYMOUTH_THEME_PATH=\"$(PLYMOUTH_THEME_PATH)/\" \ -DPLYMOUTH_POLICY_DIR=\"$(PLYMOUTH_POLICY_DIR)/\" \ -DPLYMOUTH_RUNTIME_DIR=\"$(PLYMOUTH_RUNTIME_DIR)\" \ -DPLYMOUTH_CONF_DIR=\"$(PLYMOUTH_CONF_DIR)/\" -plymouthd_LDADD = $(PLYMOUTH_LIBS) $(UDEV_LIBS) libply/libply.la libply-splash-core/libply-splash-core.la +plymouthd_LDADD = $(PLYMOUTH_LIBS) libply/libply.la libply-splash-core/libply-splash-core.la plymouthd_SOURCES = \ ply-boot-protocol.h \ ply-boot-server.h \ diff --git a/src/libply-splash-core/Makefile.am b/src/libply-splash-core/Makefile.am index 960ffdc..d07d7f1 100644 --- a/src/libply-splash-core/Makefile.am +++ b/src/libply-splash-core/Makefile.am @@ -27,12 +27,12 @@ libply_splash_core_HEADERS = \ ply-text-progress-bar.h \ ply-text-step-bar.h -libply_splash_core_la_CFLAGS = $(PLYMOUTH_CFLAGS) \ +libply_splash_core_la_CFLAGS = $(PLYMOUTH_CFLAGS) $(UDEV_CFLAGS) \ -DPLYMOUTH_BACKGROUND_COLOR=$(background_color) \ -DPLYMOUTH_BACKGROUND_END_COLOR=$(background_end_color) \ -DPLYMOUTH_BACKGROUND_START_COLOR=$(background_start_color) \ -DPLYMOUTH_PLUGIN_PATH=\"$(PLYMOUTH_PLUGIN_PATH)\" -libply_splash_core_la_LIBADD = $(PLYMOUTH_LIBS) ../libply/libply.la +libply_splash_core_la_LIBADD = $(PLYMOUTH_LIBS) $(UDEV_LIBS) ../libply/libply.la libply_splash_core_la_LDFLAGS = -export-symbols-regex '^[^_].*' \ -version-info $(LT_CURRENT):$(LT_REVISION):$(LT_AGE) \ -no-undefined -- 1.8.4 ------------------------------ Message: 2 Date: Sat, 14 Dec 2013 19:46:26 -0500 From: Ray Strode <halfl...@gmail.com> To: Cristian Rodr?guez <crrodrig...@opensuse.org> Cc: plymouth <plymouth@lists.freedesktop.org> Subject: Re: [PATCH] fix build, incorrect linking Message-ID: <caa_uwz+dkqd+vxdxqw2mlc1fvw7k_ubz4sq04s-as63fhb4...@mail.gmail.com> Content-Type: text/plain; charset=UTF-8 thanks, pushed On Sat, Dec 14, 2013 at 6:22 PM, Cristian Rodr?guez <crrodrig...@opensuse.org> wrote: > It is splashy-core that needs to be linked with udev, not plymouthd > this causes linking to fail. > --- > src/Makefile.am | 3 +-- > src/libply-splash-core/Makefile.am | 4 ++-- > 2 files changed, 3 insertions(+), 4 deletions(-) > > diff --git a/src/Makefile.am b/src/Makefile.am > index 3803448..152cd43 100644 > --- a/src/Makefile.am > +++ b/src/Makefile.am > @@ -15,13 +15,12 @@ plymouthdbindir = $(plymouthdaemondir) > plymouthdbin_PROGRAMS = plymouthd > > plymouthd_CFLAGS = $(PLYMOUTH_CFLAGS) > \ > - $(UDEV_CFLAGS) > \ > -DPLYMOUTH_PLUGIN_PATH=\"$(PLYMOUTH_PLUGIN_PATH)\" > \ > -DPLYMOUTH_THEME_PATH=\"$(PLYMOUTH_THEME_PATH)/\" > \ > -DPLYMOUTH_POLICY_DIR=\"$(PLYMOUTH_POLICY_DIR)/\" > \ > -DPLYMOUTH_RUNTIME_DIR=\"$(PLYMOUTH_RUNTIME_DIR)\" > \ > -DPLYMOUTH_CONF_DIR=\"$(PLYMOUTH_CONF_DIR)/\" > -plymouthd_LDADD = $(PLYMOUTH_LIBS) $(UDEV_LIBS) libply/libply.la > libply-splash-core/libply-splash-core.la > +plymouthd_LDADD = $(PLYMOUTH_LIBS) libply/libply.la > libply-splash-core/libply-splash-core.la > plymouthd_SOURCES = > \ > ply-boot-protocol.h > \ > ply-boot-server.h > \ > diff --git a/src/libply-splash-core/Makefile.am > b/src/libply-splash-core/Makefile.am > index 960ffdc..d07d7f1 100644 > --- a/src/libply-splash-core/Makefile.am > +++ b/src/libply-splash-core/Makefile.am > @@ -27,12 +27,12 @@ libply_splash_core_HEADERS = \ > ply-text-progress-bar.h > \ > ply-text-step-bar.h > > -libply_splash_core_la_CFLAGS = $(PLYMOUTH_CFLAGS) > \ > +libply_splash_core_la_CFLAGS = $(PLYMOUTH_CFLAGS) $(UDEV_CFLAGS) > \ > -DPLYMOUTH_BACKGROUND_COLOR=$(background_color) > \ > > -DPLYMOUTH_BACKGROUND_END_COLOR=$(background_end_color) \ > > -DPLYMOUTH_BACKGROUND_START_COLOR=$(background_start_color) \ > -DPLYMOUTH_PLUGIN_PATH=\"$(PLYMOUTH_PLUGIN_PATH)\" > -libply_splash_core_la_LIBADD = $(PLYMOUTH_LIBS) ../libply/libply.la > +libply_splash_core_la_LIBADD = $(PLYMOUTH_LIBS) $(UDEV_LIBS) > ../libply/libply.la > libply_splash_core_la_LDFLAGS = -export-symbols-regex '^[^_].*' \ > -version-info $(LT_CURRENT):$(LT_REVISION):$(LT_AGE) \ > -no-undefined > -- > 1.8.4 > > _______________________________________________ > plymouth mailing list > plymouth@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/plymouth ------------------------------ _______________________________________________ plymouth mailing list plymouth@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/plymouth End of plymouth Digest, Vol 55, Issue 2 ***************************************