qt: add font license information
PATCH v2 1/1 qt: add font license information
---
Changes v1 -> v2:
- update most font licenses (thanks to Arnout Vandecappelle)
- add comma-separator between the licenses (no nice solution, will add ' ,', but without the starting problem at:
QT_LICENSE = LGPLv2.1 with exceptions or GPLv3 ifneq ($(BR2_PACKAGE_QT_LICENSE_APPROVED),y) QT_LICENSE += or Digia Qt Commercial license endif
--- package/qt/qt.mk | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+)
package/qt/qt.mk +20 -0
@@ -660,13 +660,33 @@ define QT_INSTALL_TARGET_FONTS mkdir -p $(TARGET_DIR)/usr/lib/fonts cp -dpf $(QT_FONTS) $(TARGET_DIR)/usr/lib/fonts endef +ifeq ($(BR2_PACKAGE_QT_FONT_MICRO),y) +# Public Domain as stated in the font source src/3rdparty/fonts/micro.bdf +QT_LICENSE += , Public Domain (Micro font) endif +ifeq ($(BR2_PACKAGE_QT_FONT_FIXED),y) +# according to https://launchpad.net/ubuntu/vivid/+source/qtbase-opensource-src/+copyright +QT_LICENSE += , Public Domain (Fixed font) +endif +ifneq ($(BR2_PACKAGE_QT_FONT_HELVETICA)$(BR2_PACKAGE_QT_FONT_JAPANESE),) +QT_LICENSE += , Adobe Helvetica license (Helvetica/Japanese fonts) +QT_LICENSE_FILES += src/3rdparty/fonts/COPYING.Helvetica +endif +ifeq ($(BR2_PACKAGE_QT_FONT_UNIFONT),y) +QT_LICENSE += , Freeware (Unifont font) +QT_LICENSE_FILES += src/3rdparty/fonts/COPYRIGHT.Unifont +endif +endif # QT_FONTS ifeq ($(BR2_PACKAGE_QT_QTFREETYPE)$(BR2_PACKAGE_QT_SYSTEMFREETYPE),y) define QT_INSTALL_TARGET_FONTS_TTF mkdir -p $(TARGET_DIR)/usr/lib/fonts cp -dpf $(STAGING_DIR)/usr/lib/fonts/*.ttf $(TARGET_DIR)/usr/lib/fonts endef +QT_LICENSE += , Bitstream license (DejaVu/Vera TrueType fonts) +QT_LICENSE_FILES += src/3rdparty/fonts/COPYRIGHT.DejaVu \ + src/3rdparty/fonts/README.DejaVu \ + src/3rdparty/fonts/COPYRIGHT.Vera endif endif # BR2_PACKAGE_QT_EMBEDDED
Signature · 1 line
2.1.4
PATCH v2 1/1 qt: add font license information
On 02/23/16 22:52, Peter Seiderer wrote:
I would mention the discussion about the commas in the commit log, like this:
-----
With this implementation, the different licenses will be separated with <space><comma>. This is not nice, but to avoid it a pretty ugly construct is needed:QT_LICENSE = $(shell echo $(QT_LICENSE_BITS) | sed 's% , %, %g')
Since the <space><comma> is just a minor nuisance, we're not going to that kind of complexity.
-----
Quoted 20 lines
Signed-off-by Peter Seiderer <[email protected]>---
Changes v1 -> v2:
- update most font licenses (thanks to Arnout Vandecappelle)
- add comma-separator between the licenses (no nice solution, will add ' ,', but without the starting problem at:
QT_LICENSE= LGPLv2.1 with exceptions or GPLv3 ifneq ($(BR2_PACKAGE_QT_LICENSE_APPROVED),y)QT_LICENSE+= or Digia Qt Commercial license endif--- package/qt/qt.mk | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+)package/qt/qt.mk +0
-0
package/qt/qt.mk define QT_INSTALL_TARGET_FONTS · line 668
Quoted 9 lines
package/qt/qt.mk +5
-0@@ -660,13 +660,33 @@ define QT_INSTALL_TARGET_FONTS mkdir -p $(TARGET_DIR)/usr/lib/fonts cp -dpf $(QT_FONTS) $(TARGET_DIR)/usr/lib/fonts endef +ifeq ($(BR2_PACKAGE_QT_FONT_MICRO),y) +# Public Domain as stated in the font source src/3rdparty/fonts/micro.bdf +QT_LICENSE += , Public Domain (Micro font) endif +ifeq ($(BR2_PACKAGE_QT_FONT_FIXED),y) +# according to https://launchpad.net/ubuntu/vivid/+source/qtbase-opensource-src/+copyright
Given that they were wrong before, this isn't a great reference :-) But it is
in fact specified in src/3rdparty/fonts/5x7.bdf
package/qt/qt.mk define QT_INSTALL_TARGET_FONTS · line 669
Quoted 1 line
package/qt/qt.mk +1
-0+QT_LICENSE += , Public Domain (Fixed font)
Same issue as with helvetica/japanese: same license is specified twice. So use
the same approach as below.
Quoted 24 lines
package/qt/qt.mk +14
-0+endif +ifneq ($(BR2_PACKAGE_QT_FONT_HELVETICA)$(BR2_PACKAGE_QT_FONT_JAPANESE),) +QT_LICENSE += , Adobe Helvetica license (Helvetica/Japanese fonts) +QT_LICENSE_FILES += src/3rdparty/fonts/COPYING.Helvetica +endif +ifeq ($(BR2_PACKAGE_QT_FONT_UNIFONT),y) +QT_LICENSE += , Freeware (Unifont font) +QT_LICENSE_FILES += src/3rdparty/fonts/COPYRIGHT.Unifont +endif +endif # QT_FONTS ifeq ($(BR2_PACKAGE_QT_QTFREETYPE)$(BR2_PACKAGE_QT_SYSTEMFREETYPE),y) define QT_INSTALL_TARGET_FONTS_TTF mkdir -p $(TARGET_DIR)/usr/lib/fonts cp -dpf $(STAGING_DIR)/usr/lib/fonts/*.ttf $(TARGET_DIR)/usr/lib/fonts endef +QT_LICENSE += , Bitstream license (DejaVu/Vera TrueType fonts) +QT_LICENSE_FILES += src/3rdparty/fonts/COPYRIGHT.DejaVu \ + src/3rdparty/fonts/README.DejaVu \ + src/3rdparty/fonts/COPYRIGHT.Vera endif endif # BR2_PACKAGE_QT_EMBEDDED
Signature · 6 lines
Arnout Vandecappelle arnout at mind be Senior Embedded Software Architect +32-16-286500 Essensium/Mind http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
PATCH v2 1/1 qt: add font license information
Hello Arnout,
Quoted 4 lines · Arnout Vandecappelle
On 02/23/16 22:52, Peter Seiderer wrote:
I would mention the discussion about the commas in the commit log, like this:
Quoted 8 lines
-----
With this implementation, the different licenses will be separated with <space><comma>. This is not nice, but to avoid it a pretty ugly construct is needed:QT_LICENSE= $(shell echo $(QT_LICENSE_BITS) | sed 's% , %, %g')Since the <space><comma> is just a minor nuisance, we're not going to that kind of complexity.
-----
O.k. will add this to the commit log...
Quoted 25 lines
Quoted 25 lines
Signed-off-by Peter Seiderer <[email protected]>---
Changes v1 -> v2:
- update most font licenses (thanks to Arnout Vandecappelle)
- add comma-separator between the licenses (no nice solution, will add ' ,', but without the starting problem at:
QT_LICENSE= LGPLv2.1 with exceptions or GPLv3 ifneq ($(BR2_PACKAGE_QT_LICENSE_APPROVED),y)QT_LICENSE+= or Digia Qt Commercial license endif--- package/qt/qt.mk | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+)package/qt/qt.mk +1
-0@@ -660,13 +660,33 @@ define QT_INSTALL_TARGET_FONTS mkdir -p $(TARGET_DIR)/usr/lib/fonts cp -dpf $(QT_FONTS) $(TARGET_DIR)/usr/lib/fonts endef +ifeq ($(BR2_PACKAGE_QT_FONT_MICRO),y)
Quoted 9 lines
package/qt/qt.mk define QT_INSTALL_TARGET_FONTS · line 668
Quoted 5 lines
package/qt/qt.mk +4
-0+# Public Domain as stated in the font source src/3rdparty/fonts/micro.bdf +QT_LICENSE += , Public Domain (Micro font) endif +ifeq ($(BR2_PACKAGE_QT_FONT_FIXED),y) +# according to https://launchpad.net/ubuntu/vivid/+source/qtbase-opensource-src/+copyrightGiven that they were wrong before, this isn't a great reference :-) But it is
in fact specified in src/3rdparty/fonts/5x7.bdf
O.k, thanks for the hint...
Quoted 5 lines
package/qt/qt.mk define QT_INSTALL_TARGET_FONTS · line 669
Quoted 1 line
package/qt/qt.mk +1
-0+QT_LICENSE += , Public Domain (Fixed font)Same issue as with helvetica/japanese: same license is specified twice. So use
the same approach as below.
O.k. will do...
Regards,
Peter
Quoted 26 lines
Quoted 24 lines
package/qt/qt.mk +14
-0+endif +ifneq ($(BR2_PACKAGE_QT_FONT_HELVETICA)$(BR2_PACKAGE_QT_FONT_JAPANESE),) +QT_LICENSE += , Adobe Helvetica license (Helvetica/Japanese fonts) +QT_LICENSE_FILES += src/3rdparty/fonts/COPYING.Helvetica +endif +ifeq ($(BR2_PACKAGE_QT_FONT_UNIFONT),y) +QT_LICENSE += , Freeware (Unifont font) +QT_LICENSE_FILES += src/3rdparty/fonts/COPYRIGHT.Unifont +endif +endif # QT_FONTS ifeq ($(BR2_PACKAGE_QT_QTFREETYPE)$(BR2_PACKAGE_QT_SYSTEMFREETYPE),y) define QT_INSTALL_TARGET_FONTS_TTF mkdir -p $(TARGET_DIR)/usr/lib/fonts cp -dpf $(STAGING_DIR)/usr/lib/fonts/*.ttf $(TARGET_DIR)/usr/lib/fonts endef +QT_LICENSE += , Bitstream license (DejaVu/Vera TrueType fonts) +QT_LICENSE_FILES += src/3rdparty/fonts/COPYRIGHT.DejaVu \ + src/3rdparty/fonts/README.DejaVu \ + src/3rdparty/fonts/COPYRIGHT.Vera endif endif # BR2_PACKAGE_QT_EMBEDDED