-
Notifications
You must be signed in to change notification settings - Fork 76
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: support multi-base recipes #2054
Conversation
f5e1f1c
to
292d9fc
Compare
Signed-off-by: Callahan Kovacs <[email protected]>
Signed-off-by: Callahan Kovacs <[email protected]>
292d9fc
to
750daa8
Compare
Signed-off-by: Callahan Kovacs <[email protected]>
Signed-off-by: Callahan Kovacs <[email protected]>
Signed-off-by: Callahan Kovacs <[email protected]>
Update: it was a transient store-side failure |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! It would be nice to have a description of the overall platforms handling strategy in the PR description to help our confused future selves understand what's happening here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nicely done! Lots of moving parts here.
Adds [reference](https://canonical-charmcraft--2080.com.readthedocs.build/en/2080/reference/platforms/) and [how-to](https://canonical-charmcraft--2080.com.readthedocs.build/en/2080/howto/build-guides/platforms/) documentation for platforms, bases, and architectures in Charmcraft, including information on the new [multi-base support](#2054). Fixes #2009 (CRAFT-3720) --------- Signed-off-by: Callahan Kovacs <[email protected]> Co-authored-by: Alex Lowe <[email protected]>
Adds support for multi-base recipes with the
platforms
keyword.Internally, this moves to a new practice where the application stops modifying project data before passing the it to craft-platforms. This will ensure consistent processing by charmcraft and by launchpad.
Documentation is coming soon via canonical/craft-platforms#75 and #2009.
To summarize the usage,
base
andbuild-base
are removed from the project and the base is encoded into the platform.Platforms can be defined in a shorthand notation:
Or they can be defined in standard form:
Fixes #1789
Fixes #2008
Creates canonical/craft-platforms#85
Found during implementation canonical/craft-platforms#80
(CRAFT-3718)