Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rpi-base: filter None values from DTB boot files #1402

Open
wants to merge 1 commit into
base: scarthgap
Choose a base branch
from

Conversation

lhoward
Copy link

@lhoward lhoward commented Jan 12, 2025

Filter None values from DTB boot files, to avoid errors expanding IMAGE_BOOT_FILES

@lhoward
Copy link
Author

lhoward commented Jan 12, 2025

I don't really know Python, so not sure what it's about the version I've installed that caused this. It would be great if (after review) it could also be back-ported to the scarthgap branch.

@agherzan
Copy link
Owner

I don't really know Python, so not sure what it's about the version I've installed that caused this. It would be great if (after review) it could also be back-ported to the scarthgap branch.

Thanks for this. I would like to understand why this happened in the first place. Could you dump the environment and check the value there? Also, a fix need to firstly land in master (and you can follow up with a cherry pick PR in other branches).

@lhoward
Copy link
Author

lhoward commented Jan 16, 2025

Note to self: a colleague suggested

⁠ return ' '.join(x for x in (transform(dtb) for dtb in alldtbs.split(' ') if dtb) if x is not None) ⁠

would be more idiomatic

@lhoward
Copy link
Author

lhoward commented Jan 16, 2025

I'll rerun the build without the patch when I'm at my desk, but my hunch is that the whitespaces in IMAGE_BOOT_FILES are being expanded split to None (at least on my system).

@lhoward
Copy link
Author

lhoward commented Jan 17, 2025

Error I saw was:

ERROR: ExpansionError during parsing /home/lukeh/CVSRoot/padl/inferno/build/xebros/meta/recipes-core/images/core-image-minimal.bb
bb.data_smart.ExpansionError: Failure expanding variable IMAGE_BOOT_FILES, expression was bootfiles/*                  ${@make_dtb_boot_files(d)}                   	Image u-boot.bin;kernel8.img boot.scr 	                   which triggered exception TypeError: sequence item 59: expected str instance, NoneType found
The variable dependency chain for the failure is: IMAGE_BOOT_FILES

I don't know enough about BitBake to know how to dump more...

Filter None values from DTB boot files, to avoid errors expanding
IMAGE_BOOT_FILES

Signed-off-by: Luke Howard <[email protected]>
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.

2 participants