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
***************************************

Reply via email to