[Buildroot] [PATCH v1] qt: add font license information
Peter Seiderer
ps.report at gmx.net
Tue Feb 23 20:19:46 UTC 2016
Hello Thomas,
On Tue, 23 Feb 2016 13:51:10 +0100, Thomas Petazzoni <thomas.petazzoni at 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
More information about the buildroot
mailing list