All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe
@ 2017-09-13 20:05 Fabien Lahoudere
  2017-09-13 20:05 ` [meta-multimedia][PATCH v4 1/2] libupnp: Add recipe for version 1.8.2 Fabien Lahoudere
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Fabien Lahoudere @ 2017-09-13 20:05 UTC (permalink / raw)
  To: openembedded-devel

In order to build gerbera, we need to support libupnp >= 1.8.
However mpd won't build with this version probably due to API break.

So we add a new recipe libupnp1.8 to be able to use both version.
Both version can be run in parallel.

?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-neon/libupnp_1.6.21-r0_armhf.deb 
drwxrwxrwx root/root         0 2017-09-13 17:26 ./
drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/
drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/lib/
-rwxr-xr-x root/root     31600 2017-09-13 17:26 ./usr/lib/libixml.so.2.0.8
-rwxr-xr-x root/root     17912 2017-09-13 17:26 ./usr/lib/libthreadutil.so.6.0.4
-rwxr-xr-x root/root    117676 2017-09-13 17:26 ./usr/lib/libupnp.so.6.3.4
lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libixml.so.2 -> libixml.so.2.0.8
lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libthreadutil.so.6 -> libthreadutil.so.6.0.4
lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libupnp.so.6 -> libupnp.so.6.3.4
?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-neon/libupnp1.8_1.8.2-r0_armhf.deb 
drwxrwxrwx root/root         0 2017-09-13 17:43 ./
drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/
drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/lib/
-rwxr-xr-x root/root     35696 2017-09-13 17:43 ./usr/lib/libixml.so.10.0.1
-rwxr-xr-x root/root    155128 2017-09-13 17:43 ./usr/lib/libupnp.so.10.1.1
lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libixml.so.10 -> libixml.so.10.0.1
lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libupnp.so.10 -> libupnp.so.10.1.1

mpd and gerbera have been built successfully with this fix.

The patch also clean a bit gerbera recipe and fix dependency issue.

Changes since v3:
	- Drop RDEPENDS from gerbera recipe.
	- use git fetcher instead of github archive for libupnp

Fabien Lahoudere (2):
  libupnp: Add recipe for version 1.8.2
  gerbera: clean recipe

 .../recipes-connectivity/libupnp/libupnp1.8.bb       | 20 ++++++++++++++++++++
 .../recipes-multimedia/gerbera/gerbera_git.bb        | 12 ++++++------
 2 files changed, 26 insertions(+), 6 deletions(-)
 create mode 100644 meta-multimedia/recipes-connectivity/libupnp/libupnp1.8.bb

-- 
2.11.0



^ permalink raw reply	[flat|nested] 8+ messages in thread

* [meta-multimedia][PATCH v4 1/2] libupnp: Add recipe for version 1.8.2
  2017-09-13 20:05 [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe Fabien Lahoudere
@ 2017-09-13 20:05 ` Fabien Lahoudere
  2017-09-13 20:05 ` [meta-multimedia][PATCH v4 2/2] gerbera: clean recipe Fabien Lahoudere
  2017-09-14  8:01 ` [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe Peter Kjellerstedt
  2 siblings, 0 replies; 8+ messages in thread
From: Fabien Lahoudere @ 2017-09-13 20:05 UTC (permalink / raw)
  To: openembedded-devel

This version is needed for gerbera, however this version break mpd.
So we keep both recipe.

Signed-off-by: Fabien Lahoudere <fabien.lahoudere@collabora.co.uk>
---
 .../recipes-connectivity/libupnp/libupnp1.8.bb       | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)
 create mode 100644 meta-multimedia/recipes-connectivity/libupnp/libupnp1.8.bb

diff --git a/meta-multimedia/recipes-connectivity/libupnp/libupnp1.8.bb b/meta-multimedia/recipes-connectivity/libupnp/libupnp1.8.bb
new file mode 100644
index 000000000..7a524f2d5
--- /dev/null
+++ b/meta-multimedia/recipes-connectivity/libupnp/libupnp1.8.bb
@@ -0,0 +1,20 @@
+SUMMARY = "Portable SDK for UPnP* Devices"
+DESCRIPTION = "The Portable SDK for UPnP Devices is an SDK for development of \
+UPnP device and control point applications. It consists of the core UPnP \
+protocols along with a UPnP-specific eXtensible Markup Language (XML) parser \
+supporting the Document Object Model (DOM) Level 2 API and an optional, \
+integrated mini web server for serving UPnP related documents."
+HOMEPAGE = "http://pupnp.sourceforge.net/"
+LICENSE = "BSD"
+
+LIC_FILES_CHKSUM = "file://LICENSE;md5=b3190d5244e08e78e4c8ee78544f4863"
+
+PV = "1.8.2+git${SRCPV}"
+SRCREV = "56d6042abae861e8838a4e6b6b5b575b99e38f34"
+SRC_URI = "git://github.com/mrjimenez/pupnp.git;protocol=https"
+
+S="${WORKDIR}/git"
+
+inherit autotools
+
+EXTRA_OECONF += "--enable-reuseaddr"
\ No newline at end of file
-- 
2.11.0



