On Tue, 2021-01-26 at 00:08 +0100, Dorinda wrote:
> If a user builds in a path in PSEUDO_IGNORE_PATHS, random failures are
> generated. Hence this patch adds a sanity check in sanity.bbclass to ensure
> that a user isn't building in PSEUDO_IGNORE_PATHS.
>
> [YOCTO #14179]
>
> Signed-off-by: Dorinda Bassey <dorindabas...@gmail.com>
> ---
> meta/classes/sanity.bbclass | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
> index 2040b48595..14271c1ca9 100644
> --- a/meta/classes/sanity.bbclass
> +++ b/meta/classes/sanity.bbclass
> @@ -703,6 +703,13 @@ def check_sanity_version_change(status, d):
> if (tmpdirmode & stat.S_ISUID):
> status.addresult("TMPDIR is setuid, please don't build in a setuid
> directory")
>
>
>
>
> + # Check that a user isn't building in a path in PSEUDO_IGNORE_PATHS
> + pseudoignorepaths = d.getVar('PSEUDO_IGNORE_PATHS',
> expand=True).split(",")
> + workdir = d.getVar('WORKDIR', expand=True)
> + for i in pseudoignorepaths:
> + if i and workdir.startswith(i):
> + status.addresult("You are building in a path in
> PSEUDO_IGNORE_PATHS: " + str(i) + " please don't build in this directory: " +
> str(workdir) + "\n")
>
To ensure the user fully understands this, should this be something
like:
status.addresult("You are building in a path included in
PSEUDO_IGNORE_PATHS " + str(i) + " which will not work, please locate
the build outside this path.\n")
as we then tell the user what to do (and don't repeat effectively the
same information twice).
Cheers,
Richard
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#147234):
https://lists.openembedded.org/g/openembedded-core/message/147234
Mute This Topic: https://lists.openembedded.org/mt/80117826/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
