From: Alexander Kanavin <alex.kanavin@gmail.com>
To: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
Cc: "openembedded-core@lists.openembedded.org"
<openembedded-core@lists.openembedded.org>,
Alexander Kanavin <alex@linutronix.de>
Subject: Re: [OE-core] [PATCH] scripts/oe-setup-builddir: migrate build/conf/templateconf.cfg to new template locations
Date: Fri, 2 Sep 2022 06:31:50 +0200 [thread overview]
Message-ID: <CANNYZj9cXbFogXaJdTeb2r89-AaPyL3txoJ4xn3ER=Dq7jy0mA@mail.gmail.com> (raw)
In-Reply-To: <d9ec6fa663d94bcc9229066100a140fc@axis.com>
On Fri, 2 Sept 2022 at 02:12, Peter Kjellerstedt
<peter.kjellerstedt@axis.com> wrote:
> Since the test you added previously now requires that the path contains
> "/templates/", we might as well do this for any faulty configuration. I.e.:
>
> # The path in $TEMPLATECONF must contain "/templates/". If it doesn't,
> # unset it to have it replaced by a (hopefully) correct path.
> case $TEMPLATECONF in
> */templates/*)
> ;;
> *)
> unset TEMPLATECONF
> rm "$BUILDDIR/conf/templateconf.cfg"
> ;;
> esac
>
> If .templateconf hasn't been updated to comply with the new rules, the
> test later will point this out. But if it has been updated, the case
> statement above will avoid the problem of an old, non-complying
> configuration regardless of which layer it comes from.
.templateconf is not a universal file that can be placed into any
layer, it is specific to poky/oe-core, as it is read from $OEROOT,
which in turn is defined as location of oe-init-build-env.
So if the fixing up is made generic as suggested, it will only replace
incorrect but at least helpful value of the original custom template
location with the generic default template location in poky/core.
We can however add a hint to check what is in
build/conf/templateconf.cfg if the test fails, as it is not obvious
that TEMPLATECONF obtained from there takes priority over other
sources (which are, in order, TEMPLATECONF passed via environment,
then .templateconf next to oe-init-build-env).
Alex
next prev parent reply other threads:[~2022-09-02 4:32 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-01 14:23 [PATCH] scripts/oe-setup-builddir: migrate build/conf/templateconf.cfg to new template locations Alexander Kanavin
2022-09-02 0:12 ` [OE-core] " Peter Kjellerstedt
2022-09-02 4:31 ` Alexander Kanavin [this message]
2022-09-02 14:47 ` Peter Kjellerstedt
2022-09-02 15:08 ` Alexander Kanavin
2022-09-02 15:41 ` Peter Kjellerstedt
2022-09-02 16:50 ` Alexander Kanavin
[not found] ` <1710F0E3F6A43399.12033@lists.openembedded.org>
2022-09-02 7:16 ` Alexander Kanavin
2022-09-02 7:37 ` Richard Purdie
2022-09-02 7:41 ` Alexander Kanavin
[not found] ` <1710FB3AA9843A6F.13805@lists.openembedded.org>
2022-09-02 10:57 ` Alexander Kanavin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CANNYZj9cXbFogXaJdTeb2r89-AaPyL3txoJ4xn3ER=Dq7jy0mA@mail.gmail.com' \
--to=alex.kanavin@gmail.com \
--cc=alex@linutronix.de \
--cc=openembedded-core@lists.openembedded.org \
--cc=peter.kjellerstedt@axis.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.