Hi Khem, Thanks for your suggestion. Yes, we can avoid dependency on coreutils by using 'readlink -f'.
I checked the git history a little bit, resolvconf wants to depend on coreutils, and looking at the '-m' option, we can reasonably guess it wants to handle the case of parent directory not existing. But in our case, /etc is always there. There's really no need to use '-m'. 'readlink -f' can also work. I've used a patch to avoid coreutils dependency in V3. It will be sent out soon. Regards, Qi -----Original Message----- From: Khem Raj <[email protected]> Sent: Wednesday, November 16, 2022 2:36 PM To: Chen, Qi <[email protected]> Cc: [email protected] Subject: Re: [OE-core][PATCH] resolvconf: make it work On Tue, Nov 15, 2022 at 5:45 AM Chen Qi <[email protected]> wrote: > > The current resolvconf does not work. Make it work with the following > changes. > 1. Install normalize-resolvconf, which is used by resolvconf. > 2. Add dependencies sed, coreutils, util-linux-flock. > util-linux-flock is needed by our busybox does not support '-w' > by default. coreutils is needed by the busybox's readlink does > not support '-m'. sed is needed because we want to avoid package > QA issue complaining sed is needed by no one provides it. Can we avoid dependency on coreutils ? maybe check why -m is needed and perhaps just use absolute link. > > Signed-off-by: Chen Qi <[email protected]> > --- > meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb > b/meta/recipes-connectivity/resolvconf/resolvconf_1.91.bb > index 94fd2c1a70..3bf5ec6c32 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 coreutils util-linux-flock" > > SRC_URI = > "git://salsa.debian.org/debian/resolvconf.git;protocol=https;branch=unstable \ > file://99_resolvconf \ > @@ -39,12 +39,14 @@ do_install () { > fi > install -d ${D}${base_libdir}/${BPN} > install -d ${D}${sysconfdir}/${BPN} > + install -d ${D}${nonarch_base_libdir}/${BPN} > ln -snf ${localstatedir}/run/${BPN} ${D}${sysconfdir}/${BPN}/run > install -d ${D}${sysconfdir} ${D}${base_sbindir} > install -d ${D}${mandir}/man8 ${D}${docdir}/${P} > cp -pPR etc/resolvconf ${D}${sysconfdir}/ > chown -R root:root ${D}${sysconfdir}/ > install -m 0755 bin/resolvconf ${D}${base_sbindir}/ > + install -m 0755 bin/normalize-resolvconf > + ${D}${nonarch_base_libdir}/${BPN} > install -m 0755 bin/list-records ${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 > @@ -64,4 +66,4 @@ pkg_postinst:${PN} () { > fi > } > > -FILES:${PN} += "${base_libdir}/${BPN}" > +FILES:${PN} += "${base_libdir}/${BPN} ${nonarch_base_libdir}/${BPN}" > -- > 2.17.1 > > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#173416): https://lists.openembedded.org/g/openembedded-core/message/173416 Mute This Topic: https://lists.openembedded.org/mt/95042786/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
