GNU bug report logs - #74344
[PATCH 0/7] gnu: Add jamesdsp.

Previous Next

Full log

View this message in rfc822 format

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Leo Famulari <leo <at> famulari.name>
Cc: gabriel <at> erlikon.ch, sughosha <at> disroot.org, 74344 <at> debbugs.gnu.org
Subject: [bug#74344] [PATCH v3 7/7] gnu: Add jamesdsp.
Date: Sun, 16 Feb 2025 15:46:27 +0900

Hi,

Leo Famulari <leo <at> famulari.name> writes:

> On Tue, Feb 11, 2025 at 07:50:51PM +0530, Sughosha via Guix-patches via wrote:
>> * gnu/packages/audio.scm (jamesdsp): New variable.
>
>> +       (modules '((guix build utils)))
>> +       (snippet
>> +        ;; Unbundle 3rd party libraries.
>
> Okay
>
>> +           #~(modify-phases %standard-phases
>> +               (replace 'configure
>
> Add a code comment explaining why this phase is replaced.
>
>> +    (license (list license:gpl3+
>> +                   license:gpl2      ;LiquidEqualizerWidget's 3rd party library
>> +                   license:gpl2+     ;GraphicEQWidget
>> +                   license:expat)))) ;QAnimatedSlider and QCodeEditor
>
> The overall package is distributed by us as GPL3+, despite some
> subcomponents bearing different licenses, so this field should only
> include gpl3+. It's okay to keep the info about the other licenses in a
> code comment.

Perhaps that's an error on Sugosha's part, but if this project is truly
combining gpl2 *only* with gpl3+, that's problematic as these are not
compatible licenses.

-- 
Thanks,
Maxim

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.