From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Bavier Subject: Re: [PATCH] gnu: Add cool-retro-term. Date: Tue, 2 May 2017 08:51:06 -0500 Message-ID: <20170502085106.434f29ff@centurylink.net> References: <20170423135526.5ed9d143@mykolab.ch> <87inlvhr0y.fsf@gmail.com> <20170423231207.4adb15e7@mykolab.ch> <87vapuwvj0.fsf@gmail.com> <20170424143026.53563696@mykolab.ch> <20170425223627.47d5a3fc@centurylink.net> <20170426200933.45340e03@mykolab.ch> <20170427204754.10212a01@centurylink.net> <20170428141037.6a4038f6@mykolab.ch> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; boundary="Sig_/Flzg/NVEu+Mx3mCv7Y6SRVD"; protocol="application/pgp-signature" Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:35523) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d5YD8-0003UV-0Y for guix-devel@gnu.org; Tue, 02 May 2017 09:51:27 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d5YD4-0006UK-1w for guix-devel@gnu.org; Tue, 02 May 2017 09:51:26 -0400 Received: from mail.centurylink.net ([205.219.233.9]:52234 helo=smtp.centurylink.net) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d5YD3-0006SN-Ol for guix-devel@gnu.org; Tue, 02 May 2017 09:51:21 -0400 In-Reply-To: <20170428141037.6a4038f6@mykolab.ch> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Petter Cc: guix-devel@gnu.org --Sig_/Flzg/NVEu+Mx3mCv7Y6SRVD Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Hi Petter, Thanks again for your patience with this review. On Fri, 28 Apr 2017 14:10:37 +0200 Petter wrote: > On Thu, 27 Apr 2017 20:47:54 -0500 > Eric Bavier wrote: >=20 > > > > > + (inputs > > > > > + `(("qt" ,qt))) =20 > > > >=20 > > > > Does this package need all of qt, or could we use the component > > > > packages instead? =20 > > >=20 > > > I don't know. I'll do some experiments and see. =20 > >=20 > > OK, great. =20 >=20 > Fixed! >=20 > Now it is: > (inputs > `(("qtbase" ,qtbase) > ("qtdeclarative" ,qtdeclarative) > ("qtgraphicaleffects" ,qtgraphicaleffects) > ("qtquickcontrols" ,qtquickcontrols))) Excellent. >=20 > > > > > + (description > > > > > + "Cool-retro-term (crt) is a terminal emulator which mimics = the > > > > > look and +feel of the old cathode ray tube (CRT) screens. It has= been > > > > > designed to be +eye-candy, customizable, and reasonably lightweig= ht.") > > > > > + (home-page "https://github.com/Swordfish90/cool-retro-term") > > > > > + (license (list license:gpl2 license:gpl3))))) =20 > > > >=20 > > > > The license seems unclear: 1) there is both a gpl-2.txt and gpl-3.t= xt > > > > in $topsrcdir, but no mention of which actually applies in any of t= he > > > > source files, 2) $topsrcdir/packaging/debian/copyright claims GPL-3, > > > > while 4) $topsrcdir/packaging/rpm/cool-retro-term.spec and > > > > $topsrcdir/packaging/appdata/cool-retro-term.appdata.xml both claim > > > > GPL-3.0+, and 5) the qmltermwidget component appears to be under > > > > gpl2+. IANAL but this suggests license:gpl2+ to me. Does that seem > > > > right? =20 > > >=20 > > > I can't answer this, hopefully someone else will chime in. =20 > >=20 > > I did a little more looking: The sources in app/qml seem to have > > license headers marked as GPLv3+, so we can mark it as such. =20 >=20 > I changed the GPL3 license GPL3+. >=20 > The owner has replied in an issue about this. >=20 > ---[Swordfish90 commented on Dec 1, 2015]--- > Hi @nadams810 and sorry for the late response. Since the project is split > into two different components, the idea behind the licensing was the > following: >=20 > 1. All the files belonging to QMLTermWidget which is a separate libra= ry > forked by qtermwidget are licensed under GPL2 (as the original library). > 2. All the files belonging to CRT are licensed under GPL3. > =20 > To be fair I'm not really an expert when it comes to licensing and if the= re > is something wrong with the following approach just let me know. > ---end-quote--- >=20 > https://github.com/Swordfish90/cool-retro-term/issues/269 "gpl2" in the license field should be "gpl2+" according to qmltermwidget license headers. Could you also include a comment noting the component split? Nothing fancy, just "for qmltermwidget" or so next to gpl2+ and "for cool-retro-term" next to gpl3+. >=20 > > > =20 > > > > It would not hurt to also list the SIL license, for > > > > $topsrcdir/app/qml/fonts/modern-hermit, and the X11 license for > > > > fonts/modern-pro-font-win-tweaked and fonts/modern-proggy-tiny. =20 >=20 > Added. >=20 > > > > The fonts/modern-envy-code-r font probably needs to be stripped; it= 's > > > > license is non-free; from 'Read Me.txt': > > > >=20 > > > > "Free to use but redistribution prohibited." > > > > =20 >=20 > This font was removed Mar 13, 2016. > https://github.com/Swordfish90/cool-retro-term/commit/c505dfaa11166054686= df7390cf04dba324ad869 I see. I must have been browsing while pinned at an older commit. No rationale given. Hmm. >=20 > > > > and the fonts/1977-apple2, fonts/1977-commodore-pet, probably also = needs > > > > to be stripped as non-free; from 'FreeLicense.txt': > > > >=20 > > > > "1. The User may not sell copies of the Software for a fee." > > > >=20 > > > > The fonts/1979-atari-400-800 font seems to be non-free license, whi= ch is > > > > not mentioned in the 'ReadMe.rtf' but only on the WayBack Machine at > > > > https://web-beta.archive.org/web/20080907111008/http://www2.bitstre= am.net/~marksim/atarimac/fonts.html : > > > >=20 > > > > "These fonts are freeware and may not be offered for sale seperat= ely > > > > or as part of a collection." > > > >=20 > > > > The fonts/1982-commodore64 font is licensed under non-free terms; s= ee > > > > fonts/1982-commodore64/license.txt > > > >=20 > > > > The only mention of a license for fonts/1985-ibm-pc-vga is at > > > > http://www.dafont.com/font-comment.php?file=3Dperfect_dos_vga_437 w= here > > > > the author says "The license is this: this font is free to use in > > > > whatever you want.", but that probably "doesn't count". > > > >=20 > > > > I could not find license info for a few of the other fonts. > > > >=20 > > > > OTOH I recall a discussion on IRC recently about fonts embedded in > > > > packages being treated as non-functional data, and thus OK from a F= SDG > > > > perspective. But I would want verification on that. =20 > > >=20 > > > Let's see if someone can shine some light on this. > > > =20 > >=20 > > Leo pointed me at https://bugs.gnu.org/26588 which references > > https://www.gnu.org/distros/free-system-distribution-guidelines.html#no= n-functional-data . > > I think many of these fonts would need to be stripped out or > > substituted because their license prohibits commercial/non-commercial > > redistribution. =20 >=20 > Recipe is now removing the following from disk and GUI: > '("1977-apple2" > "1977-commodore-pet" > "1979-atari-400-800" > "1982-commodore64" > "1985-ibm-pc-vga") The code you've added to delete these fonts and adjust the source accordingly would be better in an origin 'snippet' so that `guix build -S cool-retro-term` does not include the fonts. >=20 > Should more be removed? modern-monaco. It's a font from Apple which is specifically *not* freely distributable, but the ttf file is still floating around the internet, giving many projects the impression that it is. >=20 > What do we do about the fonts without licensing info? I agree with Chris that we need to remove them. >=20 > Also, I've added 3 patches from lxde/qtermwidget. The 2 with possible sec= urity > implications, and 1 memory leak (of those you listed). With the remaining= two > you linked, memory leak and perfomance fix, git was unable to apply to the > current qmltermwidget codebase. Excellent. Thanks, `~Eric --Sig_/Flzg/NVEu+Mx3mCv7Y6SRVD Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQEzBAEBCgAdFiEEP6ffQNOS1+XjkicBHrvSBHgfliwFAlkIjsoACgkQHrvSBHgf lixj8Af/cm/I9LtifHXUgRKuqkLDaH5JfymKFOuygduvmXfFD9xmOFwb74SThDSO PmVK9Dyik/4qX0GgD3gh2EFTNqqGgabktparLvakEi2zt5e7zTYvqsnXCIC3tSVc 4luIjkotJtky3OutZSQPV+ZGzQJ9ceAz5Sj8ncYXm72TZEpHsfbIxkXhGRYlbaiK KaBWwXwLLGb9+g5M4g3fzMY7ps3kHhavVBuJnt9OzCv/yI6/866lZf/T9n//HUxt R0wvNGcFTEAyBy7DOZejS/kQoTHYjVvdq0mxL7pN4ckSr+npC+/PvACa36Ok9Dp8 q6BeFTzHmWZHJTap7SaAyUdxrRxaRQ== =QXde -----END PGP SIGNATURE----- --Sig_/Flzg/NVEu+Mx3mCv7Y6SRVD--