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

bundle.bbclass: Some Cleanups and Fixes #323

Merged
merged 4 commits into from
Jun 10, 2024
Merged

Conversation

ejoerns
Copy link
Member

@ejoerns ejoerns commented Jun 7, 2024

  • fix hooks example (addressed by Multiple hooks targets #321)
  • inherit nopackages
  • remove duplicate hyphen in bundle compatible
  • no need to set deploy cleandirs

Using install and post-install hook together will not work since RAUC
will ignore the post-install hook.
Thus better use a working combination for the example.

Signed-off-by: Enrico Jörns <[email protected]>
Instead of deleting some package tasks, setting some to
[noexec] and keeping some, use the nopackages class that contains the
full list of tasks to delete.

Signed-off-by: Enrico Jörns <[email protected]>
TARGET_VENDOR is already expected to contain a hyphen (defaults to
'-oe').

Signed-off-by: Enrico Jörns <[email protected]>
The deploy.bbclass already does that.

Signed-off-by: Enrico Jörns <[email protected]>
Copy link

@gbetous gbetous left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks !

@ejoerns ejoerns merged commit d2bb20a into rauc:master Jun 10, 2024
2 checks passed
@ejoerns ejoerns deleted the bundle-fixes branch July 3, 2024 13:17
@ejoerns
Copy link
Member Author

ejoerns commented Jul 3, 2024

All (except dbade34) backported to scarthgap

@ejoerns
Copy link
Member Author

ejoerns commented Aug 14, 2024

All (except dbade34) backported to kirkstone

@ejoerns ejoerns added kirkstone kirkstone release-related and removed backport candidate labels Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants