* [PATCH] add missing normalize-resolvconf file since it has been moved to a separate file since version 1.90
@ 2022-06-20 8:08 Qorin
2022-06-20 9:20 ` [OE-core] " Luca Ceresoli
2022-06-21 8:11 ` Luca Ceresoli
0 siblings, 2 replies; 6+ messages in thread
From: Qorin @ 2022-06-20 8:08 UTC (permalink / raw)
To: openembedded-core; +Cc: Qorin
From: Qorin <qorin.qorinna@nl.abb.com>
---
meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb b/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb
index 94fd2c1a70..d2d9973852 100644
--- a/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb
+++ b/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb
@@ -9,7 +9,7 @@ LICENSE = "GPL-2.0-or-later"
LIC_FILES_CHKSUM = "file://COPYING;md5=c93c0550bd3173f4504b2cbd8991e50b"
AUTHOR = "Thomas Hood"
HOMEPAGE = "http://packages.debian.org/resolvconf"
-RDEPENDS:${PN} = "bash"
+RDEPENDS:${PN} = "bash sed"
SRC_URI = "git://salsa.debian.org/debian/resolvconf.git;protocol=https;branch=unstable \
file://99_resolvconf \
@@ -46,6 +46,7 @@ do_install () {
chown -R root:root ${D}${sysconfdir}/
install -m 0755 bin/resolvconf ${D}${base_sbindir}/
install -m 0755 bin/list-records ${D}${base_libdir}/${BPN}
+ install -m 0755 bin/normalize-resolvconf ${D}${base_libdir}/${BPN}
install -d ${D}/${sysconfdir}/network/if-up.d
install -m 0755 debian/resolvconf.000resolvconf.if-up ${D}/${sysconfdir}/network/if-up.d/000resolvconf
install -d ${D}/${sysconfdir}/network/if-down.d
--
2.30.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [OE-core] [PATCH] add missing normalize-resolvconf file since it has been moved to a separate file since version 1.90
2022-06-20 8:08 [PATCH] add missing normalize-resolvconf file since it has been moved to a separate file since version 1.90 Qorin
@ 2022-06-20 9:20 ` Luca Ceresoli
[not found] ` <AM0PR06MB4625172F0F099B35BB980A7BD9B09@AM0PR06MB4625.eurprd06.prod.outlook.com>
2022-06-21 8:11 ` Luca Ceresoli
1 sibling, 1 reply; 6+ messages in thread
From: Luca Ceresoli @ 2022-06-20 9:20 UTC (permalink / raw)
To: Qorin Wu; +Cc: openembedded-core, Qorin
Hi Qorin,
On Mon, 20 Jun 2022 10:08:20 +0200
"Qorin Wu" <qorin.wu@gmail.com> wrote:
> From: Qorin <qorin.qorinna@nl.abb.com>
>
> ---
> meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb b/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb
> index 94fd2c1a70..d2d9973852 100644
> --- a/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb
> +++ b/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb
> @@ -9,7 +9,7 @@ LICENSE = "GPL-2.0-or-later"
> LIC_FILES_CHKSUM = "file://COPYING;md5=c93c0550bd3173f4504b2cbd8991e50b"
> AUTHOR = "Thomas Hood"
> HOMEPAGE = "http://packages.debian.org/resolvconf"
> -RDEPENDS:${PN} = "bash"
> +RDEPENDS:${PN} = "bash sed"
Can you pelase explain how adding the 'sed' RDEPENDS is related to
the normalize-resolvconf change?
> SRC_URI = "git://salsa.debian.org/debian/resolvconf.git;protocol=https;branch=unstable \
> file://99_resolvconf \
> @@ -46,6 +46,7 @@ do_install () {
> chown -R root:root ${D}${sysconfdir}/
> install -m 0755 bin/resolvconf ${D}${base_sbindir}/
> install -m 0755 bin/list-records ${D}${base_libdir}/${BPN}
> + install -m 0755 bin/normalize-resolvconf ${D}${base_libdir}/${BPN}
> install -d ${D}/${sysconfdir}/network/if-up.d
> install -m 0755 debian/resolvconf.000resolvconf.if-up ${D}/${sysconfdir}/network/if-up.d/000resolvconf
> install -d ${D}/${sysconfdir}/network/if-down.d
--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [OE-core] [PATCH] add missing normalize-resolvconf file since it has been moved to a separate file since version 1.90
[not found] ` <AM0PR06MB4625172F0F099B35BB980A7BD9B09@AM0PR06MB4625.eurprd06.prod.outlook.com>
@ 2022-06-20 9:49 ` Luca Ceresoli
2022-06-20 10:55 ` Peter Kjellerstedt
0 siblings, 1 reply; 6+ messages in thread
From: Luca Ceresoli @ 2022-06-20 9:49 UTC (permalink / raw)
To: Qorin Qorinna; +Cc: Qorin Wu, openembedded-core
Hi Qorin,
On Mon, 20 Jun 2022 09:24:22 +0000
Qorin Qorinna <qorin.qorinna@nl.abb.com> wrote:
> Hi Luca,
>
> it's because the normalize-resolvconf requires sed.
>
> These are the error messages that I got when I built the package w/o sed
> ERROR: resolvconf-1.91-r0 do_package_qa: QA Issue: /lib/resolvconf/normalize-resolvconf contained in package resolvconf requires /bin/sed, but no providers found in RDEPENDS:resolvconf? [file-rdeps]
> ERROR: resolvconf-1.91-r0 do_package_qa: Fatal QA errors were found, failing task.
I see. In this case it would be nice to add either a line to the commit
message or a comment in the .bb file, just to avoid others to wonder
why sed is needed.
I added a line to the commit message on my branch that I'm going to
test. It would be great if you could do so for future patches. Thanks!
--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [OE-core] [PATCH] add missing normalize-resolvconf file since it has been moved to a separate file since version 1.90
2022-06-20 9:49 ` Luca Ceresoli
@ 2022-06-20 10:55 ` Peter Kjellerstedt
2022-06-20 13:15 ` Qorin Wu
0 siblings, 1 reply; 6+ messages in thread
From: Peter Kjellerstedt @ 2022-06-20 10:55 UTC (permalink / raw)
To: luca.ceresoli, Qorin Qorinna; +Cc: Qorin Wu, openembedded-core
> -----Original Message-----
> From: openembedded-core@lists.openembedded.org <openembedded-
> core@lists.openembedded.org> On Behalf Of Luca Ceresoli via
> lists.openembedded.org
> Sent: den 20 juni 2022 11:50
> To: Qorin Qorinna <qorin.qorinna@nl.abb.com>
> Cc: Qorin Wu <qorin.wu@gmail.com>; openembedded-
> core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH] add missing normalize-resolvconf file since it has been moved to a separate file since version 1.90
It is expected that the subject line is prefixed with the recipe name.
It is also a very long subject line. Better would be to move half of
it to the commit message body, e.g.:
resolvconf: Install missing normalize-resolvconf file
It was moved to a separate file in version 1.90.
> Hi Qorin,
>
> On Mon, 20 Jun 2022 09:24:22 +0000
> Qorin Qorinna <qorin.qorinna@nl.abb.com> wrote:
>
> > Hi Luca,
> >
> > it's because the normalize-resolvconf requires sed.
> >
> > These are the error messages that I got when I built the package w/o sed
> > ERROR: resolvconf-1.91-r0 do_package_qa: QA Issue: /lib/resolvconf/normalize-resolvconf contained in package resolvconf requires /bin/sed, but no providers found in RDEPENDS:resolvconf? [file-rdeps]
> > ERROR: resolvconf-1.91-r0 do_package_qa: Fatal QA errors were found, failing task.
>
> I see. In this case it would be nice to add either a line to the commit
> message or a comment in the .bb file, just to avoid others to wonder
> why sed is needed.
The problem here is that sed is provided by busybox and in almost all
cases you do not want an explicit dependency on sed as that instead will
install the sed program from the sed recipe.
In this case I would recommend to add something like this:
# The normalize-resolvconf sed script uses /bin/sed on the shebang line.
# Ignore the file-rdeps test to avoid having to add a runtime dependency on sed.
INSANE_SKIP:${PN} += "file-rdeps"
> I added a line to the commit message on my branch that I'm going to
> test. It would be great if you could do so for future patches. Thanks!
>
> --
> Luca Ceresoli, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
//Peter
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [OE-core] [PATCH] add missing normalize-resolvconf file since it has been moved to a separate file since version 1.90
2022-06-20 10:55 ` Peter Kjellerstedt
@ 2022-06-20 13:15 ` Qorin Wu
0 siblings, 0 replies; 6+ messages in thread
From: Qorin Wu @ 2022-06-20 13:15 UTC (permalink / raw)
To: Peter Kjellerstedt; +Cc: luca.ceresoli, Qorin Qorinna, openembedded-core
[-- Attachment #1: Type: text/plain, Size: 2528 bytes --]
Hi Peter and Luca,
Thanks. I have updated the sed dependency and fixed the commit message.
Ignore the 2nd mail, the 3rd mail was the correct one.
Regards,
Qorin
On Mon, Jun 20, 2022 at 12:55 PM Peter Kjellerstedt <
peter.kjellerstedt@axis.com> wrote:
> > -----Original Message-----
> > From: openembedded-core@lists.openembedded.org <openembedded-
> > core@lists.openembedded.org> On Behalf Of Luca Ceresoli via
> > lists.openembedded.org
> > Sent: den 20 juni 2022 11:50
> > To: Qorin Qorinna <qorin.qorinna@nl.abb.com>
> > Cc: Qorin Wu <qorin.wu@gmail.com>; openembedded-
> > core@lists.openembedded.org
> > Subject: Re: [OE-core] [PATCH] add missing normalize-resolvconf file
> since it has been moved to a separate file since version 1.90
>
> It is expected that the subject line is prefixed with the recipe name.
> It is also a very long subject line. Better would be to move half of
> it to the commit message body, e.g.:
>
> resolvconf: Install missing normalize-resolvconf file
>
> It was moved to a separate file in version 1.90.
>
> > Hi Qorin,
> >
> > On Mon, 20 Jun 2022 09:24:22 +0000
> > Qorin Qorinna <qorin.qorinna@nl.abb.com> wrote:
> >
> > > Hi Luca,
> > >
> > > it's because the normalize-resolvconf requires sed.
> > >
> > > These are the error messages that I got when I built the package w/o
> sed
> > > ERROR: resolvconf-1.91-r0 do_package_qa: QA Issue:
> /lib/resolvconf/normalize-resolvconf contained in package resolvconf
> requires /bin/sed, but no providers found in RDEPENDS:resolvconf?
> [file-rdeps]
> > > ERROR: resolvconf-1.91-r0 do_package_qa: Fatal QA errors were found,
> failing task.
> >
> > I see. In this case it would be nice to add either a line to the commit
> > message or a comment in the .bb file, just to avoid others to wonder
> > why sed is needed.
>
> The problem here is that sed is provided by busybox and in almost all
> cases you do not want an explicit dependency on sed as that instead will
> install the sed program from the sed recipe.
>
> In this case I would recommend to add something like this:
>
> # The normalize-resolvconf sed script uses /bin/sed on the shebang line.
> # Ignore the file-rdeps test to avoid having to add a runtime dependency
> on sed.
> INSANE_SKIP:${PN} += "file-rdeps"
>
> > I added a line to the commit message on my branch that I'm going to
> > test. It would be great if you could do so for future patches. Thanks!
> >
> > --
> > Luca Ceresoli, Bootlin
> > Embedded Linux and Kernel engineering
> > https://bootlin.com
>
> //Peter
>
>
[-- Attachment #2: Type: text/html, Size: 3755 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [OE-core] [PATCH] add missing normalize-resolvconf file since it has been moved to a separate file since version 1.90
2022-06-20 8:08 [PATCH] add missing normalize-resolvconf file since it has been moved to a separate file since version 1.90 Qorin
2022-06-20 9:20 ` [OE-core] " Luca Ceresoli
@ 2022-06-21 8:11 ` Luca Ceresoli
1 sibling, 0 replies; 6+ messages in thread
From: Luca Ceresoli @ 2022-06-21 8:11 UTC (permalink / raw)
To: Qorin Wu; +Cc: openembedded-core, Qorin
Hi Qorin,
On Mon, 20 Jun 2022 10:08:20 +0200
"Qorin Wu" <qorin.wu@gmail.com> wrote:
> From: Qorin <qorin.qorinna@nl.abb.com>
>
> ---
> meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb b/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb
> index 94fd2c1a70..d2d9973852 100644
> --- a/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb
> +++ b/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb
> @@ -9,7 +9,7 @@ LICENSE = "GPL-2.0-or-later"
> LIC_FILES_CHKSUM = "file://COPYING;md5=c93c0550bd3173f4504b2cbd8991e50b"
> AUTHOR = "Thomas Hood"
> HOMEPAGE = "http://packages.debian.org/resolvconf"
> -RDEPENDS:${PN} = "bash"
> +RDEPENDS:${PN} = "bash sed"
>
> SRC_URI = "git://salsa.debian.org/debian/resolvconf.git;protocol=https;branch=unstable \
> file://99_resolvconf \
> @@ -46,6 +46,7 @@ do_install () {
> chown -R root:root ${D}${sysconfdir}/
> install -m 0755 bin/resolvconf ${D}${base_sbindir}/
> install -m 0755 bin/list-records ${D}${base_libdir}/${BPN}
> + install -m 0755 bin/normalize-resolvconf ${D}${base_libdir}/${BPN}
> install -d ${D}/${sysconfdir}/network/if-up.d
> install -m 0755 debian/resolvconf.000resolvconf.if-up ${D}/${sysconfdir}/network/if-up.d/000resolvconf
> install -d ${D}/${sysconfdir}/network/if-down.d
When running this patch in the autobuilders, there are failures related
to resolvconf:
ERROR: When reparsing /tmp/selftest-fetchpkz67tpd/test.bb:do_checkuri,
the basehash value changed from 84b9e23a83a764e7a5d8a96ee523199e2b1c604f4280a45030e248de0ff52962 to 4123da244f465feffecb2b4fd2b132b64851981ec2fb70bf9445df8b0ea6a69e. The metadata is not deterministic and this needs to be fixed.
I must say there is no clear reason for me for this error given the
simple patch you sent. Other here can be more helpful. I'm waiting
for this to be understood before taking updated version of your patch.
Here are some logs showing the error:
https://autobuilder.yoctoproject.org/typhoon/#/builders/80/builds/3671/steps/14/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/86/builds/3687/steps/14/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/87/builds/3731/steps/14/logs/stdio
https://autobuilder.yoctoproject.org/typhoon/#/builders/79/builds/3723/steps/15/logs/stdio
--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-06-21 8:11 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-20 8:08 [PATCH] add missing normalize-resolvconf file since it has been moved to a separate file since version 1.90 Qorin
2022-06-20 9:20 ` [OE-core] " Luca Ceresoli
[not found] ` <AM0PR06MB4625172F0F099B35BB980A7BD9B09@AM0PR06MB4625.eurprd06.prod.outlook.com>
2022-06-20 9:49 ` Luca Ceresoli
2022-06-20 10:55 ` Peter Kjellerstedt
2022-06-20 13:15 ` Qorin Wu
2022-06-21 8:11 ` Luca Ceresoli
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.