* [Buildroot] [PATCH 2/5] package/gdk-pixbuf: add gio to Requires.private
2019-11-14 19:13 [Buildroot] [PATCH 1/5] package/spice: add optional gdk-pixbuf dependency Fabrice Fontaine
@ 2019-11-14 19:13 ` Fabrice Fontaine
2019-11-18 22:23 ` Thomas Petazzoni
2019-11-14 19:13 ` [Buildroot] [PATCH 3/5] package/spice: needs dynamic library Fabrice Fontaine
` (3 subsequent siblings)
4 siblings, 1 reply; 9+ messages in thread
From: Fabrice Fontaine @ 2019-11-14 19:13 UTC (permalink / raw)
To: buildroot
Fixes:
- http://autobuild.buildroot.org/results/96c786f85d35f33508e9c71778043d16b87f72cd
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
....0.pc.in-add-gio-to-Requires.private.patch | 32 +++++++++++++++++++
1 file changed, 32 insertions(+)
create mode 100644 package/gdk-pixbuf/0001-gdk-pixbuf-2.0.pc.in-add-gio-to-Requires.private.patch
diff --git a/package/gdk-pixbuf/0001-gdk-pixbuf-2.0.pc.in-add-gio-to-Requires.private.patch b/package/gdk-pixbuf/0001-gdk-pixbuf-2.0.pc.in-add-gio-to-Requires.private.patch
new file mode 100644
index 0000000000..b9505cc5ef
--- /dev/null
+++ b/package/gdk-pixbuf/0001-gdk-pixbuf-2.0.pc.in-add-gio-to-Requires.private.patch
@@ -0,0 +1,32 @@
+From d7f003e47a1c4a531e30e0633c7f6d3b11ef6e2f Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Date: Thu, 14 Nov 2019 09:52:11 +0100
+Subject: [PATCH] gdk-pixbuf-2.0.pc.in: add gio to Requires.private
+
+Fix static linking with spice
+
+Fixes:
+ - http://autobuild.buildroot.org/results/96c786f85d35f33508e9c71778043d16b87f72cd
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+[Upstream status: Not upstreamable (upstream switched to meson)]
+---
+ gdk-pixbuf-2.0.pc.in | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/gdk-pixbuf-2.0.pc.in b/gdk-pixbuf-2.0.pc.in
+index 7addefcbd..ff262bc68 100644
+--- a/gdk-pixbuf-2.0.pc.in
++++ b/gdk-pixbuf-2.0.pc.in
+@@ -12,7 +12,7 @@ Name: GdkPixbuf
+ Description: Image loading and scaling
+ Version: @VERSION@
+ Requires: gobject-2.0
+-Requires.private: gmodule-no-export-2.0 @PNG_DEP_CFLAGS_PACKAGES@
++Requires.private: gmodule-no-export-2.0 gio-2.0 @PNG_DEP_CFLAGS_PACKAGES@
+ Libs: -L${libdir} -lgdk_pixbuf- at GDK_PIXBUF_API_VERSION@
+ Libs.private: @GDK_PIXBUF_EXTRA_LIBS@
+ Cflags: -I${includedir}/gdk-pixbuf- at GDK_PIXBUF_API_VERSION@ @GDK_PIXBUF_EXTRA_CFLAGS@
+--
+2.23.0
+
--
2.23.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [Buildroot] [PATCH 3/5] package/spice: needs dynamic library
2019-11-14 19:13 [Buildroot] [PATCH 1/5] package/spice: add optional gdk-pixbuf dependency Fabrice Fontaine
2019-11-14 19:13 ` [Buildroot] [PATCH 2/5] package/gdk-pixbuf: add gio to Requires.private Fabrice Fontaine
@ 2019-11-14 19:13 ` Fabrice Fontaine
2019-11-14 19:13 ` [Buildroot] [PATCH 4/5] package/spice-protocol: bump to version 0.14.0 Fabrice Fontaine
` (2 subsequent siblings)
4 siblings, 0 replies; 9+ messages in thread
From: Fabrice Fontaine @ 2019-11-14 19:13 UTC (permalink / raw)
To: buildroot
spice can optionally uses opus and celt051 which will result in a static
build failure as opus embeds its own celt version.
So add a dynamic library dependency on spice as this is the only package
using celt051 and because next version of spice will also fails to build
statically because of
https://gitlab.freedesktop.org/spice/spice-common/commit/16aa8c98d89fdfc49b7aa58d60f1cd82c075cb1f
which redefines the X509_NAME_add_entry_by_txt openssl function
Fixes:
- http://autobuild.buildroot.org/results/96c786f85d35f33508e9c71778043d16b87f72cd
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
package/spice/Config.in | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/package/spice/Config.in b/package/spice/Config.in
index 2241b55b3d..586f4cb1ed 100644
--- a/package/spice/Config.in
+++ b/package/spice/Config.in
@@ -1,12 +1,14 @@
-comment "spice server needs a toolchain w/ wchar, threads"
+comment "spice server needs a toolchain w/ wchar, threads, dynamic library"
depends on BR2_i386 || BR2_x86_64
- depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
+ depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS || \
+ BR2_STATIC_LIBS
config BR2_PACKAGE_SPICE
bool "spice server"
depends on BR2_i386 || BR2_x86_64
depends on BR2_USE_WCHAR # libglib2
depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2
+ depends on !BR2_STATIC_LIBS
select BR2_PACKAGE_JPEG
select BR2_PACKAGE_LIBGLIB2
select BR2_PACKAGE_OPENSSL
--
2.23.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [Buildroot] [PATCH 4/5] package/spice-protocol: bump to version 0.14.0
2019-11-14 19:13 [Buildroot] [PATCH 1/5] package/spice: add optional gdk-pixbuf dependency Fabrice Fontaine
2019-11-14 19:13 ` [Buildroot] [PATCH 2/5] package/gdk-pixbuf: add gio to Requires.private Fabrice Fontaine
2019-11-14 19:13 ` [Buildroot] [PATCH 3/5] package/spice: needs dynamic library Fabrice Fontaine
@ 2019-11-14 19:13 ` Fabrice Fontaine
2019-11-14 19:13 ` [Buildroot] [PATCH 5/5] package/spice: security bump to version 0.14.2 Fabrice Fontaine
2019-11-15 20:20 ` [Buildroot] [PATCH 1/5] package/spice: add optional gdk-pixbuf dependency Yann E. MORIN
4 siblings, 0 replies; 9+ messages in thread
From: Fabrice Fontaine @ 2019-11-14 19:13 UTC (permalink / raw)
To: buildroot
- This bump is needed for spice 0.14.2
- Add license hash
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
package/spice-protocol/spice-protocol.hash | 5 ++++-
package/spice-protocol/spice-protocol.mk | 2 +-
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/package/spice-protocol/spice-protocol.hash b/package/spice-protocol/spice-protocol.hash
index e72bb2d712..c23135141a 100644
--- a/package/spice-protocol/spice-protocol.hash
+++ b/package/spice-protocol/spice-protocol.hash
@@ -1,2 +1,5 @@
# Locally calculated after checking pgp signature
-sha256 20350bc4309039fdf0d29ee4fd0033cde27bccf33501e13b3c1befafde9d0c9c spice-protocol-0.12.14.tar.bz2
+sha256 b6a4aa1ca32668790b45a494bbd000e9d05797b391d5a5d4b91adf1118216eac spice-protocol-0.14.0.tar.bz2
+
+# Hash for license file
+sha256 eb55ea746be694c449224c7ddce6340648b075fd377281712a19094e7fb7ef92 COPYING
diff --git a/package/spice-protocol/spice-protocol.mk b/package/spice-protocol/spice-protocol.mk
index 7392e9533d..f8c55c7b01 100644
--- a/package/spice-protocol/spice-protocol.mk
+++ b/package/spice-protocol/spice-protocol.mk
@@ -4,7 +4,7 @@
#
################################################################################
-SPICE_PROTOCOL_VERSION = 0.12.14
+SPICE_PROTOCOL_VERSION = 0.14.0
SPICE_PROTOCOL_SOURCE = spice-protocol-$(SPICE_PROTOCOL_VERSION).tar.bz2
SPICE_PROTOCOL_SITE = http://www.spice-space.org/download/releases
SPICE_PROTOCOL_LICENSE = BSD-3-Clause
--
2.23.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [Buildroot] [PATCH 5/5] package/spice: security bump to version 0.14.2
2019-11-14 19:13 [Buildroot] [PATCH 1/5] package/spice: add optional gdk-pixbuf dependency Fabrice Fontaine
` (2 preceding siblings ...)
2019-11-14 19:13 ` [Buildroot] [PATCH 4/5] package/spice-protocol: bump to version 0.14.0 Fabrice Fontaine
@ 2019-11-14 19:13 ` Fabrice Fontaine
2019-11-15 20:20 ` [Buildroot] [PATCH 1/5] package/spice: add optional gdk-pixbuf dependency Yann E. MORIN
4 siblings, 0 replies; 9+ messages in thread
From: Fabrice Fontaine @ 2019-11-14 19:13 UTC (permalink / raw)
To: buildroot
- Fix CVE-2019-3813: fix off-by-one error in group/slot boundary check
- Add license hash
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
package/spice/spice.hash | 3 ++-
package/spice/spice.mk | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/package/spice/spice.hash b/package/spice/spice.hash
index 1a25926ab2..4243244129 100644
--- a/package/spice/spice.hash
+++ b/package/spice/spice.hash
@@ -1,2 +1,3 @@
# Locally calculated
-sha256 1ead5de63d06eededed4017db37240f07bef0abffbaf621899647e7e685a1519 spice-0.14.1.tar.bz2
+sha256 b203b3882e06f4c7249a3150d90c84e1a90490d41ead255a3d2cede46f4a29a7 spice-0.14.2.tar.bz2
+sha256 dc626520dcd53a22f727af3ee42c770e56c97a64fe3adb063799d8ab032fe551 COPYING
diff --git a/package/spice/spice.mk b/package/spice/spice.mk
index f5801ce0ff..ab34d69d2c 100644
--- a/package/spice/spice.mk
+++ b/package/spice/spice.mk
@@ -4,7 +4,7 @@
#
################################################################################
-SPICE_VERSION = 0.14.1
+SPICE_VERSION = 0.14.2
SPICE_SOURCE = spice-$(SPICE_VERSION).tar.bz2
SPICE_SITE = http://www.spice-space.org/download/releases/spice-server
SPICE_LICENSE = LGPL-2.1+
--
2.23.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [Buildroot] [PATCH 1/5] package/spice: add optional gdk-pixbuf dependency
2019-11-14 19:13 [Buildroot] [PATCH 1/5] package/spice: add optional gdk-pixbuf dependency Fabrice Fontaine
` (3 preceding siblings ...)
2019-11-14 19:13 ` [Buildroot] [PATCH 5/5] package/spice: security bump to version 0.14.2 Fabrice Fontaine
@ 2019-11-15 20:20 ` Yann E. MORIN
2019-11-15 20:27 ` Fabrice Fontaine
4 siblings, 1 reply; 9+ messages in thread
From: Yann E. MORIN @ 2019-11-15 20:20 UTC (permalink / raw)
To: buildroot
Fabrice, All,
On 2019-11-14 20:13 +0100, Fabrice Fontaine spake thusly:
> This dependency has been added in version 0.14.1 with
> https://gitlab.freedesktop.org/spice/spice-common/commit/d58a5b6a5a25134396661e1d805dfc567181e3af
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
> package/spice/spice.mk | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/package/spice/spice.mk b/package/spice/spice.mk
> index 16e57441a8..f5801ce0ff 100644
> --- a/package/spice/spice.mk
> +++ b/package/spice/spice.mk
> @@ -13,6 +13,7 @@ SPICE_INSTALL_STAGING = YES
> SPICE_DEPENDENCIES = \
> host-pkgconf \
> jpeg \
> + $(if $(BR2_PACKAGE_GDK_PIXBUF),gdk-pixbuf) \
I see you've gine for conciseness here. However, I think we want to
handle this optional dependency like we handle the optional dependency
on opus, lines 53..56.
Regards,
Yann E. MORIN.
> libglib2 \
> openssl \
> pixman \
> --
> 2.23.0
>
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 9+ messages in thread* [Buildroot] [PATCH 1/5] package/spice: add optional gdk-pixbuf dependency
2019-11-15 20:20 ` [Buildroot] [PATCH 1/5] package/spice: add optional gdk-pixbuf dependency Yann E. MORIN
@ 2019-11-15 20:27 ` Fabrice Fontaine
2019-11-17 15:40 ` Arnout Vandecappelle
0 siblings, 1 reply; 9+ messages in thread
From: Fabrice Fontaine @ 2019-11-15 20:27 UTC (permalink / raw)
To: buildroot
Hi Yann,
Le ven. 15 nov. 2019 ? 21:20, Yann E. MORIN <yann.morin.1998@free.fr> a ?crit :
>
> Fabrice, All,
>
> On 2019-11-14 20:13 +0100, Fabrice Fontaine spake thusly:
> > This dependency has been added in version 0.14.1 with
> > https://gitlab.freedesktop.org/spice/spice-common/commit/d58a5b6a5a25134396661e1d805dfc567181e3af
> >
> > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > ---
> > package/spice/spice.mk | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/package/spice/spice.mk b/package/spice/spice.mk
> > index 16e57441a8..f5801ce0ff 100644
> > --- a/package/spice/spice.mk
> > +++ b/package/spice/spice.mk
> > @@ -13,6 +13,7 @@ SPICE_INSTALL_STAGING = YES
> > SPICE_DEPENDENCIES = \
> > host-pkgconf \
> > jpeg \
> > + $(if $(BR2_PACKAGE_GDK_PIXBUF),gdk-pixbuf) \
>
> I see you've gine for conciseness here. However, I think we want to
> handle this optional dependency like we handle the optional dependency
> on opus, lines 53..56.
OK, I can send a v2 with this change however I've also think about it
and I can also send a v2 that disable tests in spice-common. This will
avoid this gdk-pixbuf dependency as well as the dynamic library
dependency (I'll move it to celt051). The only drawback is that I'll
have to add a hook to call autoreconf in subprojects/spice-common.
What do you think about this proposal?
>
> Regards,
> Yann E. MORIN.
>
> > libglib2 \
> > openssl \
> > pixman \
> > --
> > 2.23.0
> >
>
> --
> .-----------------.--------------------.------------------.--------------------.
> | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
> | +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
> '------------------------------^-------^------------------^--------------------'
Best Regards,
Fabrice
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Buildroot] [PATCH 1/5] package/spice: add optional gdk-pixbuf dependency
2019-11-15 20:27 ` Fabrice Fontaine
@ 2019-11-17 15:40 ` Arnout Vandecappelle
0 siblings, 0 replies; 9+ messages in thread
From: Arnout Vandecappelle @ 2019-11-17 15:40 UTC (permalink / raw)
To: buildroot
On 15/11/2019 21:27, Fabrice Fontaine wrote:
> Hi Yann,
>
> Le ven. 15 nov. 2019 ? 21:20, Yann E. MORIN <yann.morin.1998@free.fr> a ?crit :
>>
>> Fabrice, All,
>>
>> On 2019-11-14 20:13 +0100, Fabrice Fontaine spake thusly:
>>> This dependency has been added in version 0.14.1 with
>>> https://gitlab.freedesktop.org/spice/spice-common/commit/d58a5b6a5a25134396661e1d805dfc567181e3af
>>>
>>> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
>>> ---
>>> package/spice/spice.mk | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/package/spice/spice.mk b/package/spice/spice.mk
>>> index 16e57441a8..f5801ce0ff 100644
>>> --- a/package/spice/spice.mk
>>> +++ b/package/spice/spice.mk
>>> @@ -13,6 +13,7 @@ SPICE_INSTALL_STAGING = YES
>>> SPICE_DEPENDENCIES = \
>>> host-pkgconf \
>>> jpeg \
>>> + $(if $(BR2_PACKAGE_GDK_PIXBUF),gdk-pixbuf) \
>>
>> I see you've gine for conciseness here. However, I think we want to
>> handle this optional dependency like we handle the optional dependency
>> on opus, lines 53..56.
> OK, I can send a v2 with this change however I've also think about it
> and I can also send a v2 that disable tests in spice-common.
Disabling tests is clearly better, even without the considerations below.
> This will
> avoid this gdk-pixbuf dependency as well as the dynamic library
> dependency (I'll move it to celt051). The only drawback is that I'll
> have to add a hook to call autoreconf in subprojects/spice-common.
Why? A toplevel AUTORECONF=YES seems to work fine.
Regards,
Arnout
> What do you think about this proposal?
>>
>> Regards,
>> Yann E. MORIN.
>>
>>> libglib2 \
>>> openssl \
>>> pixman \
>>> --
>>> 2.23.0
>>>
>>
>> --
>> .-----------------.--------------------.------------------.--------------------.
>> | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
>> | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
>> | +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
>> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
>> '------------------------------^-------^------------------^--------------------'
> Best Regards,
>
> Fabrice
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
>
^ permalink raw reply [flat|nested] 9+ messages in thread