* [Buildroot] [PATCH v1] qt: add font license information
@ 2015-06-12 20:22 Peter Seiderer
2016-02-22 23:09 ` Arnout Vandecappelle
0 siblings, 1 reply; 6+ messages in thread
From: Peter Seiderer @ 2015-06-12 20:22 UTC (permalink / raw)
To: buildroot
Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
Not sure about the japanese font and about MIT or MIT-like
licenses (no exact match of the wording to other MIT licenses).
---
package/qt/qt.mk | 30 ++++++++++++++++++++++++++++++
1 file changed, 30 insertions(+)
diff --git a/package/qt/qt.mk b/package/qt/qt.mk
index a7bbe5f..3fbb3e1 100644
--- a/package/qt/qt.mk
+++ b/package/qt/qt.mk
@@ -656,6 +656,32 @@ 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)
+QT_LICENSE += Public Domain (Micro font)
+# disabled because this file (containing the Public Domain notice) is
+# the font source file too...
+#QT_LICENSE_FILES += src/3rdparty/fonts/micro.bdf
+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
+ifeq ($(BR2_PACKAGE_QT_FONT_HELVETICA),y)
+QT_LICENSE += MIT-like (Helvetica fonts)
+QT_LICENSE_FILES += src/3rdparty/fonts/COPYING.Helvetica
+endif
+ifeq ($(BR2_PACKAGE_QT_FONT_JAPANESE),y)
+# according to https://launchpad.net/ubuntu/vivid/+source/qtbase-opensource-src/+copyright
+# Public Domain
+QT_LICENSE += Public Domain (Japanese font)
+# according to lib/fonts/README same as Helvetica fonts?
+#QT_LICENSE += MIT-like (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
ifeq ($(BR2_PACKAGE_QT_QTFREETYPE)$(BR2_PACKAGE_QT_SYSTEMFREETYPE),y)
@@ -663,6 +689,10 @@ 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 += MIT-like (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
--
2.1.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v1] qt: add font license information
2015-06-12 20:22 [Buildroot] [PATCH v1] qt: add font license information Peter Seiderer
@ 2016-02-22 23:09 ` Arnout Vandecappelle
2016-02-23 12:51 ` Thomas Petazzoni
2016-02-23 20:02 ` Peter Seiderer
0 siblings, 2 replies; 6+ messages in thread
From: Arnout Vandecappelle @ 2016-02-22 23:09 UTC (permalink / raw)
To: buildroot
On 06/12/15 22:22, Peter Seiderer wrote:
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> ---
> Not sure about the japanese font and about MIT or MIT-like
> licenses (no exact match of the wording to other MIT licenses).
> ---
> package/qt/qt.mk | 30 ++++++++++++++++++++++++++++++
> 1 file changed, 30 insertions(+)
>
> diff --git a/package/qt/qt.mk b/package/qt/qt.mk
> index a7bbe5f..3fbb3e1 100644
> --- a/package/qt/qt.mk
> +++ b/package/qt/qt.mk
> @@ -656,6 +656,32 @@ 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)
> +QT_LICENSE += Public Domain (Micro font)
> +# disabled because this file (containing the Public Domain notice) is
> +# the font source file too...
> +#QT_LICENSE_FILES += src/3rdparty/fonts/micro.bdf
IMHO LICENSE_FILES are not needed for Public Domain. The point of the license
file is to offer the complete license text, not to prove that the license is
right. For public domain, a complete license text is not needed.
It is useful to have a comment of where you found this license, but if it is
just in the file itself, that's not really relevant.
> +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
> +ifeq ($(BR2_PACKAGE_QT_FONT_HELVETICA),y)
> +QT_LICENSE += MIT-like (Helvetica fonts)
Hm, doesn't look that MIT-like to me. I'd call it 'Adobe Helvetica license'.
> +QT_LICENSE_FILES += src/3rdparty/fonts/COPYING.Helvetica
> +endif
> +ifeq ($(BR2_PACKAGE_QT_FONT_JAPANESE),y)
> +# according to https://launchpad.net/ubuntu/vivid/+source/qtbase-opensource-src/+copyright
> +# Public Domain
I think that's a mistake... I submitted bug #815616 to debian.
> +QT_LICENSE += Public Domain (Japanese font)
> +# according to lib/fonts/README same as Helvetica fonts?
Yep, it's from the same source.
It's a bit annoying that you'd have
Adobe Helvetica license (Helvetica fonts) Adobe Helvetica license (Japanese font)
i.e. the same thing twice. So I'd make it:
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
> +#QT_LICENSE += MIT-like (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
>
> ifeq ($(BR2_PACKAGE_QT_QTFREETYPE)$(BR2_PACKAGE_QT_SYSTEMFREETYPE),y)
> @@ -663,6 +689,10 @@ 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 += MIT-like (DejaVu/Vera TrueType fonts)
Also not that MIT-like, so make it 'Bitstream license'.
The problem with the way that it's done now is that the different licenses will
not be comma-separated while they usually are. But adding commas is difficult in
this case, so OK.
Regards,
Arnout
> +QT_LICENSE_FILES += src/3rdparty/fonts/COPYRIGHT.DejaVu \
> + src/3rdparty/fonts/README.DejaVu \
> + src/3rdparty/fonts/COPYRIGHT.Vera
> endif
> endif # BR2_PACKAGE_QT_EMBEDDED
>
>
--
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v1] qt: add font license information
2016-02-22 23:09 ` Arnout Vandecappelle
@ 2016-02-23 12:51 ` Thomas Petazzoni
2016-02-23 19:39 ` Arnout Vandecappelle
2016-02-23 20:19 ` Peter Seiderer
2016-02-23 20:02 ` Peter Seiderer
1 sibling, 2 replies; 6+ messages in thread
From: Thomas Petazzoni @ 2016-02-23 12:51 UTC (permalink / raw)
To: buildroot
Arnout,
Thanks a lot for having reviewed this patch, which is not very simple
in terms of licensing details.
On Tue, 23 Feb 2016 00:09:24 +0100, Arnout Vandecappelle wrote:
> The problem with the way that it's done now is that the different licenses will
> not be comma-separated while they usually are. But adding commas is difficult in
> this case, so OK.
I thought we could do something such as:
QT_LICENSE += Foo (bar),
QT_LICENSE += Bar (foo),
QT_LICENSE += Baz (boz),
QT_LICENSE := $(patsubst %$(comma),%,$(QT_LICENSE))
But it unfortunately seems to remove all commas. We could then do:
QT_FINAL_LICENSE = $(shell echo "$(QT_LICENSE)" | sed 's:\(.*\),:\1:')
Not super pretty, but it's just one line of crap.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v1] qt: add font license information
2016-02-23 12:51 ` Thomas Petazzoni
@ 2016-02-23 19:39 ` Arnout Vandecappelle
2016-02-23 20:19 ` Peter Seiderer
1 sibling, 0 replies; 6+ messages in thread
From: Arnout Vandecappelle @ 2016-02-23 19:39 UTC (permalink / raw)
To: buildroot
On 02/23/16 13:51, Thomas Petazzoni wrote:
> Arnout,
>
> Thanks a lot for having reviewed this patch, which is not very simple
> in terms of licensing details.
>
> On Tue, 23 Feb 2016 00:09:24 +0100, Arnout Vandecappelle wrote:
>
>> The problem with the way that it's done now is that the different licenses will
>> not be comma-separated while they usually are. But adding commas is difficult in
>> this case, so OK.
>
> I thought we could do something such as:
>
> QT_LICENSE += Foo (bar),
> QT_LICENSE += Bar (foo),
> QT_LICENSE += Baz (boz),
> QT_LICENSE := $(patsubst %$(comma),%,$(QT_LICENSE))
>
> But it unfortunately seems to remove all commas. We could then do:
>
> QT_FINAL_LICENSE = $(shell echo "$(QT_LICENSE)" | sed 's:\(.*\),:\1:')
>
> Not super pretty, but it's just one line of crap.
Actually, that last bit could be moved to the legal-info infra itself, e.g. in
the legal-manifest function.
But that's for another patch :-)
Regards,
Arnout
>
> Thomas
>
--
Arnout Vandecappelle arnout dot vandecappelle at essensium dot com
Senior Embedded Software Architect . . . . . . +32-478-010353 (mobile)
Essensium, Mind division . . . . . . . . . . . . . . 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
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v1] qt: add font license information
2016-02-22 23:09 ` Arnout Vandecappelle
2016-02-23 12:51 ` Thomas Petazzoni
@ 2016-02-23 20:02 ` Peter Seiderer
1 sibling, 0 replies; 6+ messages in thread
From: Peter Seiderer @ 2016-02-23 20:02 UTC (permalink / raw)
To: buildroot
Hello Arnout,
On Tue, 23 Feb 2016 00:09:24 +0100, Arnout Vandecappelle <arnout@mind.be> wrote:
> On 06/12/15 22:22, Peter Seiderer wrote:
> > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> > ---
> > Not sure about the japanese font and about MIT or MIT-like
> > licenses (no exact match of the wording to other MIT licenses).
> > ---
> > package/qt/qt.mk | 30 ++++++++++++++++++++++++++++++
> > 1 file changed, 30 insertions(+)
> >
> > diff --git a/package/qt/qt.mk b/package/qt/qt.mk
> > index a7bbe5f..3fbb3e1 100644
> > --- a/package/qt/qt.mk
> > +++ b/package/qt/qt.mk
> > @@ -656,6 +656,32 @@ 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)
> > +QT_LICENSE += Public Domain (Micro font)
> > +# disabled because this file (containing the Public Domain notice) is
> > +# the font source file too...
> > +#QT_LICENSE_FILES += src/3rdparty/fonts/micro.bdf
>
> IMHO LICENSE_FILES are not needed for Public Domain. The point of the license
> file is to offer the complete license text, not to prove that the license is
> right. For public domain, a complete license text is not needed.
>
O.k, if no proof for Public Domain via a license file is necessary...
> It is useful to have a comment of where you found this license, but if it is
> just in the file itself, that's not really relevant.
>
Will just add a little comment...
> > +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
> > +ifeq ($(BR2_PACKAGE_QT_FONT_HELVETICA),y)
> > +QT_LICENSE += MIT-like (Helvetica fonts)
>
> Hm, doesn't look that MIT-like to me. I'd call it 'Adobe Helvetica license'.
>
O.k.
> > +QT_LICENSE_FILES += src/3rdparty/fonts/COPYING.Helvetica
> > +endif
> > +ifeq ($(BR2_PACKAGE_QT_FONT_JAPANESE),y)
> > +# according to https://launchpad.net/ubuntu/vivid/+source/qtbase-opensource-src/+copyright
> > +# Public Domain
>
> I think that's a mistake... I submitted bug #815616 to debian.
>
> > +QT_LICENSE += Public Domain (Japanese font)
> > +# according to lib/fonts/README same as Helvetica fonts?
>
> Yep, it's from the same source.
>
Thanks for investigation...
> It's a bit annoying that you'd have
>
> Adobe Helvetica license (Helvetica fonts) Adobe Helvetica license (Japanese font)
>
> i.e. the same thing twice. So I'd make it:
>
> 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
O.k.
>
> > +#QT_LICENSE += MIT-like (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
> >
> > ifeq ($(BR2_PACKAGE_QT_QTFREETYPE)$(BR2_PACKAGE_QT_SYSTEMFREETYPE),y)
> > @@ -663,6 +689,10 @@ 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 += MIT-like (DejaVu/Vera TrueType fonts)
>
> Also not that MIT-like, so make it 'Bitstream license'.
>
O.k.
>
> The problem with the way that it's done now is that the different licenses will
> not be comma-separated while they usually are. But adding commas is difficult in
> this case, so OK.
>
Will try Thomas suggestion for adding commas....
Wanted to do a similar patch for qt5base, but did not find the license files,
see QTBUG-46655 ([1]), after some discussion on the mailing list ([2]) the
decision was to drop the bundled fonts (abandoned: [3], [4], merged: [5], [6])...
Regards,
Peter
[1] https://bugreports.qt.io/browse/QTBUG-46655
[2] http://lists.qt-project.org/pipermail/development/2015-June/021924.html
[3] https://codereview.qt-project.org/#/c/114503/
[4] https://codereview.qt-project.org/#/c/114512/
[5] https://codereview.qt-project.org/#/c/123568/
[6] https://codereview.qt-project.org/#/c/123567/
>
> Regards,
> Arnout
>
>
> > +QT_LICENSE_FILES += src/3rdparty/fonts/COPYRIGHT.DejaVu \
> > + src/3rdparty/fonts/README.DejaVu \
> > + src/3rdparty/fonts/COPYRIGHT.Vera
> > endif
> > endif # BR2_PACKAGE_QT_EMBEDDED
> >
> >
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v1] qt: add font license information
2016-02-23 12:51 ` Thomas Petazzoni
2016-02-23 19:39 ` Arnout Vandecappelle
@ 2016-02-23 20:19 ` Peter Seiderer
1 sibling, 0 replies; 6+ messages in thread
From: Peter Seiderer @ 2016-02-23 20:19 UTC (permalink / raw)
To: buildroot
Hello Thomas,
On Tue, 23 Feb 2016 13:51:10 +0100, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:
> Arnout,
>
> Thanks a lot for having reviewed this patch, which is not very simple
> in terms of licensing details.
>
> On Tue, 23 Feb 2016 00:09:24 +0100, Arnout Vandecappelle wrote:
>
> > The problem with the way that it's done now is that the different licenses will
> > not be comma-separated while they usually are. But adding commas is difficult in
> > this case, so OK.
>
> I thought we could do something such as:
>
> QT_LICENSE += Foo (bar),
> QT_LICENSE += Bar (foo),
> QT_LICENSE += Baz (boz),
> QT_LICENSE := $(patsubst %$(comma),%,$(QT_LICENSE))
>
> But it unfortunately seems to remove all commas. We could then do:
>
> QT_FINAL_LICENSE = $(shell echo "$(QT_LICENSE)" | sed 's:\(.*\),:\1:')
>
> Not super pretty, but it's just one line of crap.
O.k. will take it for the next patch version...
Regards,
Peter
>
> Thomas
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-02-23 20:19 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-12 20:22 [Buildroot] [PATCH v1] qt: add font license information Peter Seiderer
2016-02-22 23:09 ` Arnout Vandecappelle
2016-02-23 12:51 ` Thomas Petazzoni
2016-02-23 19:39 ` Arnout Vandecappelle
2016-02-23 20:19 ` Peter Seiderer
2016-02-23 20:02 ` Peter Seiderer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox