Message ID | 20250923005234.2952070-1-Randy.MacLeod@windriver.com |
---|---|
State | New |
Headers | show |
Series | [v3] gawk: disable persistent memory allocator due to licensing | expand |
On Tue, 23 Sept 2025 at 02:52, Randy MacLeod via lists.openembedded.org <randy.macleod=windriver.com@lists.openembedded.org> wrote: > +# pma adds AGPL license > +LICENSE:append = " ${@bb.utils.contains('PACKAGECONFIG', 'pma', ' & AGPL-3.0-or-later', '', d)}" > +LIC_FILES_CHKSUM:append = " ${@bb.utils.contains('PACKAGECONFIG', 'pma', \ > + 'file://support/pma.c;md5=bb0026ee5e8b950e67d670dd2d60cc93;beginline=8;endline=19 \ > + file://support/pma.h;md5=bb0026ee5e8b950e67d670dd2d60cc93;beginline=8;endline=19', '', d) \ > +}" This is incorrect, and has already been discussed in a similar patch. LICENSE is for the source code license, and should be set unconditionally regardless of whether pma is enabled. Then you can set licenses for particular binary packages subject to whether pma is enabled or not. Alex
On 2025-09-23 4:14 a.m., Alexander Kanavin wrote: > On Tue, 23 Sept 2025 at 02:52, Randy MacLeod via > lists.openembedded.org > <randy.macleod=windriver.com@lists.openembedded.org> wrote: >> +# pma adds AGPL license >> +LICENSE:append = " ${@bb.utils.contains('PACKAGECONFIG', 'pma', ' & AGPL-3.0-or-later', '', d)}" >> +LIC_FILES_CHKSUM:append = " ${@bb.utils.contains('PACKAGECONFIG', 'pma', \ >> + 'file://support/pma.c;md5=bb0026ee5e8b950e67d670dd2d60cc93;beginline=8;endline=19 \ >> +file://support/pma.h;md5=bb0026ee5e8b950e67d670dd2d60cc93;beginline=8;endline=19', '', d) \ >> +}" > This is incorrect, and has already been discussed in a similar patch. Ah, I had missed Robert's email since I wasn't CCed: https://lore.kernel.org/openembedded-core/cover.1758264088.git.liezhi.yang@windriver.com/ > LICENSE is for the source code license, and should be set > unconditionally regardless of whether pma is enabled. Then you can set > licenses for particular binary packages subject to whether pma is > enabled or not. I don't think that rule/guideline applies here since pma.o is added or excluded from being linked with the main gawk binary based on the outcome of configure. That's now determined by what PACKAGECONFIG is set, so one or both licenses applies to the gawk binary. There is no separate binary produced using uses pma.o unlike the cairo case that Do we really want: LICENSE = "GPL-3.0-or-later & AGPL-3.0-or-later" LICENSE:${PN} = "GPL-3.0-or-later" LICENSE${PN}:append = " ${@bb.utils.contains('PACKAGECONFIG', 'pma', ' & AGPL-3.0-or-later', '', d)}" ? ../Randy > > Alex
On Tue, 23 Sept 2025 at 15:08, Randy MacLeod via lists.openembedded.org <randy.macleod=windriver.com@lists.openembedded.org> wrote: > I don't think that rule/guideline applies here since pma.o is added or excluded from > being linked with the main gawk binary based on the outcome of configure. > That's now determined by what PACKAGECONFIG is set, so one or both licenses applies to the gawk binary. > There is no separate binary produced using uses pma.o unlike the cairo case that Neverthless, LICENSE is supposed to cover everything in the source tree, regardless of build time choices. There are a few recipes that violate this, they should be fixed as well. > Do we really want: > LICENSE = "GPL-3.0-or-later & AGPL-3.0-or-later" > LICENSE:${PN} = "GPL-3.0-or-later" > LICENSE${PN}:append = " ${@bb.utils.contains('PACKAGECONFIG', 'pma', ' & AGPL-3.0-or-later', '', d)}" > > ? Yes please. Alex
diff --git a/meta/recipes-extended/gawk/gawk_5.3.2.bb b/meta/recipes-extended/gawk/gawk_5.3.2.bb index 833fe78bf9..b4b7419c34 100644 --- a/meta/recipes-extended/gawk/gawk_5.3.2.bb +++ b/meta/recipes-extended/gawk/gawk_5.3.2.bb @@ -6,14 +6,21 @@ HOMEPAGE = "https://www.gnu.org/software/gawk/" BUGTRACKER = "bug-gawk@gnu.org" SECTION = "console/utils" -# gawk <= 3.1.5: GPL-2.0-only -# gawk >= 3.1.6: GPL-3.0-only LICENSE = "GPL-3.0-only" LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504" +# pma adds AGPL license +LICENSE:append = " ${@bb.utils.contains('PACKAGECONFIG', 'pma', ' & AGPL-3.0-or-later', '', d)}" +LIC_FILES_CHKSUM:append = " ${@bb.utils.contains('PACKAGECONFIG', 'pma', \ + 'file://support/pma.c;md5=bb0026ee5e8b950e67d670dd2d60cc93;beginline=8;endline=19 \ + file://support/pma.h;md5=bb0026ee5e8b950e67d670dd2d60cc93;beginline=8;endline=19', '', d) \ +}" + PACKAGECONFIG ??= "readline mpfr" PACKAGECONFIG[readline] = "--with-readline,--without-readline,readline" PACKAGECONFIG[mpfr] = "--with-mpfr,--without-mpfr, mpfr" +# pma: persistent memory allocator: off by default due to licensing. +PACKAGECONFIG[pma] = "--enable-pma,--disable-pma, " SRC_URI = "${GNU_MIRROR}/gawk/gawk-${PV}.tar.gz \ file://run-ptest \