Skip to content

sdcard_image-rpi: Fix ROOTFS_POSTPROCESS_COMMAND - #1638

Merged
kraj merged 1 commit into
agherzan:masterfrom
leon-anavi:leon/fix-ROOTFS_POSTPROCESS_COMMAND
Oct 27, 2026
Merged

kraj merged 1 commit into
agherzan:masterfrom
leon-anavi:leon/fix-ROOTFS_POSTPROCESS_COMMAND

Conversation

@leon-anavi

Copy link
Copy Markdown
Contributor

- What I did

Fixed:

WARNING: ROOTFS_POSTPROCESS_COMMAND contains ';' but is a space separated list of functions, remove the ';' delimiters or the functions will not be part of the task signature

- How I did it

Dropped ; from ROOTFS_POSTPROCESS_COMMAND in sdcard_image-rpi.bbclass.

Drop ; from ROOTFS_POSTPROCESS_COMMAND in sdcard_image-rpi.bbclass.

Fixes:

WARNING: ROOTFS_POSTPROCESS_COMMAND contains ';' but is a space
separated list of functions, remove the ';' delimiters or the
functions will not be part of the task signature

This work was sponsored by GOVCERT.LU.

Signed-off-by: Leon Anavi <leon.anavi@konsulko.com>
@jwinarske

Copy link
Copy Markdown
Contributor

Tested on master with raspberrypi5. Without this every image in the build throws the warning (core-image-*, meta-oe, meta-security, meta-flutter), with it they go away.

Note the ';' also keeps rpi_generate_sysctl_config out of the do_rootfs signature, so changes to it don't trigger a rebuild. wrynose has the same line, so this should probably be backported there too.

Tested-by: Joel Winarske joel.winarske@linux.com

@leon-anavi

Copy link
Copy Markdown
Contributor Author

@jwinarske, thank you testing my change.

wrynose has the same line, so this should probably be backported there too.

Yes, once this patch is merged, I plan to backport it to the Wrynose release as well.

@kraj
kraj enabled auto-merge (rebase) October 26, 2026 14:40
@kraj
kraj merged commit 96d0b6d into agherzan:master Oct 27, 2026
20 checks passed
@leon-anavi

Copy link
Copy Markdown
Contributor Author

I cherry-picked the patch to Wrynose. After testing it on my side with Raspberry Pi I opened anoter GitHub Pull Request against branch wrynose: #1639

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants