file /etc/ethertypes conflicts between netbase-1:6.2-r0.corei7_64 and ebtables-2.0.10+4-r4.corei7_64

Submitted by Outback Dingo on Dec. 14, 2020, 10:32 a.m. | Patch ID: 178950

Details

Message ID 20201214103248.2227-1-outbackdingo@gmail.com
State New
Headers show

Commit Message

Outback Dingo Dec. 14, 2020, 10:32 a.m.
---
 meta/recipes-core/netbase/netbase_6.2.bb | 1 -
 1 file changed, 1 deletion(-)

-- 
2.20.1
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145539): https://lists.openembedded.org/g/openembedded-core/message/145539
Mute This Topic: https://lists.openembedded.org/mt/78947734/1003190
Group Owner: openembedded-core+owner@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [mhalstead@linuxfoundation.org]
-=-=-=-=-=-=-=-=-=-=-=-

Patch hide | download patch | download mbox

diff --git a/meta/recipes-core/netbase/netbase_6.2.bb b/meta/recipes-core/netbase/netbase_6.2.bb
index a54d2e7764..262b2cf1bc 100644
--- a/meta/recipes-core/netbase/netbase_6.2.bb
+++ b/meta/recipes-core/netbase/netbase_6.2.bb
@@ -19,5 +19,4 @@  do_install () {
 	install -m 0644 ${S}/etc/rpc ${D}${sysconfdir}/rpc
 	install -m 0644 ${S}/etc/protocols ${D}${sysconfdir}/protocols
 	install -m 0644 ${S}/etc/services ${D}${sysconfdir}/services
-	install -m 0644 ${S}/etc/ethertypes ${D}${sysconfdir}/ethertypes
 }

Comments

Alexander Kanavin Dec. 14, 2020, 10:39 a.m.
Sorry, but you need to explain why the file from from ebtables (an optional
package from a 3rd party layer) takes precedence over file from netbase
(which is a core item).

Alex

On Mon, 14 Dec 2020 at 11:33, Outback Dingo <outbackdingo@gmail.com> wrote:

> ---

>  meta/recipes-core/netbase/netbase_6.2.bb | 1 -

>  1 file changed, 1 deletion(-)

>

> diff --git a/meta/recipes-core/netbase/netbase_6.2.bb

> b/meta/recipes-core/netbase/netbase_6.2.bb

> index a54d2e7764..262b2cf1bc 100644

> --- a/meta/recipes-core/netbase/netbase_6.2.bb

> +++ b/meta/recipes-core/netbase/netbase_6.2.bb

> @@ -19,5 +19,4 @@ do_install () {

>         install -m 0644 ${S}/etc/rpc ${D}${sysconfdir}/rpc

>         install -m 0644 ${S}/etc/protocols ${D}${sysconfdir}/protocols

>         install -m 0644 ${S}/etc/services ${D}${sysconfdir}/services

> -       install -m 0644 ${S}/etc/ethertypes ${D}${sysconfdir}/ethertypes

>  }

> --

> 2.20.1

>

>

> 

>

>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145541): https://lists.openembedded.org/g/openembedded-core/message/145541
Mute This Topic: https://lists.openembedded.org/mt/78947734/1003190
Group Owner: openembedded-core+owner@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [mhalstead@linuxfoundation.org]
-=-=-=-=-=-=-=-=-=-=-=-
Mark Hatle Dec. 14, 2020, 10:47 a.m.
I agree.  Netbase is the required for all installs.  ebtables is only used in
some installs.

So unless ebtables has a more up-to-date version of the file, it seems like the
bug is that ebtables needs to either remove the file or sync to the netbase version.

(You won't get a conflict if both files are identical, md5sum/shasum)

--Mark

On 12/14/20 4:39 AM, Alexander Kanavin wrote:
> Sorry, but you need to explain why the file from from ebtables (an optional

> package from a 3rd party layer) takes precedence over file from netbase (which

> is a core item).

> 

> Alex

> 

> On Mon, 14 Dec 2020 at 11:33, Outback Dingo <outbackdingo@gmail.com

> <mailto:outbackdingo@gmail.com>> wrote:

> 

>     ---

>      meta/recipes-core/netbase/netbase_6.2.bb <http://netbase_6.2.bb> | 1 -

>      1 file changed, 1 deletion(-)

> 

>     diff --git a/meta/recipes-core/netbase/netbase_6.2.bb

>     <http://netbase_6.2.bb> b/meta/recipes-core/netbase/netbase_6.2.bb

>     <http://netbase_6.2.bb>

>     index a54d2e7764..262b2cf1bc 100644

>     --- a/meta/recipes-core/netbase/netbase_6.2.bb <http://netbase_6.2.bb>

>     +++ b/meta/recipes-core/netbase/netbase_6.2.bb <http://netbase_6.2.bb>

>     @@ -19,5 +19,4 @@ do_install () {

>             install -m 0644 ${S}/etc/rpc ${D}${sysconfdir}/rpc

>             install -m 0644 ${S}/etc/protocols ${D}${sysconfdir}/protocols

>             install -m 0644 ${S}/etc/services ${D}${sysconfdir}/services

>     -       install -m 0644 ${S}/etc/ethertypes ${D}${sysconfdir}/ethertypes

>      }

>     -- 

>     2.20.1

> 

> 

> 

> 

> 

> 

> 

>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145542): https://lists.openembedded.org/g/openembedded-core/message/145542
Mute This Topic: https://lists.openembedded.org/mt/78947734/1003190
Group Owner: openembedded-core+owner@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [mhalstead@linuxfoundation.org]
-=-=-=-=-=-=-=-=-=-=-=-
Outback Dingo Dec. 14, 2020, 10:49 a.m.
On Mon, Dec 14, 2020 at 5:39 PM Alexander Kanavin
<alex.kanavin@gmail.com> wrote:
>

> Sorry, but you need to explain why the file from from ebtables (an optional package from a 3rd party layer) takes precedence over file from netbase (which is a core item).


I happened to run across it using poky and it was suggested i send a
patch, the fact it conflicts with an optional package, well i guess
Ill have to look at both, might just be that
it should come out of ebtables, either way i hit it due to the recent
change commit in netbase_6.2.bb telling me it was not previously there
until you actually added that line this month!

git show ba069a5a3dae374744f68a57ab9e7869167beb2b

-SRC_URI = "${DEBIAN_MIRROR}/main/n/${BPN}/${BPN}_${PV}~bpo10+1.tar.xz"
-S = "${WORKDIR}/${BPN}-${PV}~bpo10+1"
+SRC_URI = "${DEBIAN_MIRROR}/main/n/${BPN}/${BPN}_${PV}.tar.xz"

-SRC_URI[md5sum] = "4fa7517285b4045ac0dc8dbf6730dd7a"
-SRC_URI[sha256sum] =
"4e9c3082dff8896cb6b6bea9bb2200d82fb0d7c8d8c8fc9b18704fe553316237"
+inherit allarch
+
+SRC_URI[sha256sum] =
"309a24146a06347d654b261e9e07a82fab844b173674a42e223803dd8258541e"

 UPSTREAM_CHECK_URI = "${DEBIAN_MIRROR}/main/n/netbase/"
-do_install () {

-       install -d ${D}/${mandir}/man8 ${D}${sysconfdir}
+do_install () {
+       install -d ${D}${sysconfdir}
        install -m 0644 ${S}/etc/rpc ${D}${sysconfdir}/rpc
        install -m 0644 ${S}/etc/protocols ${D}${sysconfdir}/protocols
        install -m 0644 ${S}/etc/services ${D}${sysconfdir}/services
+       install -m 0644 ${S}/etc/ethertypes ${D}${sysconfdir}/ethertypes
 }



>

> Alex

>

> On Mon, 14 Dec 2020 at 11:33, Outback Dingo <outbackdingo@gmail.com> wrote:

>>

>> ---

>>  meta/recipes-core/netbase/netbase_6.2.bb | 1 -

>>  1 file changed, 1 deletion(-)

>>

>> diff --git a/meta/recipes-core/netbase/netbase_6.2.bb b/meta/recipes-core/netbase/netbase_6.2.bb

>> index a54d2e7764..262b2cf1bc 100644

>> --- a/meta/recipes-core/netbase/netbase_6.2.bb

>> +++ b/meta/recipes-core/netbase/netbase_6.2.bb

>> @@ -19,5 +19,4 @@ do_install () {

>>         install -m 0644 ${S}/etc/rpc ${D}${sysconfdir}/rpc

>>         install -m 0644 ${S}/etc/protocols ${D}${sysconfdir}/protocols

>>         install -m 0644 ${S}/etc/services ${D}${sysconfdir}/services

>> -       install -m 0644 ${S}/etc/ethertypes ${D}${sysconfdir}/ethertypes

>>  }

>> --

>> 2.20.1

>>

>>

>> 

>>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145543): https://lists.openembedded.org/g/openembedded-core/message/145543
Mute This Topic: https://lists.openembedded.org/mt/78947734/1003190
Group Owner: openembedded-core+owner@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [mhalstead@linuxfoundation.org]
-=-=-=-=-=-=-=-=-=-=-=-
Outback Dingo Dec. 14, 2020, 10:55 a.m.
On Mon, Dec 14, 2020 at 5:39 PM Alexander Kanavin
<alex.kanavin@gmail.com> wrote:
>

> Sorry, but you need to explain why the file from from ebtables (an optional package from a 3rd party layer) takes precedence over file from netbase (which is a core item).


further to the previous note, it conflicts with ebtables in meta-oe
from what i can tell

commit c71a08cea8477f69ee3bc511f1f14d99e09c0a49
Author: Paul Eggleton <paul.eggleton@linux.intel.com>
Date:   Mon Dec 3 15:30:40 2012 +0000

    ebtables: add from OE-Classic, update and tidy up

    * Update to 2.0.10-4
    * Handle hardcoded paths in initscript
    * Add LIC_FILES_CHKSUM
    * Set SUMMARY (which sets DESCRIPTION)
    * Drop PRIORITY
    * Minor formatting/ordering tweaks

    Based on a patch by Vladimir Redzhepoff <vladimir.redzhepoff@promwad.com>

    Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>



