mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
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

  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