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

Add namespace fallback registry config #36

Closed
lann opened this issue Jun 12, 2024 · 4 comments · Fixed by #41
Closed

Add namespace fallback registry config #36

lann opened this issue Jun 12, 2024 · 4 comments · Fixed by #41

Comments

@lann
Copy link
Collaborator

lann commented Jun 12, 2024

In order to support a more natural registry resolution algorithm for package dependencies, add new configuration for "namespace fallback registries". The resolution algorithm would look something like:

For a parent package fetched from registry R, pick the first available:

Question: Should packages published by Registry R itself be considered before fallbacks?

@calvinrp
Copy link
Collaborator

Probably better that the namespace mappings were explicitly specified in the registry metadata well-known. Instead of attempting to fetch other non-specified namespaces from the OCI registry and hope that they are correct if available.

@lukewagner
Copy link

For the first bullet (local registry mapping), I could imagine it being useful to have a sub-option that lets specify that the local registry is just an overlay, in the sense that: if a package-name is present in the local registry, use that; otherwise fall back. This seems useful if, e.g., you're developing package ba:a and you need to patch a dependency ba:b, so you specify a local overlay for ba containing b, but everything else falls back to bytecodealliance.org.

@lann
Copy link
Collaborator Author

lann commented Jun 13, 2024

specify a local overlay for ba containing b, but everything else falls back to bytecodealliance.org

The config does already have "package overrides" which I think can serve this purpose.

@lukewagner
Copy link

Nice!

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 a pull request may close this issue.

3 participants