Message ID | 1400939383-10495-1-git-send-email-abrahamh@web.de |
---|---|
State | Accepted |
Headers | show |
Dear Heiko Abraham,
On Sat, 24 May 2014 15:49:43 +0200, Heiko Abraham wrote:
> +HWDATA_LICENSE = GPLv2 XFree86_1.0_license
This should be:
GPLv2+ or XFree86-1.0
The LICENSE file is pretty clear that it's one of the two licenses that
should be chosen. And XFree86-1.0 is, according to
http://spdx.org/licenses/, the proper encoding for the XFree86 1.0
License.
Thanks!
Thomas
>>>>> "Heiko" == Heiko Abraham <abrahamh@web.de> writes: > Add missing license info and file for package 'hwdata' > --- > package/hwdata/hwdata.mk | 2 ++ > 1 file changed, 2 insertions(+) > diff --git a/package/hwdata/hwdata.mk b/package/hwdata/hwdata.mk > index 501baec..d1044c4 100644 > --- a/package/hwdata/hwdata.mk > +++ b/package/hwdata/hwdata.mk > @@ -8,6 +8,8 @@ HWDATA_VERSION = 0.230 > HWDATA_SOURCE = hwdata_$(HWDATA_VERSION).orig.tar.gz > HWDATA_PATCH = hwdata_$(HWDATA_VERSION)-1.diff.gz > HWDATA_SITE = $(BR2_DEBIAN_MIRROR)/debian/pool/main/h/hwdata/ > +HWDATA_LICENSE = GPLv2 XFree86_1.0_license > +HWDATA_LICENSE_FILES = COPYING LICENSE LICENSE states that it is GPLv2+. We normally use 'or' between the licenses to clarify that you can choose and that it isn't a mix of both, so I've added that and committed, thanks.
diff --git a/package/hwdata/hwdata.mk b/package/hwdata/hwdata.mk index 501baec..d1044c4 100644 --- a/package/hwdata/hwdata.mk +++ b/package/hwdata/hwdata.mk @@ -8,6 +8,8 @@ HWDATA_VERSION = 0.230 HWDATA_SOURCE = hwdata_$(HWDATA_VERSION).orig.tar.gz HWDATA_PATCH = hwdata_$(HWDATA_VERSION)-1.diff.gz HWDATA_SITE = $(BR2_DEBIAN_MIRROR)/debian/pool/main/h/hwdata/ +HWDATA_LICENSE = GPLv2 XFree86_1.0_license +HWDATA_LICENSE_FILES = COPYING LICENSE define HWDATA_INSTALL_TARGET_CMDS install -D -m 644 $(@D)/pci.ids $(TARGET_DIR)/usr/share/hwdata/pci.ids