All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Richard Purdie" <richard.purdie@linuxfoundation.org>
To: ChenQi <Qi.Chen@windriver.com>, Dorinda <dorindabassey@gmail.com>,
	 openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] sanity.bbclass: verify that user isn't building in PSEUDO_IGNORE_PATHS
Date: Wed, 27 Jan 2021 09:49:35 +0000	[thread overview]
Message-ID: <423d07e79bdbbacc58d63ff64c8fa1bd8f70f9f9.camel@linuxfoundation.org> (raw)
In-Reply-To: <4e6da54d-f6ef-9c04-4bdb-5af64807cbd7@windriver.com>

Hi,

On Wed, 2021-01-27 at 17:46 +0800, ChenQi wrote:
>  Is it possible to use an opposite way for pseudo path filtering?
>  Instead of ignoring paths, only take into consideration paths in
> something like PSEUDO_CONSIDER_PATHS.
>  The list should be short. ${D}, ${PKGD}, ${PKGDEST},
> ${IMAGE_ROOTFS}?

Its a good question, I was convinced by the pseudo authors that it was
a bad idea. The reason being that it isn't just the above paths but we
also need to catch calls where files are transiently in other
directories or in other parts of the filesystem such as /tmp/. Code
creates files in interesting and convoluted ways to preserve
permissions, for security reasons and so on and there was a worry that
masking just a specific list would cause a different set of problems.
It probably shouldn't be crossing filesystem boundaries doing it but I
know there were complications due to it.

That said, now that pseudo can support path filtering, it may be worth
an experiment to see how whether it could work better than what we're
doing now. The algorithm (whether its include or exclude) should be
relatively easy to test.

I can also say the above list would need expanding as some sstate
objects need to preserve ownership/permissions during creating and so
on. It probably would still be simpler than our current ignore list
though.

Cheers,

Richard 





  reply	other threads:[~2021-01-27  9:49 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-25 23:08 [PATCH] sanity.bbclass: verify that user isn't building in PSEUDO_IGNORE_PATHS Dorinda
2021-01-25 23:54 ` [OE-core] " Richard Purdie
2021-01-27  9:46   ` Chen Qi
2021-01-27  9:49     ` Richard Purdie [this message]
2021-01-28  8:43       ` Chen Qi
2021-01-29  8:49       ` Chen Qi
2021-01-26  0:53 Dorinda
2021-01-26  9:20 ` [OE-core] " Quentin Schulz

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=423d07e79bdbbacc58d63ff64c8fa1bd8f70f9f9.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=Qi.Chen@windriver.com \
    --cc=dorindabassey@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    /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.