On 2020-12-18 2:47 a.m., Alexander Kanavin wrote:
I’m not sure I like this approach. It doesn’t look like the right place to decide whether to include or not some packages based on specific licensing requirements, licenses for these and other items change over time, and there may be more undesirable licenses than just this one.

Why not simply create a new package group?

Alex

Makes sense to me, Mingli?

I'm replying months later because I see that we fixed this locally
without getting agreement with Alex and other oe-core devs. Not a
big deal but we want to minimize our collection of WR specific patches.

../Randy

On Fri 18. Dec 2020 at 6.37, Yu, Mingli <[email protected] <mailto:[email protected]>> wrote:

    From: Mingli Yu <[email protected]
    <mailto:[email protected]>>

    packagegroup-core-base-utils repends on some packages which are
    gplv3 compatilbe.

    Signed-off-by: Mingli Yu <[email protected]
    <mailto:[email protected]>>
    ---
      .../packagegroups/packagegroup-core-base-utils.bb
    <http://packagegroup-core-base-utils.bb>  | 14 +++++++-------
      1 file changed, 7 insertions(+), 7 deletions(-)

    diff --git
    a/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb 
<http://packagegroup-core-base-utils.bb>
    b/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb 
<http://packagegroup-core-base-utils.bb>
    index 1e63da7f16..0a3cb63bc2 100644
    ---
    a/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb 
<http://packagegroup-core-base-utils.bb>
    +++
    b/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb 
<http://packagegroup-core-base-utils.bb>
    @@ -33,18 +33,18 @@ RDEPENDS_${PN} = "\
          grep \
          gzip \
          ${@bb.utils.contains("DISTRO_FEATURES", "systemd", "",
    "ifupdown", d)} \
    -    inetutils \
    -    inetutils-ping \
    -    inetutils-telnet \
    -    inetutils-tftp \
    -    inetutils-traceroute \
    +    ${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
    'inetutils', d)} \
    +    ${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
    'inetutils-ping', d)} \
    +    ${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
    'inetutils-telnet', d)} \
    +    ${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
    'inetutils-tftp', d)} \
    +    ${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
    'inetutils-traceroute', d)} \
          iproute2 \
          ${@bb.utils.contains("MACHINE_FEATURES", "keyboard", "kbd",
    "", d)} \
          kmod \
          less \
          ncurses-tools \
          net-tools \
    -    parted \
    +    ${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
    'parted', d)} \
          patch \
          procps \
          psmisc \
    @@ -55,7 +55,7 @@ RDEPENDS_${PN} = "\
          unzip \
          util-linux \
          ${VIRTUAL-RUNTIME_vim} \
    -    wget \
    +    ${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '',
    'wget', d)} \
          which \
          xz \
          "
-- 2.26.2










--
# Randy MacLeod
# Wind River Linux
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#150613): 
https://lists.openembedded.org/g/openembedded-core/message/150613
Mute This Topic: https://lists.openembedded.org/mt/79055799/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to