diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2024-01-02 17:22:14 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2024-01-18 10:15:58 +0000 |
commit | bc883e49912b8d0cdc9ea403e85f2cb3198b48ff (patch) | |
tree | ad622ca70314eb78e12a74ceb4d09f4b75fc17a9 /meta/lib/oe | |
parent | 12619deabd1f85d88ff14d7c3861e23eeb58bf1b (diff) | |
download | poky-bc883e49912b8d0cdc9ea403e85f2cb3198b48ff.tar.gz |
rootfs-postcommands: Try and improve ordering constraints
The current code is in race to see who can set things last. This isn't scalable or
sustainable and problemtic in the face of inherit ordering changes.
Move the ordering issue into the actual code execution, which isn't ideal but
the best of several bad options and at least lets us drop the anonymous python.
(From OE-Core rev: 0ffff2c1f80a9b79b133d787764bab164d9abd70)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oe')
-rw-r--r-- | meta/lib/oe/rootfs.py | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/meta/lib/oe/rootfs.py b/meta/lib/oe/rootfs.py index 3f27164536..8cd48f9450 100644 --- a/meta/lib/oe/rootfs.py +++ b/meta/lib/oe/rootfs.py | |||
@@ -194,6 +194,18 @@ class Rootfs(object, metaclass=ABCMeta): | |||
194 | post_process_cmds = self.d.getVar("ROOTFS_POSTPROCESS_COMMAND") | 194 | post_process_cmds = self.d.getVar("ROOTFS_POSTPROCESS_COMMAND") |
195 | rootfs_post_install_cmds = self.d.getVar('ROOTFS_POSTINSTALL_COMMAND') | 195 | rootfs_post_install_cmds = self.d.getVar('ROOTFS_POSTINSTALL_COMMAND') |
196 | 196 | ||
197 | def make_last(command, commands): | ||
198 | commands = commands.split() | ||
199 | if command in commands: | ||
200 | commands.remove(command) | ||
201 | commands.append(command) | ||
202 | return "".join(commands) | ||
203 | |||
204 | # We want this to run as late as possible, in particular after | ||
205 | # systemd_sysusers_create and set_user_group. Using :append is not enough | ||
206 | make_last("tidy_shadowutils_files", post_process_cmds) | ||
207 | make_last("rootfs_reproducible", post_process_cmds) | ||
208 | |||
197 | execute_pre_post_process(self.d, pre_process_cmds) | 209 | execute_pre_post_process(self.d, pre_process_cmds) |
198 | 210 | ||
199 | if self.progress_reporter: | 211 | if self.progress_reporter: |