>

> Alex

>

> On Mon, 14 Dec 2020 at 11:33, Outback Dingo <outbackdingo@gmail.com> wrote:

>>

>> ---

>>  meta/recipes-core/netbase/netbase_6.2.bb | 1 -

>>  1 file changed, 1 deletion(-)

>>

>> diff --git a/meta/recipes-core/netbase/netbase_6.2.bb b/meta/recipes-core/netbase/netbase_6.2.bb

>> index a54d2e7764..262b2cf1bc 100644

>> --- a/meta/recipes-core/netbase/netbase_6.2.bb

>> +++ b/meta/recipes-core/netbase/netbase_6.2.bb

>> @@ -19,5 +19,4 @@ do_install () {

>>         install -m 0644 ${S}/etc/rpc ${D}${sysconfdir}/rpc

>>         install -m 0644 ${S}/etc/protocols ${D}${sysconfdir}/protocols

>>         install -m 0644 ${S}/etc/services ${D}${sysconfdir}/services

>> -       install -m 0644 ${S}/etc/ethertypes ${D}${sysconfdir}/ethertypes

>>  }

>> --

>> 2.20.1

>>

>>

>> 

>>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145547): https://lists.openembedded.org/g/openembedded-core/message/145547
Mute This Topic: https://lists.openembedded.org/mt/78947734/1003190
Group Owner: openembedded-core+owner@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [mhalstead@linuxfoundation.org]
-=-=-=-=-=-=-=-=-=-=-=-
Mark Hatle Dec. 14, 2020, 12:09 p.m.
The version in netbase is the correct one.

For a comparison, see the followng for the netbase version:

https://salsa.debian.org/md/netbase/-/blob/master/etc/ethertypes

and see the following for the ebtables version:

http://git.netfilter.org/ebtables/tree/ethertypes?h=ebtables-2.0.10-4

So the correct fix for this would be to remove ethertypes from the ebtables
recipe in meta-oe.

Additionally, it looks like there is a MUCH newer version of ebtables.  (2.0.10
was released 6 year ago, while 2.0.11 was released about 1 year ago.)

(Even 2.0.11 has an older version of the ethertypes file then what netbase has.)

--Mark

On 12/14/20 4:55 AM, Outback Dingo wrote:
> On Mon, Dec 14, 2020 at 5:39 PM Alexander Kanavin

> <alex.kanavin@gmail.com> wrote:

>>

>> Sorry, but you need to explain why the file from from ebtables (an optional package from a 3rd party layer) takes precedence over file from netbase (which is a core item).

> 

> further to the previous note, it conflicts with ebtables in meta-oe

> from what i can tell

> 

> commit c71a08cea8477f69ee3bc511f1f14d99e09c0a49

> Author: Paul Eggleton <paul.eggleton@linux.intel.com>

> Date:   Mon Dec 3 15:30:40 2012 +0000

> 

>     ebtables: add from OE-Classic, update and tidy up

> 

>     * Update to 2.0.10-4

>     * Handle hardcoded paths in initscript

>     * Add LIC_FILES_CHKSUM

>     * Set SUMMARY (which sets DESCRIPTION)

>     * Drop PRIORITY

>     * Minor formatting/ordering tweaks

> 

>     Based on a patch by Vladimir Redzhepoff <vladimir.redzhepoff@promwad.com>

> 

>     Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>

> 

> 

>>

>> Alex

>>

>> On Mon, 14 Dec 2020 at 11:33, Outback Dingo <outbackdingo@gmail.com> wrote:

>>>

>>> ---

>>>  meta/recipes-core/netbase/netbase_6.2.bb | 1 -

>>>  1 file changed, 1 deletion(-)

>>>

>>> diff --git a/meta/recipes-core/netbase/netbase_6.2.bb b/meta/recipes-core/netbase/netbase_6.2.bb

>>> index a54d2e7764..262b2cf1bc 100644

>>> --- a/meta/recipes-core/netbase/netbase_6.2.bb

>>> +++ b/meta/recipes-core/netbase/netbase_6.2.bb

>>> @@ -19,5 +19,4 @@ do_install () {

>>>         install -m 0644 ${S}/etc/rpc ${D}${sysconfdir}/rpc

>>>         install -m 0644 ${S}/etc/protocols ${D}${sysconfdir}/protocols

>>>         install -m 0644 ${S}/etc/services ${D}${sysconfdir}/services

>>> -       install -m 0644 ${S}/etc/ethertypes ${D}${sysconfdir}/ethertypes

>>>  }

>>> --

>>> 2.20.1

>>>

>>>

>>>

>>>

>>>

>>>

>>> 

>>>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145551): https://lists.openembedded.org/g/openembedded-core/message/145551
Mute This Topic: https://lists.openembedded.org/mt/78947734/3616849
Group Owner: openembedded-core+owner@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [michael@yoctoproject.org]
-=-=-=-=-=-=-=-=-=-=-=-