On Tue, Sep 1, 2020 at 3:41 PM Martin Jansa <martin.ja...@gmail.com> wrote:
>
> Looks like my "solution" also isn't working well at least for images with 
> read-only-rootfs in IMAGE_FEATURES.
>
> pkg_postinst_ontarget_${PN} is always forced to run on the target, which for 
> read-only-rootfs results in:
> log.do_rootfs:
> ...
> NOTE: If an image is being built, the postinstalls for the following packages 
> will be postponed for first boot: nss
> ...
> ERROR: The following packages could not be configured offline and rootfs is 
> read-only: ['nss']
>
> and now looking at the /usr/bin/signlibs.sh and it does pretty much
> the same as the postinst script when D isn't empty.
>
> From oe-core git history it shows that signlibs.sh was added first:
> https://git.openembedded.org/openembedded-core/commit/?id=a4580f967c8064294a06d406acf5deb24aee2acc
> then the offline version of postinst was added to support read-only-rootfs in:
> https://git.openembedded.org/openembedded-core/commit/?id=64e87fc6e99bc1d4807034166735034b1f92bad8
> and nss-native should always provide the shlibsign since:
> https://git.openembedded.org/openembedded-core/commit/?id=88540c5b08dea069660d1a68e506aebdd68e6ae0
> and only after
> https://git.openembedded.org/openembedded-core/commit/?id=8f782f7095e718dd9452055af53363beb6bdbece
> it looked like signlibs.sh was something special only for target.
>
> So it looks to me, that we should just remove signlibs.sh script and let the 
> same postinst be used on target and offline
> (with or without D being empty).
>
> Signed-off-by: Martin Jansa <martin.ja...@gmail.com>
> ---
>  meta-oe/recipes-support/nss/nss/signlibs.sh | 20 --------------------
>  meta-oe/recipes-support/nss/nss_3.56.bb     |  6 ------
>  2 files changed, 26 deletions(-)
>  delete mode 100644 meta-oe/recipes-support/nss/nss/signlibs.sh
>
> diff --git a/meta-oe/recipes-support/nss/nss/signlibs.sh 
> b/meta-oe/recipes-support/nss/nss/signlibs.sh
> deleted file mode 100644
> index a74e499f8c..0000000000
> --- a/meta-oe/recipes-support/nss/nss/signlibs.sh
> +++ /dev/null
> @@ -1,20 +0,0 @@
> -#!/bin/sh
> -
> -# signlibs.sh
> -#
> -# (c)2010 Wind River Systems, Inc.
> -#
> -# regenerates the .chk files for the NSS libraries that require it
> -# since the ones that are built have incorrect checksums that were
> -# calculated on the host where they really need to be done on the
> -# target
> -

if we remove this then how do we solve above said issue? is it still an issue ?

> -CHK_FILES=`ls /lib*/*.chk /usr/lib*/*.chk 2>/dev/null`
> -SIGN_BINARY=`which shlibsign`
> -for I in $CHK_FILES
> -do
> -       DN=`dirname $I`
> -       BN=`basename $I .chk`
> -       FN=$DN/$BN.so
> -       $SIGN_BINARY -i $FN
> -done
> diff --git a/meta-oe/recipes-support/nss/nss_3.56.bb 
> b/meta-oe/recipes-support/nss/nss_3.56.bb
> index 107b0a0152..997f8950fd 100644
> --- a/meta-oe/recipes-support/nss/nss_3.56.bb
> +++ b/meta-oe/recipes-support/nss/nss_3.56.bb
> @@ -21,7 +21,6 @@ VERSION_DIR = "${@d.getVar('BP').upper().replace('-', 
> '_').replace('.', '_') + '
>
>  SRC_URI = 
> "http://ftp.mozilla.org/pub/mozilla.org/security/nss/releases/${VERSION_DIR}/src/${BP}.tar.gz
>  \
>             file://nss.pc.in \
> -           file://signlibs.sh \
>             file://0001-nss-fix-support-cross-compiling.patch \
>             file://nss-no-rpath-for-cross-compiling.patch \
>             file://nss-fix-incorrect-shebang-of-perl.patch \
> @@ -214,7 +213,6 @@ do_install_append() {
>          touch ${D}/${libdir}/$file
>          chmod 755 ${D}/${libdir}/$file
>      done
> -    install -D -m 755 ${WORKDIR}/signlibs.sh ${D}/${bindir}/signlibs.sh
>
>      install -d ${D}${libdir}/pkgconfig/
>      sed 's/%NSS_VERSION%/${PV}/' ${WORKDIR}/nss.pc.in | sed 
> 's/%NSPR_VERSION%/4.9.2/' > ${D}${libdir}/pkgconfig/nss.pc
> @@ -251,10 +249,6 @@ pkg_postinst_${PN} () {
>      done
>  }
>
> -pkg_postinst_ontarget_${PN} () {
> -    ${bindir}/signlibs.sh
> -}
> -
>  PACKAGES =+ "${PN}-smime"
>  FILES_${PN}-smime = "\
>      ${bindir}/smime \
> --
> 2.25.1
>
> 
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

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

Reply via email to