From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from dude02.hi.pengutronix.de ([2001:67c:670:100:1d::28] helo=dude02.lab.pengutronix.de) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1jhBaK-00038c-Kn for ptxdist@pengutronix.de; Fri, 05 Jun 2020 14:36:32 +0200 Received: from mol by dude02.lab.pengutronix.de with local (Exim 4.92) (envelope-from ) id 1jhBaK-0002x0-Cl for ptxdist@pengutronix.de; Fri, 05 Jun 2020 14:36:32 +0200 Date: Fri, 5 Jun 2020 14:36:32 +0200 From: Michael Olbrich Message-ID: <20200605123632.GB1716@pengutronix.de> References: <20200527112204.ll7ij44ahllid6ip@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200527112204.ll7ij44ahllid6ip@pengutronix.de> Subject: Re: [ptxdist] [PULL] rules: add license information List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ptxdist-bounces@pengutronix.de Sender: "ptxdist" To: PTXdist Mailing List Hi, On Wed, May 27, 2020 at 01:22:04PM +0200, Roland Hieber wrote: > Felicitas has spent a good amount of time adding missing _LICENSE > and _LICENSE_FILES variables to (mostly older) package rules. I've > looked over her series, and there were some remaining Makefile syntax > errors, which I've fixed, and I also did a review of those rules that > failed during the build. For the other rules, I'm trusting her legal > expertise :-) While at it, I also fixed some issues which I discovered > on the way. > > With this series, 93% of all target package rules now have license > information, which is an increase of 18%. > > Since this series consists of 169 patches, I'm rather submitting it as a > pull request than single patch mails. So, I've started reviewing this. For all patches: please replace 'unknown' with 'custom' as discussed elsewhere. I've reviewed for argtables2 to libmemcached so far. I've queued the patches that are ok. They should hit master soon, so you can rebase the rest. Unless I have an explicit comment or 'unknown' is used, the patch is ok. The following patches need improvements: - argtables2: the source files say LGPL-2.0-or-later - memstat: It just says 'GPL' without any license text. I'm not comfortable with GPL-1.0-or-later for this. Maybe put it in staging? It's really old and has not been touched in a long time. - ckermit: A copyright statement of one of the source files should be added - db: src/clib/qsort.c and others look like BSD-4-Clause to me - grep: the source files say GPL-3.0-or-later - gtk2: the source files say LGPL-2.0-or-later - jvisu: I think this is "GPL-2.0-only AND (GPL-2.0-only WITH eCos-exception-2.0)" - libcaca: again, -or-later for the GPL licenses, I think - libgsf: typo in the subject: s/libsgf/libgsf/ Michael -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de