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

Workspaces Part II #1161

Open
wants to merge 4 commits into
base: workspaces
Choose a base branch
from
Open

Workspaces Part II #1161

wants to merge 4 commits into from

Conversation

rydrman
Copy link
Collaborator

@rydrman rydrman commented Dec 18, 2024

Builds on #1160 as part of #1121

Makes use of the versions specified for any given template file by parsing the version from the request and only finding spec files that are appropriate. Uses the LeastSpecifiedVersionRange to allow partial versions to be given.

As a practical example, the workspace.spk.yaml from this repository lists versions for each python spec, and so you can run spk build python/3 to build the default version of the python 3 package spec.

This does not yet make use of the versions in any particular way, but
exposes the configuration for each template in the workspace API so that
it can at least be seen.

Signed-off-by: Ryan Bottriell <[email protected]>
Now that workspaces are loaded up-front and hold templates for a
reasonable lifetime in the CLI crates, simple references are enough

Signed-off-by: Ryan Bottriell <[email protected]>
Versions can be given for each template in the workspace that define
which versions can be built from that file. Then, when running commands
like 'spk build' we can further disambiguate possible errors by
specifying an exact version to build.

Signed-off-by: Ryan Bottriell <[email protected]>
This allows a better user experience by only requiring enough of a
version number to disambiguate spec files rather than an exact version
to build.

Signed-off-by: Ryan Bottriell <[email protected]>
@rydrman rydrman requested review from jrray and dcookspi December 18, 2024 21:35
@rydrman rydrman self-assigned this Dec 18, 2024
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.

1 participant