^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [meta-multimedia][PATCH v4 2/2] gerbera: clean recipe
  2017-09-13 20:05 [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe Fabien Lahoudere
  2017-09-13 20:05 ` [meta-multimedia][PATCH v4 1/2] libupnp: Add recipe for version 1.8.2 Fabien Lahoudere
@ 2017-09-13 20:05 ` Fabien Lahoudere
  2017-09-14  8:01 ` [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe Peter Kjellerstedt
  2 siblings, 0 replies; 8+ messages in thread
From: Fabien Lahoudere @ 2017-09-13 20:05 UTC (permalink / raw)
  To: openembedded-devel

Clean the recipe and change the name of libupnp to libupnp1.8.

Signed-off-by: Fabien Lahoudere <fabien.lahoudere@collabora.co.uk>
---
 meta-multimedia/recipes-multimedia/gerbera/gerbera_git.bb | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/meta-multimedia/recipes-multimedia/gerbera/gerbera_git.bb b/meta-multimedia/recipes-multimedia/gerbera/gerbera_git.bb
index 2bbedc0d1..c618d4a0c 100644
--- a/meta-multimedia/recipes-multimedia/gerbera/gerbera_git.bb
+++ b/meta-multimedia/recipes-multimedia/gerbera/gerbera_git.bb
@@ -6,22 +6,22 @@ LIC_FILES_CHKSUM = "file://LICENSE.md;md5=25cdec9afe3f1f26212ead6bd2f7fac8"
 SRC_URI = "git://github.com/v00d00/gerbera.git;protocol=https \
 	   file://gerbera.service "
 
-
-# Modify these as desired
 PV = "1.0+git${SRCPV}"
 SRCREV = "4435b165d121195aefd67d0287f7f9349e181d45"
 
 S = "${WORKDIR}/git"
 
-DEPENDS = "expat zlib curl libupnp e2fsprogs sqlite3 taglib "
-RDEPENDS_${PN} = "expat zlib curl libupnp e2fsprogs taglib "
+DEPENDS = "expat zlib curl libupnp1.8 e2fsprogs sqlite3 "
+#RDEPENDS_${PN} = "expat zlib curl libupnp e2fsprogs "
 
 SYSTEMD_SERVICE_${PN} = "gerbera.service"
-SYSTEMD_AUTO_ENABLE = "enable"
 
 inherit cmake systemd
 
-EXTRA_OECMAKE = "-DWITH_JS=FALSE -DWITH_TAGLIB=FALSE -DWITH_MAGIC=FALSE -DWITH_EXIF=FALSE -DLIBUUID_INCLUDE_DIRS=${WORKDIR}/recipe-sysroot/usr/include -DLIBUUID_LIBRARIES=-luuid"
+PACKAGECONFIG = ""
+PACKAGECONFIG[taglib] = "-DWITH_TAGLIB=TRUE,-DWITH_TAGLIB=FALSE,taglib"
+
+EXTRA_OECMAKE = "-DWITH_JS=FALSE -DWITH_MAGIC=FALSE -DWITH_EXIF=FALSE -DLIBUUID_INCLUDE_DIRS=${STAGING_INCDIR} -DLIBUUID_LIBRARIES=-luuid"
 
 do_install_append() {
 	install -d ${D}/root/.config/
-- 
2.11.0



^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe
  2017-09-13 20:05 [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe Fabien Lahoudere
  2017-09-13 20:05 ` [meta-multimedia][PATCH v4 1/2] libupnp: Add recipe for version 1.8.2 Fabien Lahoudere
  2017-09-13 20:05 ` [meta-multimedia][PATCH v4 2/2] gerbera: clean recipe Fabien Lahoudere
@ 2017-09-14  8:01 ` Peter Kjellerstedt
  2017-09-14 12:24   ` Fabien Lahoudere
  2017-09-14 14:46   ` Martin Jansa
  2 siblings, 2 replies; 8+ messages in thread
From: Peter Kjellerstedt @ 2017-09-14  8:01 UTC (permalink / raw)
  To: Fabien Lahoudere, openembedded-devel@lists.openembedded.org

> -----Original Message-----
> From: openembedded-devel-bounces@lists.openembedded.org
> [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of
> Fabien Lahoudere
> Sent: den 13 september 2017 22:06
> To: openembedded-devel@lists.openembedded.org
> Subject: [oe] [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe
> 
> In order to build gerbera, we need to support libupnp >= 1.8.
> However mpd won't build with this version probably due to API break.

Wouldn't it be more appropriate to copy the current libupnp 1.6 recipe 
to libupnp16 and then update the libupnp recipe to 1.8? Presumably we 
want to get rid of the 1.6 version whenever mpd is updated to support 
the new API.

> So we add a new recipe libupnp1.8 to be able to use both version.
> Both version can be run in parallel.
> 
> ?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-
> neon/libupnp_1.6.21-r0_armhf.deb
> drwxrwxrwx root/root         0 2017-09-13 17:26 ./
> drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/
> drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/lib/
> -rwxr-xr-x root/root     31600 2017-09-13 17:26
> ./usr/lib/libixml.so.2.0.8
> -rwxr-xr-x root/root     17912 2017-09-13 17:26
> ./usr/lib/libthreadutil.so.6.0.4
> -rwxr-xr-x root/root    117676 2017-09-13 17:26
> ./usr/lib/libupnp.so.6.3.4
> lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libixml.so.2
> -> libixml.so.2.0.8
> lrwxrwxrwx root/root         0 2017-09-13 17:26
> ./usr/lib/libthreadutil.so.6 -> libthreadutil.so.6.0.4
> lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libupnp.so.6
> -> libupnp.so.6.3.4
> ?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-
> neon/libupnp1.8_1.8.2-r0_armhf.deb
> drwxrwxrwx root/root         0 2017-09-13 17:43 ./
> drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/
> drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/lib/
> -rwxr-xr-x root/root     35696 2017-09-13 17:43
> ./usr/lib/libixml.so.10.0.1
> -rwxr-xr-x root/root    155128 2017-09-13 17:43
> ./usr/lib/libupnp.so.10.1.1
> lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libixml.so.10
> -> libixml.so.10.0.1
> lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libupnp.so.10
> -> libupnp.so.10.1.1
> 
> mpd and gerbera have been built successfully with this fix.
> 
> The patch also clean a bit gerbera recipe and fix dependency issue.
> 
> Changes since v3:
> 	- Drop RDEPENDS from gerbera recipe.
> 	- use git fetcher instead of github archive for libupnp
> 
> Fabien Lahoudere (2):
>   libupnp: Add recipe for version 1.8.2
>   gerbera: clean recipe
> 
>  .../recipes-connectivity/libupnp/libupnp1.8.bb       | 20
> ++++++++++++++++++++
>  .../recipes-multimedia/gerbera/gerbera_git.bb        | 12 ++++++------
>  2 files changed, 26 insertions(+), 6 deletions(-)
>  create mode 100644 meta-multimedia/recipes-
> connectivity/libupnp/libupnp1.8.bb
> 
> --
> 2.11.0

//Peter



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe
  2017-09-14  8:01 ` [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe Peter Kjellerstedt
@ 2017-09-14 12:24   ` Fabien Lahoudere
  2017-09-14 14:46   ` Martin Jansa
  1 sibling, 0 replies; 8+ messages in thread
From: Fabien Lahoudere @ 2017-09-14 12:24 UTC (permalink / raw)
  To: Peter Kjellerstedt, openembedded-devel@lists.openembedded.org

On Thu, 2017-09-14 at 08:01 +0000, Peter Kjellerstedt wrote:
> > -----Original Message-----
> > From: openembedded-devel-bounces@lists.openembedded.org
> > [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of
> > Fabien Lahoudere
> > Sent: den 13 september 2017 22:06
> > To: openembedded-devel@lists.openembedded.org
> > Subject: [oe] [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe
> > 
> > In order to build gerbera, we need to support libupnp >= 1.8.
> > However mpd won't build with this version probably due to API break.
> 
> Wouldn't it be more appropriate to copy the current libupnp 1.6 recipe 
> to libupnp16 and then update the libupnp recipe to 1.8? Presumably we 
> want to get rid of the 1.6 version whenever mpd is updated to support 
> the new API.

It is not a problem for me. Both solution are fine.
The decision concerns only maintainers and their preferences.

> 
> > So we add a new recipe libupnp1.8 to be able to use both version.
> > Both version can be run in parallel.
> > 
> > ?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-
> > neon/libupnp_1.6.21-r0_armhf.deb
> > drwxrwxrwx root/root         0 2017-09-13 17:26 ./
> > drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/
> > drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/lib/
> > -rwxr-xr-x root/root     31600 2017-09-13 17:26
> > ./usr/lib/libixml.so.2.0.8
> > -rwxr-xr-x root/root     17912 2017-09-13 17:26
> > ./usr/lib/libthreadutil.so.6.0.4
> > -rwxr-xr-x root/root    117676 2017-09-13 17:26
> > ./usr/lib/libupnp.so.6.3.4
> > lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libixml.so.2
> > -> libixml.so.2.0.8
> > lrwxrwxrwx root/root         0 2017-09-13 17:26
> > ./usr/lib/libthreadutil.so.6 -> libthreadutil.so.6.0.4
> > lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libupnp.so.6
> > -> libupnp.so.6.3.4
> > ?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-
> > neon/libupnp1.8_1.8.2-r0_armhf.deb
> > drwxrwxrwx root/root         0 2017-09-13 17:43 ./
> > drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/
> > drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/lib/
> > -rwxr-xr-x root/root     35696 2017-09-13 17:43
> > ./usr/lib/libixml.so.10.0.1
> > -rwxr-xr-x root/root    155128 2017-09-13 17:43
> > ./usr/lib/libupnp.so.10.1.1
> > lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libixml.so.10
> > -> libixml.so.10.0.1
> > lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libupnp.so.10
> > -> libupnp.so.10.1.1
> > 
> > mpd and gerbera have been built successfully with this fix.
> > 
> > The patch also clean a bit gerbera recipe and fix dependency issue.
> > 
> > Changes since v3:
> > 	- Drop RDEPENDS from gerbera recipe.
> > 	- use git fetcher instead of github archive for libupnp
> > 
> > Fabien Lahoudere (2):
> >   libupnp: Add recipe for version 1.8.2
> >   gerbera: clean recipe
> > 
> >  .../recipes-connectivity/libupnp/libupnp1.8.bb       | 20
> > ++++++++++++++++++++
> >  .../recipes-multimedia/gerbera/gerbera_git.bb        | 12 ++++++------
> >  2 files changed, 26 insertions(+), 6 deletions(-)
> >  create mode 100644 meta-multimedia/recipes-
> > connectivity/libupnp/libupnp1.8.bb
> > 
> > --
> > 2.11.0
> 
> //Peter
> 


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe
  2017-09-14  8:01 ` [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe Peter Kjellerstedt
  2017-09-14 12:24   ` Fabien Lahoudere
@ 2017-09-14 14:46   ` Martin Jansa
  2017-09-14 22:37     ` Khem Raj
  1 sibling, 1 reply; 8+ messages in thread
From: Martin Jansa @ 2017-09-14 14:46 UTC (permalink / raw)
  To: Peter Kjellerstedt; +Cc: openembedded-devel@lists.openembedded.org

[-- Attachment #1: Type: text/plain, Size: 3755 bytes --]

On Thu, Sep 14, 2017 at 08:01:49AM +0000, Peter Kjellerstedt wrote:
> > -----Original Message-----
> > From: openembedded-devel-bounces@lists.openembedded.org
> > [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of
> > Fabien Lahoudere
> > Sent: den 13 september 2017 22:06
> > To: openembedded-devel@lists.openembedded.org
> > Subject: [oe] [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe
> > 
> > In order to build gerbera, we need to support libupnp >= 1.8.
> > However mpd won't build with this version probably due to API break.
> 
> Wouldn't it be more appropriate to copy the current libupnp 1.6 recipe 
> to libupnp16 and then update the libupnp recipe to 1.8? Presumably we 
> want to get rid of the 1.6 version whenever mpd is updated to support 
> the new API.

Makes sense, I've renamed it in master-next.

I've also noticed that libupnp is optionally used by vlc, that's not
tested by my world builds, because PACKAGECONFIG for it isn't enabled by
default, but maybe vlc is also incompatible with libupnp 1.8.

> > So we add a new recipe libupnp1.8 to be able to use both version.
> > Both version can be run in parallel.
> > 
> > ?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-
> > neon/libupnp_1.6.21-r0_armhf.deb
> > drwxrwxrwx root/root         0 2017-09-13 17:26 ./
> > drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/
> > drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/lib/
> > -rwxr-xr-x root/root     31600 2017-09-13 17:26
> > ./usr/lib/libixml.so.2.0.8
> > -rwxr-xr-x root/root     17912 2017-09-13 17:26
> > ./usr/lib/libthreadutil.so.6.0.4
> > -rwxr-xr-x root/root    117676 2017-09-13 17:26
> > ./usr/lib/libupnp.so.6.3.4
> > lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libixml.so.2
> > -> libixml.so.2.0.8
> > lrwxrwxrwx root/root         0 2017-09-13 17:26
> > ./usr/lib/libthreadutil.so.6 -> libthreadutil.so.6.0.4
> > lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libupnp.so.6
> > -> libupnp.so.6.3.4
> > ?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-
> > neon/libupnp1.8_1.8.2-r0_armhf.deb
> > drwxrwxrwx root/root         0 2017-09-13 17:43 ./
> > drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/
> > drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/lib/
> > -rwxr-xr-x root/root     35696 2017-09-13 17:43
> > ./usr/lib/libixml.so.10.0.1
> > -rwxr-xr-x root/root    155128 2017-09-13 17:43
> > ./usr/lib/libupnp.so.10.1.1
> > lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libixml.so.10
> > -> libixml.so.10.0.1
> > lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libupnp.so.10
> > -> libupnp.so.10.1.1
> > 
> > mpd and gerbera have been built successfully with this fix.
> > 
> > The patch also clean a bit gerbera recipe and fix dependency issue.
> > 
> > Changes since v3:
> > 	- Drop RDEPENDS from gerbera recipe.
> > 	- use git fetcher instead of github archive for libupnp
> > 
> > Fabien Lahoudere (2):
> >   libupnp: Add recipe for version 1.8.2
> >   gerbera: clean recipe
> > 
> >  .../recipes-connectivity/libupnp/libupnp1.8.bb       | 20
> > ++++++++++++++++++++
> >  .../recipes-multimedia/gerbera/gerbera_git.bb        | 12 ++++++------
> >  2 files changed, 26 insertions(+), 6 deletions(-)
> >  create mode 100644 meta-multimedia/recipes-
> > connectivity/libupnp/libupnp1.8.bb
> > 
> > --
> > 2.11.0
> 
> //Peter
> 
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 201 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe
  2017-09-14 14:46   ` Martin Jansa
@ 2017-09-14 22:37     ` Khem Raj
  2017-09-14 22:52       ` Khem Raj
  0 siblings, 1 reply; 8+ messages in thread
From: Khem Raj @ 2017-09-14 22:37 UTC (permalink / raw)
  To: Martin Jansa
  Cc: openembedded-devel@lists.openembedded.org, Peter Kjellerstedt

On Thu, Sep 14, 2017 at 7:46 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> On Thu, Sep 14, 2017 at 08:01:49AM +0000, Peter Kjellerstedt wrote:
>> > -----Original Message-----
>> > From: openembedded-devel-bounces@lists.openembedded.org
>> > [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of
>> > Fabien Lahoudere
>> > Sent: den 13 september 2017 22:06
>> > To: openembedded-devel@lists.openembedded.org
>> > Subject: [oe] [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe
>> >
>> > In order to build gerbera, we need to support libupnp >= 1.8.
>> > However mpd won't build with this version probably due to API break.
>>
>> Wouldn't it be more appropriate to copy the current libupnp 1.6 recipe
>> to libupnp16 and then update the libupnp recipe to 1.8? Presumably we
>> want to get rid of the 1.6 version whenever mpd is updated to support
>> the new API.
>
> Makes sense, I've renamed it in master-next.
>

you forgot to rename the patch directory which should now be called
libupnp1 as well.

> I've also noticed that libupnp is optionally used by vlc, that's not
> tested by my world builds, because PACKAGECONFIG for it isn't enabled by
> default, but maybe vlc is also incompatible with libupnp 1.8.
>
>> > So we add a new recipe libupnp1.8 to be able to use both version.
>> > Both version can be run in parallel.
>> >
>> > ?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-
>> > neon/libupnp_1.6.21-r0_armhf.deb
>> > drwxrwxrwx root/root         0 2017-09-13 17:26 ./
>> > drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/
>> > drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/lib/
>> > -rwxr-xr-x root/root     31600 2017-09-13 17:26
>> > ./usr/lib/libixml.so.2.0.8
>> > -rwxr-xr-x root/root     17912 2017-09-13 17:26
>> > ./usr/lib/libthreadutil.so.6.0.4
>> > -rwxr-xr-x root/root    117676 2017-09-13 17:26
>> > ./usr/lib/libupnp.so.6.3.4
>> > lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libixml.so.2
>> > -> libixml.so.2.0.8
>> > lrwxrwxrwx root/root         0 2017-09-13 17:26
>> > ./usr/lib/libthreadutil.so.6 -> libthreadutil.so.6.0.4
>> > lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libupnp.so.6
>> > -> libupnp.so.6.3.4
>> > ?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-
>> > neon/libupnp1.8_1.8.2-r0_armhf.deb
>> > drwxrwxrwx root/root         0 2017-09-13 17:43 ./
>> > drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/
>> > drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/lib/
>> > -rwxr-xr-x root/root     35696 2017-09-13 17:43
>> > ./usr/lib/libixml.so.10.0.1
>> > -rwxr-xr-x root/root    155128 2017-09-13 17:43
>> > ./usr/lib/libupnp.so.10.1.1
>> > lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libixml.so.10
>> > -> libixml.so.10.0.1
>> > lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libupnp.so.10
>> > -> libupnp.so.10.1.1
>> >
>> > mpd and gerbera have been built successfully with this fix.
>> >
>> > The patch also clean a bit gerbera recipe and fix dependency issue.
>> >
>> > Changes since v3:
>> >     - Drop RDEPENDS from gerbera recipe.
>> >     - use git fetcher instead of github archive for libupnp
>> >
>> > Fabien Lahoudere (2):
>> >   libupnp: Add recipe for version 1.8.2
>> >   gerbera: clean recipe
>> >
>> >  .../recipes-connectivity/libupnp/libupnp1.8.bb       | 20
>> > ++++++++++++++++++++
>> >  .../recipes-multimedia/gerbera/gerbera_git.bb        | 12 ++++++------
>> >  2 files changed, 26 insertions(+), 6 deletions(-)
>> >  create mode 100644 meta-multimedia/recipes-
>> > connectivity/libupnp/libupnp1.8.bb
>> >
>> > --
>> > 2.11.0
>>
>> //Peter
>>
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>
> --
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe
  2017-09-14 22:37     ` Khem Raj
@ 2017-09-14 22:52       ` Khem Raj
  0 siblings, 0 replies; 8+ messages in thread
From: Khem Raj @ 2017-09-14 22:52 UTC (permalink / raw)
  To: Martin Jansa
  Cc: openembedded-devel@lists.openembedded.org, Peter Kjellerstedt

On Thu, Sep 14, 2017 at 3:37 PM, Khem Raj <raj.khem@gmail.com> wrote:
> On Thu, Sep 14, 2017 at 7:46 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
>> On Thu, Sep 14, 2017 at 08:01:49AM +0000, Peter Kjellerstedt wrote:
>>> > -----Original Message-----
>>> > From: openembedded-devel-bounces@lists.openembedded.org
>>> > [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of
>>> > Fabien Lahoudere
>>> > Sent: den 13 september 2017 22:06
>>> > To: openembedded-devel@lists.openembedded.org
>>> > Subject: [oe] [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe
>>> >
>>> > In order to build gerbera, we need to support libupnp >= 1.8.
>>> > However mpd won't build with this version probably due to API break.
>>>
>>> Wouldn't it be more appropriate to copy the current libupnp 1.6 recipe
>>> to libupnp16 and then update the libupnp recipe to 1.8? Presumably we
>>> want to get rid of the 1.6 version whenever mpd is updated to support
>>> the new API.
>>
>> Makes sense, I've renamed it in master-next.
>>
>
> you forgot to rename the patch directory which should now be called
> libupnp1 as well.
>

additionally -dev package from both versions will be in conflict too,
this issue is
less prevalent now that we have rss, but we will have this issue once
you install
the dev packages into image or generate SDKs from images which have both
libupnp versions

>> I've also noticed that libupnp is optionally used by vlc, that's not
>> tested by my world builds, because PACKAGECONFIG for it isn't enabled by
>> default, but maybe vlc is also incompatible with libupnp 1.8.
>>
>>> > So we add a new recipe libupnp1.8 to be able to use both version.
>>> > Both version can be run in parallel.
>>> >
>>> > ?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-
>>> > neon/libupnp_1.6.21-r0_armhf.deb
>>> > drwxrwxrwx root/root         0 2017-09-13 17:26 ./
>>> > drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/
>>> > drwxr-xr-x root/root         0 2017-09-13 17:26 ./usr/lib/
>>> > -rwxr-xr-x root/root     31600 2017-09-13 17:26
>>> > ./usr/lib/libixml.so.2.0.8
>>> > -rwxr-xr-x root/root     17912 2017-09-13 17:26
>>> > ./usr/lib/libthreadutil.so.6.0.4
>>> > -rwxr-xr-x root/root    117676 2017-09-13 17:26
>>> > ./usr/lib/libupnp.so.6.3.4
>>> > lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libixml.so.2
>>> > -> libixml.so.2.0.8
>>> > lrwxrwxrwx root/root         0 2017-09-13 17:26
>>> > ./usr/lib/libthreadutil.so.6 -> libthreadutil.so.6.0.4
>>> > lrwxrwxrwx root/root         0 2017-09-13 17:26 ./usr/lib/libupnp.so.6
>>> > -> libupnp.so.6.3.4
>>> > ?:~/src/oe-core/build$ dpkg-deb -c tmp-glibc/deploy/deb/cortexa9hf-
>>> > neon/libupnp1.8_1.8.2-r0_armhf.deb
>>> > drwxrwxrwx root/root         0 2017-09-13 17:43 ./
>>> > drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/
>>> > drwxr-xr-x root/root         0 2017-09-13 17:43 ./usr/lib/
>>> > -rwxr-xr-x root/root     35696 2017-09-13 17:43
>>> > ./usr/lib/libixml.so.10.0.1
>>> > -rwxr-xr-x root/root    155128 2017-09-13 17:43
>>> > ./usr/lib/libupnp.so.10.1.1
>>> > lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libixml.so.10
>>> > -> libixml.so.10.0.1
>>> > lrwxrwxrwx root/root         0 2017-09-13 17:43 ./usr/lib/libupnp.so.10
>>> > -> libupnp.so.10.1.1
>>> >
>>> > mpd and gerbera have been built successfully with this fix.
>>> >
>>> > The patch also clean a bit gerbera recipe and fix dependency issue.
>>> >
>>> > Changes since v3:
>>> >     - Drop RDEPENDS from gerbera recipe.
>>> >     - use git fetcher instead of github archive for libupnp
>>> >
>>> > Fabien Lahoudere (2):
>>> >   libupnp: Add recipe for version 1.8.2
>>> >   gerbera: clean recipe
>>> >
>>> >  .../recipes-connectivity/libupnp/libupnp1.8.bb       | 20
>>> > ++++++++++++++++++++
>>> >  .../recipes-multimedia/gerbera/gerbera_git.bb        | 12 ++++++------
>>> >  2 files changed, 26 insertions(+), 6 deletions(-)
>>> >  create mode 100644 meta-multimedia/recipes-
>>> > connectivity/libupnp/libupnp1.8.bb
>>> >
>>> > --
>>> > 2.11.0
>>>
>>> //Peter
>>>
>>> --
>>> _______________________________________________
>>> Openembedded-devel mailing list
>>> Openembedded-devel@lists.openembedded.org
>>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>>
>> --
>> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
>>
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>>


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2017-09-14 22:52 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-13 20:05 [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe Fabien Lahoudere
2017-09-13 20:05 ` [meta-multimedia][PATCH v4 1/2] libupnp: Add recipe for version 1.8.2 Fabien Lahoudere
2017-09-13 20:05 ` [meta-multimedia][PATCH v4 2/2] gerbera: clean recipe Fabien Lahoudere
2017-09-14  8:01 ` [meta-multimedia][PATCH v4 0/2] Add libupnp1.8 recipe Peter Kjellerstedt
2017-09-14 12:24   ` Fabien Lahoudere
2017-09-14 14:46   ` Martin Jansa
2017-09-14 22:37     ` Khem Raj
2017-09-14 22:52       ` Khem Raj

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.