From: Alexander Dahl <ada@thorsis.com>
To: Roland Hieber <rhi@pengutronix.de>, ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH 3/6] libqmi: Add license file hashes
Date: Thu, 25 Mar 2021 07:31:44 +0100 (CET) [thread overview]
Message-ID: <1057802717.12289.1616653905020@seven.thorsis.com> (raw)
In-Reply-To: <20210324204522.fhscezwk5iywmvhu@pengutronix.de>
Hello Roland,
> Roland Hieber <rhi@pengutronix.de> hat am 24.03.2021 21:45 geschrieben:
>
>
> On Wed, Mar 24, 2021 at 07:35:37AM +0100, Alexander Dahl wrote:
> > From: Alexander Dahl <ada@thorsis.com>
> >
> > Signed-off-by: Alexander Dahl <ada@thorsis.com>
> > ---
> > rules/libqmi.make | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/rules/libqmi.make b/rules/libqmi.make
> > index 2b3e40140..3342eed20 100644
> > --- a/rules/libqmi.make
> > +++ b/rules/libqmi.make
> > @@ -22,6 +22,9 @@ LIBQMI_URL := http://www.freedesktop.org/software/libqmi/$(LIBQMI).$(LIBQMI_SUFF
> > LIBQMI_SOURCE := $(SRCDIR)/$(LIBQMI).$(LIBQMI_SUFFIX)
> > LIBQMI_DIR := $(BUILDDIR)/$(LIBQMI)
> > LIBQMI_LICENSE := GPL-2.0-or-later AND LGPL-2.1-or-later
> > +LIBQMI_LICENSE_FILES := \
> > + file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
> > + file://COPYING.LIB;md5=4fbd65380cdd255951079008b364516c
>
> Same here, someone seems to have copied the wrong license. Files in
> src/libqmi-glib/ and src/libqrtr-glib/ carry LGPL-2.0-or-later headers,
> and the README says that too. I didn't mention this in my previous mail,
> but I would rather not include COPYING.LIB then, and add a short comment
> why it isn't in there.
I'm not sure about this one. File headers say "Lesser GPL". The original LGPL 2.0 said "Library GPL". Looks to me like someone wanted 2.1 already, also in file headers.
Should we also open a ticket at upstream to request clarification? That would be here in this case:
https://gitlab.freedesktop.org/mobile-broadband/libqmi
(It's a different maintainer than for libgudev btw.)
Greets
Alex
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
next prev parent reply other threads:[~2021-03-25 6:31 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-24 6:35 [ptxdist] [PATCH 0/6] ModemManager: version bump Alexander Dahl
2021-03-24 6:35 ` [ptxdist] [PATCH 1/6] libgudev: Add license file hash Alexander Dahl
2021-03-24 20:36 ` Roland Hieber
2021-03-25 6:19 ` Alexander Dahl
2021-03-24 6:35 ` [ptxdist] [PATCH 2/6] libgudev: version bump 230 -> 234 Alexander Dahl
2021-03-24 6:35 ` [ptxdist] [PATCH 3/6] libqmi: Add license file hashes Alexander Dahl
2021-03-24 20:45 ` Roland Hieber
2021-03-25 6:31 ` Alexander Dahl [this message]
2021-03-24 6:35 ` [ptxdist] [PATCH 4/6] libqmi: version bump 1.26.2 -> 1.28.2 Alexander Dahl
2021-03-24 6:35 ` [ptxdist] [PATCH 5/6] modemmanager: Add license file hashes Alexander Dahl
2021-03-24 20:31 ` Roland Hieber
2021-03-25 6:07 ` Alexander Dahl
2021-03-24 6:35 ` [ptxdist] [PATCH 6/6] modemmanager: version bump 1.14.8 -> 1.16.2 Alexander Dahl
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1057802717.12289.1616653905020@seven.thorsis.com \
--to=ada@thorsis.com \
--cc=ptxdist@pengutronix.de \
--cc=rhi@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox