diff mbox series

[2/2] update_mandb: deb fails due to missing man cache

Message ID 20230914143919.985474-3-pidge@baylibre.com
State Accepted, archived
Commit a411123a95114233c5efd762dbcc8eb513030aab
Headers show
Series Ensure sdk.py knows about nativesdk-intercepts | expand

Commit Message

Eilís 'pidge' Ní Fhlannagáin Sept. 14, 2023, 2:39 p.m. UTC
This only occurs in debian package builds when populating the sdk
and is a work around that seems to work. Eventually we should look
at why this is failing (I have ideas, it's somewhere in
lib/oe/package_management/deb/sdk.py), but for now, do this so we can
fix the core issue with nativesdk-intercepts.

Signed-off-by: Eilís 'pidge' Ní Fhlannagáin <pidge@baylibre.com>
---
 scripts/postinst-intercepts/update_mandb | 2 ++
 1 file changed, 2 insertions(+)

Comments

Khem Raj Sept. 14, 2023, 5:30 p.m. UTC | #1
On Thu, Sep 14, 2023 at 7:39 AM Eilís 'pidge' Ní Fhlannagáin
<pidge@baylibre.com> wrote:
>
> This only occurs in debian package builds when populating the sdk
> and is a work around that seems to work. Eventually we should look
> at why this is failing (I have ideas, it's somewhere in
> lib/oe/package_management/deb/sdk.py), but for now, do this so we can
> fix the core issue with nativesdk-intercepts.
>
> Signed-off-by: Eilís 'pidge' Ní Fhlannagáin <pidge@baylibre.com>
> ---
>  scripts/postinst-intercepts/update_mandb | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/scripts/postinst-intercepts/update_mandb b/scripts/postinst-intercepts/update_mandb
> index a061bb426a6..f91bafdb117 100644
> --- a/scripts/postinst-intercepts/update_mandb
> +++ b/scripts/postinst-intercepts/update_mandb
> @@ -9,6 +9,8 @@ set -eu
>  CONFIG=$(mktemp --tmpdir update-mandb.XXXXX)
>  sed "s:\(\s\)/:\1$D/:g" $D${sysconfdir}/man_db.conf > $CONFIG
>
> +mkdir -p $D${localstatedir}/cache/man/
> +

mandb will generate index cache files. into D/var/cache/man and below
qemu usermode call is doing it.
Perhaps we can look into man_db.conf to select where these build time
generated caches should be stored
/var/cache seems a wrong place to me, since its r/w and populated at
runtime in many cases.

>  PSEUDO_UNLOAD=1 ${binprefix}qemuwrapper -L $D $D${bindir}/mandb --config-file $CONFIG --create
>
>  rm -f $CONFIG
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#187644): https://lists.openembedded.org/g/openembedded-core/message/187644
> Mute This Topic: https://lists.openembedded.org/mt/101359340/1997914
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/scripts/postinst-intercepts/update_mandb b/scripts/postinst-intercepts/update_mandb
index a061bb426a6..f91bafdb117 100644
--- a/scripts/postinst-intercepts/update_mandb
+++ b/scripts/postinst-intercepts/update_mandb
@@ -9,6 +9,8 @@  set -eu
 CONFIG=$(mktemp --tmpdir update-mandb.XXXXX)
 sed "s:\(\s\)/:\1$D/:g" $D${sysconfdir}/man_db.conf > $CONFIG
 
+mkdir -p $D${localstatedir}/cache/man/
+
 PSEUDO_UNLOAD=1 ${binprefix}qemuwrapper -L $D $D${bindir}/mandb --config-file $CONFIG --create
 
 rm -f $CONFIG