Skip to contentSkip to conversation

Fix avahi x11 dependency with non-GPLv3 Build

PATCH Superseded Saul Wold 3 participants 5 messages Sep 19, 2011

v1 of 3 Superseded Latest is v3 →

lore
Share & tools
lore

Saul Wold

author
#0 lore raw

cover Fix avahi x11 dependency with non-GPLv3 Build

This address an issue where a non-GPLv3 build was blocked due to the DISTRO_FEATURE of x11 being enabled.

The following changes since commit 81274f4488fbc4d68d150870735ec0181b60b451:

freetype: disable bzip2 compressed font support (2011-09-19 13:14:03 +0100)

are available in the git repository at:
git://git.yoctoproject.org/poky-contrib sgw/oe-fix
http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=sgw/oe-fix

Saul Wold (1):
avahi: Check for GPLv3 license and disable x11

 meta/recipes-connectivity/avahi/avahi.inc       |    8 ++++++++
 meta/recipes-connectivity/avahi/avahi_0.6.30.bb |    2 +-
 2 files changed, 9 insertions(+), 1 deletions(-)
Signature · 1 line
1.7.6
author
#1 lore raw

PATCH 1/1 avahi: Check for GPLv3 license and disable x11

[YOCTO #1492]

Since the non-GPLv3 build does not want gnome, which this will pull in by the default DISTRO_FEATURE x11 being set and enabling gtk for avahi, we filter x11 out of DISTRO_FEATURES here when INCOMPATIBLE_LICENSE is set to GPLv3.

---
 meta/recipes-connectivity/avahi/avahi.inc       |    8 ++++++++
 meta/recipes-connectivity/avahi/avahi_0.6.30.bb |    2 +-
 2 files changed, 9 insertions(+), 1 deletions(-)
meta/recipes-connectivity/avahi/avahi.inc +8 -0
index 24ea458..c5c0c7c 100644
@@ -14,6 +14,8 @@ SECTION = "network"
 # python scripts are under GPLv2+
 LICENSE = "GPLv2+ & LGPLv2.1+"
 
+INC_PR = "r5"
+
 X11DEPENDS = "gtk+ libglade"
 DEPENDS = "expat libcap libdaemon dbus glib-2.0 ${@base_contains('DISTRO_FEATURES', 'x11', '${X11DEPENDS}', '', d)}"
 
@@ -39,6 +41,12 @@ EXTRA_OECONF = "--with-distro=debian \
              --with-systemdsystemunitdir=${base_libdir}/systemd/system/ \
 "
 
+python() {
+    if (d.getVar("INCOMPATIBLE_LICENSE", True) or "").find("GPLv3") != -1:
+        d.setVar("DISTRO_FEATURES", oe_filter_out('x11', d.getVar("DISTRO_FEATURES", True), d))
+
+}
+
 PACKAGES =+ "avahi-systemd avahi-daemon libavahi-common libavahi-core libavahi-client avahi-dnsconfd libavahi-glib libavahi-gobject avahi-autoipd avahi-ui-utils avahi-utils libavahi-ui libavahi-ui-dev"
 
 FILES_avahi-systemd = "${base_libdir}/systemd"
meta/recipes-connectivity/avahi/avahi_0.6.30.bb +1 -1
index 05716d0..5094961 100644
@@ -6,7 +6,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=2d5025d4aa3495befef8f17206a5b0a1 \
                     file://avahi-daemon/main.c;endline=21;md5=9ee77368c5407af77caaef1b07285969 \
                     file://avahi-client/client.h;endline=23;md5=f4ac741a25c4f434039ba3e18c8674cf"
 
-PR = "r4"
+PR = "${INC_PR}.0"
 
 SRC_URI[md5sum] = "e4db89a2a403ff4c47d66ac66fad1f43"
 SRC_URI[sha256sum] = "f9e4316c2339d0020726edd846d01bee0c39980906db0c247479e5807457ff1f"
Signature · 1 line
1.7.6
#2 lore raw

Re: [PATCH 1/1] avahi: Check for GPLv3 license and disable x11

Quoted 6 lines · Saul Wold

[YOCTO #1492]

Since the non-GPLv3 build does not want gnome, which this will pull in by the default DISTRO_FEATURE x11 being set and enabling gtk for avahi, we filter x11 out of DISTRO_FEATURES here when INCOMPATIBLE_LICENSE is set to GPLv3.

If the avahi GTK UI app is needed when X11 is enabled, this should be a decent enough way to work around the condition.

However, I question if avahi should -ever- be producing the gtk component.

From the investigation I did, the gtk component is something that helps with discovering mDNS servers (either prompting the user or simply displaying status, I'm not sure). Is this of use to anyone using oe-core? Otherwise I'd be in favor of simply removing the X11 option from the avahi package.

(I talked with Saul about this already on IRC.. he's concerned that we'd be turning of required [by someone] functionality... I'm really not sure it is required by anyone...)

--Mark

Quoted 45 lines
---
 meta/recipes-connectivity/avahi/avahi.inc       |    8 ++++++++
 meta/recipes-connectivity/avahi/avahi_0.6.30.bb |    2 +-
 2 files changed, 9 insertions(+), 1 deletions(-)
meta/recipes-connectivity/avahi/avahi.inc +8 -0
index 24ea458..c5c0c7c 100644
@@ -14,6 +14,8 @@ SECTION = "network"
 # python scripts are under GPLv2+
 LICENSE = "GPLv2+ & LGPLv2.1+"
 
+INC_PR = "r5"
+
 X11DEPENDS = "gtk+ libglade"
 DEPENDS = "expat libcap libdaemon dbus glib-2.0 ${@base_contains('DISTRO_FEATURES', 'x11', '${X11DEPENDS}', '', d)}"
 
@@ -39,6 +41,12 @@ EXTRA_OECONF = "--with-distro=debian \
              --with-systemdsystemunitdir=${base_libdir}/systemd/system/ \
 "
 
+python() {
+    if (d.getVar("INCOMPATIBLE_LICENSE", True) or "").find("GPLv3") != -1:
+        d.setVar("DISTRO_FEATURES", oe_filter_out('x11', d.getVar("DISTRO_FEATURES", True), d))
+
+}
+
 PACKAGES =+ "avahi-systemd avahi-daemon libavahi-common libavahi-core libavahi-client avahi-dnsconfd libavahi-glib libavahi-gobject avahi-autoipd avahi-ui-utils avahi-utils libavahi-ui libavahi-ui-dev"
 
 FILES_avahi-systemd = "${base_libdir}/systemd"
meta/recipes-connectivity/avahi/avahi_0.6.30.bb +1 -1
index 05716d0..5094961 100644
@@ -6,7 +6,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=2d5025d4aa3495befef8f17206a5b0a1 \
                     file://avahi-daemon/main.c;endline=21;md5=9ee77368c5407af77caaef1b07285969 \
                     file://avahi-client/client.h;endline=23;md5=f4ac741a25c4f434039ba3e18c8674cf"
 
-PR = "r4"
+PR = "${INC_PR}.0"
 
 SRC_URI[md5sum] = "e4db89a2a403ff4c47d66ac66fad1f43"
 SRC_URI[sha256sum] = "f9e4316c2339d0020726edd846d01bee0c39980906db0c247479e5807457ff1f"
#3 lore raw

Re: [PATCH 1/1] avahi: Check for GPLv3 license and disable x11

Op 19 sep. 2011, om 21:05 heeft Mark Hatle het volgende geschreven:

Quoted 13 lines · Saul Wold
Quoted 6 lines

[YOCTO #1492]

Since the non-GPLv3 build does not want gnome, which this will pull in by the default DISTRO_FEATURE x11 being set and enabling gtk for avahi, we filter x11 out of DISTRO_FEATURES here when INCOMPATIBLE_LICENSE is set to GPLv3.

If the avahi GTK UI app is needed when X11 is enabled, this should be a decent enough way to work around the condition.

However, I question if avahi should -ever- be producing the gtk component.

Quoted 8 lines

From the investigation I did, the gtk component is something that helps with discovering mDNS servers (either prompting the user or simply displaying status, I'm not sure). Is this of use to anyone using oe-core? Otherwise I'd be in favor of simply removing the X11 option from the avahi package.

(I talked with Saul about this already on IRC.. he's concerned that we'd be turning of required [by someone] functionality... I'm really not sure it is required by anyone...)

In OE classic the UI stuff in in avahi-ui.bb

author
#4 lore raw

Re: [PATCH 1/1] avahi: Check for GPLv3 license and disable x11

Quoted 21 lines · Koen Kooi

Op 19 sep. 2011, om 21:05 heeft Mark Hatle het volgende geschreven:

Quoted 18 lines · Saul Wold
Quoted 6 lines

[YOCTO #1492]

Since the non-GPLv3 build does not want gnome, which this will pull in by the default DISTRO_FEATURE x11 being set and enabling gtk for avahi, we filter x11 out of DISTRO_FEATURES here when INCOMPATIBLE_LICENSE is set to GPLv3.

If the avahi GTK UI app is needed when X11 is enabled, this should be a decent enough way to work around the condition.

However, I question if avahi should -ever- be producing the gtk component.

From the investigation I did, the gtk component is something that helps with
discovering mDNS servers (either prompting the user or simply displaying status, I'm not sure). Is this of use to anyone using oe-core? Otherwise I'd be in favor of simply removing the X11 option from the avahi package.

Quoted 5 lines
Quoted 3 lines

(I talked with Saul about this already on IRC.. he's concerned that we'd be turning of required [by someone] functionality... I'm really not sure it is required by anyone...)

In OE classic the UI stuff in in avahi-ui.bb

Clearly, a better way to go, I think my patch will work for the short term of 1.1 release, but for master moving forward I will integrate the avahi-ui.bb recipe. We just don't have the time for a major recipe re-write.

Sau!

Quoted 5 lines

_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core