This would still need to be tested with a specific image(s) from oe-core,
otherwise it is prone to bit rot. And it would not include tweaks from
meta-oe (e.g. lua) or anywhere else.

We already test no-gpl3-ness for core-image-minimal and
core-image-full-cmdline (to match the meta-gpl2 tests), and neither needs
PACKAGECONFIG tweaks:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/tree/meta/lib/oeqa/selftest/cases/incompatible_lic.py#n118

The same test could be extended for maybe core-image-sato and
core-image-weston. Whether those images will produce a nice list of tweaks,
or are good as they are, I can't tell. apr-util for example is not included
into any of them.

Alex

On Sat, 20 Jun 2020 at 16:03, Peter Kjellerstedt <
peter.kjellerst...@axis.com> wrote:

> Hmm, ok. How about a common without_gpl3.inc file? It could serve as a
> place to gather/document these PACKAGECONFIGs and other setting that need
> to be enabled to build without GPL-3.0. That way you are not forced in the
> same way as a distro feature would be, but at the same time it is easy for
> anyone who wants to disable GPL-3.0 for their build to find all the setting
> they need to tweak.
>
>
>
> //Peter
>
>
>
> *From:* Alexander Kanavin <alex.kana...@gmail.com>
> *Sent:* den 20 juni 2020 14:52
> *To:* Peter Kjellerstedt <peter.kjellerst...@axis.com>
> *Cc:* openembedded-core@lists.openembedded.org
> *Subject:* Re: [OE-core] [PATCH 1/2] apr-util: make gdbm optional
>
>
>
> In our environment gpl3 is not banned across the distro but rather banned
> only in the product image, with a targeted set of recipe tweaks aimed to
> allow that image to build. Non-product images are still permitted to
> contain gpl3 items. Since everyone has a different set of what goes into
> their product image, and what doesn't, and there are inevitably various 3rd
> party layers in the mix, I do not think a distro setting would make much
> sense.
>
>
>
> Alex
>
>
>
> On Sat, 20 Jun 2020 at 13:30, Peter Kjellerstedt <
> peter.kjellerst...@axis.com> wrote:
>
> > -----Original Message-----
> > From: openembedded-core@lists.openembedded.org <openembedded-
> > c...@lists.openembedded.org> On Behalf Of Alexander Kanavin
> > Sent: den 17 juni 2020 12:16
> > To: openembedded-core@lists.openembedded.org
> > Cc: Alexander Kanavin <alex.kana...@gmail.com>
> > Subject: [OE-core] [PATCH 1/2] apr-util: make gdbm optional
> >
> > This helps with gpl3-free builds.
> >
> > Signed-off-by: Alexander Kanavin <alex.kana...@gmail.com>
> > ---
> >  meta/recipes-support/apr/apr-util_1.6.1.bb | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/meta/recipes-support/apr/apr-util_1.6.1.bb
> b/meta/recipes-support/apr/apr-util_1.6.1.bb
> > index 3896c924f0..0dd8f025e8 100644
> > --- a/meta/recipes-support/apr/apr-util_1.6.1.bb
> > +++ b/meta/recipes-support/apr/apr-util_1.6.1.bb
> > @@ -1,7 +1,7 @@
> >  SUMMARY = "Apache Portable Runtime (APR) companion library"
> >  HOMEPAGE = "http://apr.apache.org/";
> >  SECTION = "libs"
> > -DEPENDS = "apr expat gdbm"
> > +DEPENDS = "apr expat"
> >
> >  BBCLASSEXTEND = "native nativesdk"
> >
> > @@ -23,7 +23,6 @@ EXTRA_OECONF =
> "--with-apr=${STAGING_BINDIR_CROSS}/apr-1-config \
> >               --without-odbc \
> >               --without-pgsql \
> >               --with-dbm=gdbm \
> > -             --with-gdbm=${STAGING_DIR_HOST}${prefix} \
> >               --without-sqlite2 \
> >               --with-expat=${STAGING_DIR_HOST}${prefix}"
> >
> > @@ -66,10 +65,11 @@ do_install_append_class-target() {
> >              -e 's,APU_BUILD_DIR=.*,APR_BUILD_DIR=,g'
> ${D}${bindir}/apu-1-config
> >  }
> >
> > -PACKAGECONFIG ??= "crypto"
> > +PACKAGECONFIG ??= "crypto gdbm"
>
> Since I assume this is not the only place where it is needed to disable a
> feature to achieve a GPL-3.0 free build, maybe we should introduce a
> distro
> feature to control them? E.g.:
>
> PACKAGECONFIG ??= "${@bb.utils.contains('DISTRO_FEATURES', 'allow-gpl3',
> 'gdbm', '', d)} \
>                    crypto
> <$%7b@bb.utils.contains('DISTRO_FEATURES',%20'allow-gpl3',%20'gdbm',%20'',%20d)%7d%20%5C%0b%C2%A0%20%C2%A0%20%C2%A0%20%C2%A0%20%C2%A0%20%C2%A0%20%C2%A0%20%C2%A0%20%C2%A0%20%C2%A0crypto>
> "
>
> This would make it a lot easier for anyone who wants to disable the use of
> GPL-3.0 code as they would not have to go searching through the recipes
> for
> these kind of features to disable.
>
> >  PACKAGECONFIG[ldap] = "--with-ldap,--without-ldap,openldap"
> >  PACKAGECONFIG[crypto] = "--with-openssl=${STAGING_DIR_HOST}${prefix}
> --with-crypto,--without-crypto,openssl"
> >  PACKAGECONFIG[sqlite3] =
> "--with-sqlite3=${STAGING_DIR_HOST}${prefix},--without-sqlite3,sqlite3"
> > +PACKAGECONFIG[gdbm] =
> "--with-gdbm=${STAGING_DIR_HOST}${prefix},--without-gdbm,gdbm"
> >
> >  #files ${libdir}/apr-util-1/*.so are not symlinks but loadable modules
> thus they are packaged in ${PN}
> >  FILES_${PN}     += "${libdir}/apr-util-1/apr*${SOLIBS}
> ${libdir}/apr-util-1/apr*${SOLIBSDEV}"
> > --
> > 2.27.0
>
> //Peter
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#139718): 
https://lists.openembedded.org/g/openembedded-core/message/139718
Mute This Topic: https://lists.openembedded.org/mt/74934568/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to