Message ID | 1456076407-20364-1-git-send-email-bernd.kuhls@t-online.de |
---|---|
State | Accepted |
Headers | show |
Dear Bernd Kuhls, On Sun, 21 Feb 2016 18:40:07 +0100, Bernd Kuhls wrote: > The package selects BR2_PACKAGE_LIBGLIB2 but did not depend on it. > > The buildsystem treats libglib2 as a hard-dependency: > https://cgit.freedesktop.org/spice/spice/tree/configure.ac?h=0.12#n117 > > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> I am not sure in fact. The part of configure.ac you're pointing to simply adds libglib2 to SPICE_REQUIRES, which is *only* used in the spice build system to fill in the Requires.private field of spice-server.pc. I.e, it seems like libglib2 is only an indirect dependency of spice, and a reference to libglib2 only needed when static linking. This clearly needs more investigation, as other dependencies of spice are handled in the same way. Yann, you originally added the spice package, maybe you can have a look? Thanks! Thomas
Thomas, Bernd, All, On 2016-02-21 21:12 +0100, Thomas Petazzoni spake thusly: > Dear Bernd Kuhls, > > On Sun, 21 Feb 2016 18:40:07 +0100, Bernd Kuhls wrote: > > The package selects BR2_PACKAGE_LIBGLIB2 but did not depend on it. > > > > The buildsystem treats libglib2 as a hard-dependency: > > https://cgit.freedesktop.org/spice/spice/tree/configure.ac?h=0.12#n117 > > > > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> > > I am not sure in fact. The part of configure.ac you're pointing to > simply adds libglib2 to SPICE_REQUIRES, which is *only* used in the > spice build system to fill in the Requires.private field of > spice-server.pc. I.e, it seems like libglib2 is only an indirect > dependency of spice, and a reference to libglib2 only needed when > static linking. $ git grep -E '\<glib.h\>' server/reds.c:#include <glib.h> The server is not optional (only the client is), and reds.c is not conditoinal in server/Makefile.am. So it looks like it really requires glib for itself. > This clearly needs more investigation, as other dependencies of spice > are handled in the same way. Not sure I'm following... Except for libglib2, for each select or depends on, we have the corresponding entry in _DEPENDENCIES. Unless you're referring to how spice internally treats its depednencies? > Yann, you originally added the spice package, maybe you can have a look? Are you kidding? That was more than three years ago! Hehe! ;-) And by the way, a bump would be welcome (after the release, I mean). ;-) Regards, Yann E. MORIN.
Yann, On Sun, 21 Feb 2016 23:05:52 +0100, Yann E. MORIN wrote: > $ git grep -E '\<glib.h\>' > server/reds.c:#include <glib.h> > > The server is not optional (only the client is), and reds.c is not > conditoinal in server/Makefile.am. Aaah, I missed the PKG_CHECK_MODULES([GLIB2], [glib-2.0 >= 2.22]) line. Since I believe Bernd's patch was pointing to: SPICE_REQUIRES+=" glib-2.0 >= 2.22" I thought he was referring to this line as the justification for the glib dependency. But OK, it really needs glib. I'll apply Bernd's patch. Thanks for confirming! Thomas
Dear Bernd Kuhls, On Sun, 21 Feb 2016 18:40:07 +0100, Bernd Kuhls wrote: > The package selects BR2_PACKAGE_LIBGLIB2 but did not depend on it. > > The buildsystem treats libglib2 as a hard-dependency: > https://cgit.freedesktop.org/spice/spice/tree/configure.ac?h=0.12#n117 > > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> > --- > package/spice/spice.mk | 1 + > 1 file changed, 1 insertion(+) Applied to master, thanks. Thomas
diff --git a/package/spice/spice.mk b/package/spice/spice.mk index a31d78d..61a9772 100644 --- a/package/spice/spice.mk +++ b/package/spice/spice.mk @@ -14,6 +14,7 @@ SPICE_DEPENDENCIES = \ alsa-lib \ celt051 \ jpeg \ + libglib2 \ openssl \ pixman \ python-pyparsing \
The package selects BR2_PACKAGE_LIBGLIB2 but did not depend on it. The buildsystem treats libglib2 as a hard-dependency: https://cgit.freedesktop.org/spice/spice/tree/configure.ac?h=0.12#n117 Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> --- package/spice/spice.mk | 1 + 1 file changed, 1 insertion(+)