<div dir="ltr">Do you already have similar patch for nativesdk-shadow somewhere?<div><br></div><div>One thing I've noticed with new set -e is that you get only one failure per run, which is expected.</div><div><br></div><div>Just when I had 8 different update-alternatives call failing in busybox postinst (due to different defconfig we use) I would appreciate to see all errors in one go. But I don't see how to "improve" this easily, running it twice with and without "set -e" would probably cause more confusion than helping with too many errors in one postinst.</div><div><br></div><div>It would be great if the output of postinst script could be extracted to separate file or the warning provided some simple way to re-execute just the failing postinst manually (e.g. with devtool). Currently I do it manually, which I cannot recommend to anyone (basically removing rm_work from INHERIT to keep rootfs after do_rootfs, then jumping to rootfs directory, exporting D and running the postinst from OPKGLIBDIR).</div><div><br></div><div>Regards,</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 15, 2018 at 12:27 PM, Alexander Kanavin <span dir="ltr"><<a href="mailto:alexander.kanavin@linux.intel.com" target="_blank">alexander.kanavin@linux.intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Nativesdk package has a special arrangement where the same thing is done<br>
in do_install(). It was assumed (in the comment) that postinsts don't run when<br>
installing nativesdk packages, but this was incorrect: they are run, but<br>
any failures were previously silently ignored. Now this missing failure reporting has<br>
been fixed, and so we get to see the failures.<br>
<br>
Signed-off-by: Alexander Kanavin <<a href="mailto:alexander.kanavin@linux.intel.com">alexander.kanavin@linux.<wbr>intel.com</a>><br>
---<br>
meta/recipes-support/ca-<wbr>certificates/<a href="http://ca-certificates_20170717.bb" rel="noreferrer" target="_blank">ca-certificates_<wbr>20170717.bb</a> | 5 +++--<br>
1 file changed, 3 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/meta/recipes-support/ca-<wbr>certificates/<a href="http://ca-certificates_20170717.bb" rel="noreferrer" target="_blank">ca-certificates_<wbr>20170717.bb</a> b/meta/recipes-support/ca-<wbr>certificates/<a href="http://ca-certificates_20170717.bb" rel="noreferrer" target="_blank">ca-certificates_<wbr>20170717.bb</a><br>
index 52a1a07f68c..51af72e79a3 100644<br>
--- a/meta/recipes-support/ca-<wbr>certificates/<a href="http://ca-certificates_20170717.bb" rel="noreferrer" target="_blank">ca-certificates_<wbr>20170717.bb</a><br>
+++ b/meta/recipes-support/ca-<wbr>certificates/<a href="http://ca-certificates_20170717.bb" rel="noreferrer" target="_blank">ca-certificates_<wbr>20170717.bb</a><br>
@@ -64,13 +64,14 @@ do_install_append_class-target () {<br>
${D}${mandir}/man8/update-ca-<wbr>certificates.8<br>
}<br>
<br>
-pkg_postinst_${PN} () {<br>
+pkg_postinst_${PN}_class-<wbr>target () {<br>
SYSROOT="$D" $D${sbindir}/update-ca-<wbr>certificates<br>
}<br>
<br>
CONFFILES_${PN} += "${sysconfdir}/ca-<wbr>certificates.conf"<br>
<br>
-# Postinsts don't seem to be run for nativesdk packages when populating SDKs.<br>
+# Rather than make a postinst script that works for both target and nativesdk,<br>
+# we just run update-ca-certificate from do_install() for nativesdk.<br>
CONFFILES_${PN}_append_class-<wbr>nativesdk = " ${sysconfdir}/ssl/certs/ca-<wbr>certificates.crt"<br>
do_install_append_class-<wbr>nativesdk () {<br>
SYSROOT="${D}${SDKPATHNATIVE}" ${D}${sbindir}/update-ca-<wbr>certificates<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.16.1<br>
<br>
--<br>
______________________________<wbr>_________________<br>
Openembedded-core mailing list<br>
<a href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.<wbr>openembedded.org</a><br>
<a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" rel="noreferrer" target="_blank">http://lists.openembedded.org/<wbr>mailman/listinfo/openembedded-<wbr>core</a><br>
</font></span></blockquote></div><br></div>