On 12/11/23 2:35 PM, Ross Burton wrote:
On 11 Dec 2023, at 17:35, Richard Purdie via lists.openembedded.org
<[email protected]> wrote:
Currently, pseudo tracks all files referenced within it's presence unless
they're listed in an exclusion list. The exclusion list has grown to be
fairly unwieldy.
This patch swaps PSEUDO_IGNORE_PATHS for PSEUDO_INCLUDE_PATHS which in
theory should be easier and more explicit to maintain.
I’ve never understood the rationale behind exclude over include (then again
there’s lots I don’t understand), so I’m very much for this in principle.
History:
It was originally everything, because files were referenced, linked and copied
around random parts of the system and/or build and needed preservation of
components. This was very early Yocto Project and before. (Psuedo existed for
many years before OE/YP used it!)
Then the build/work directories became more standardized, and the components
'copying in' from elsewhere was greatly reduced... but there were points where
reference files were still used (think .m4 files or tex templates for instance
for the kinds of things...) Since we didn't know what all of the reference
points (to the host) were, we decided to be conservative to add exclusion.
So excluding paths where we could count on owners/groups (and more importantly
permissions) being standard became the norm. That's why the primary exclusion was:
/usr/,/etc/,/lib,/dev/,/run/
Over time though we've gotten our dependencies understood and really severely
limited the external components we use in the build. (including .m4, autoconf,
tex, etc etc etc..) So moving to an INCLUDE seems to make more sense to me.
Since we have a clear understanding of the path structure we use....
/tmp - used by the compiler (and tons of other tooling) for temporary files, we
need this for sure.
/proc - there are referenced entries that MIGHT be needed, especially links to
/proc/mounts (where the link inherits perms)
/dev - I'm not sure if this is needed, but things like /dev/null and /dev/shm,
another tmpfs, mean we probably need to track this.
/run - often contains other short-term temp files, especially for things like
trying to process pre/post package scripting on rootfs configuration.
Then the items in WORKDIR of course. IN THE PAST, we needed source code to be
covered both to handle symlink references, as well as specific copy functions.
But I suspect this is not needed any longer. The actual build directory also is
likely not needed anymore (but had been in the past!) Leaving us the image,
package, rootfs, etc directories.
So I think this maps well to "how did we get here".
The one question I ask, is why we're using ',' as a separator and not ':' like
standard Unix style paths. But I suspect the answer is Windows level
support.... but I'd be very tempted to move it back to ':' to make this more
standard like other path separators. (But that can be done at another time!)
--Mark
Ross
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#192242):
https://lists.openembedded.org/g/openembedded-core/message/192242
Mute This Topic: https://lists.openembedded.org/mt/103113368/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
