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

feat(bundle.bbclass): simplify varflag usage #290

Closed
wants to merge 1 commit into from

Conversation

Jarsop
Copy link
Contributor

@Jarsop Jarsop commented Nov 8, 2023

In many case getVarFlag is used but slotflags coming from getVarFlags previous call in bitbake task (do_configure) it's already expanded. This improvement use slotflags.
It simplify also the loop over RAUC_META_SECTIONS.

@Jarsop
Copy link
Contributor Author

Jarsop commented Nov 8, 2023

@ejoerns the second commit is to passing ci build, if required it can be cherry-picked to another branch to isolate the ci fix and drop from this branch history

@ejoerns
Copy link
Member

ejoerns commented Nov 9, 2023

@Jarsop Thanks! I have cherry-picked the CI fix on master (and slightly adapted the commit message).

Feel free to rebase your PRs

In many case `getVarFlag` is used but `slotflags` coming from `getVarFlags`
previous call in bitbake task (`do_configure`) it's already expanded.
This improvement use `slotflags`.
It simplify also the loop over `RAUC_META_SECTIONS`.

Signed-off-by: Jean-Pierre Geslin <jarsop@outlook.com>
@Jarsop Jarsop force-pushed the feat/simplify-varflag-usage branch from 1613794 to bb1145d Compare November 9, 2023 19:35
@Jarsop
Copy link
Contributor Author

Jarsop commented Nov 9, 2023

@ejoerns Perfect ! Did it. However it seems that rauc layer is no more compatible as this commit (3 hours ago) remove mickledore core layer compatibility. So we need to add nanbield compatibility to be able to compile on master branch...

@Jarsop
Copy link
Contributor Author

Jarsop commented Nov 9, 2023

The PR #291 update our layer.conf if needed

@Jarsop
Copy link
Contributor Author

Jarsop commented Nov 10, 2023

This is no more relevant as I merged and adapted the content in #289. So I close it

@Jarsop Jarsop closed this Nov 10, 2023
@Jarsop Jarsop deleted the feat/simplify-varflag-usage branch November 10, 2023 20:03
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