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

nominated zig #134

Open
mangas opened this issue May 21, 2024 · 2 comments · May be fixed by #145
Open

nominated zig #134

mangas opened this issue May 21, 2024 · 2 comments · May be fixed by #145

Comments

@mangas
Copy link

mangas commented May 21, 2024

Seems fairly common that projects follow the mach nominated zig versions, I wonder if it could be interesting to add both a stable and nominated versions which could translate to latest stable and nominated respectively.

Could zigup also provide a zls binary for both these versions? (Optionally via cli args)

@marler8997
Copy link
Owner

Yeah I think having some sort of nominated zig version integration could be interesting. Not sure exactly what that looks like though?

@dweiller
Copy link
Contributor

dweiller commented Jul 18, 2024

Yeah I think having some sort of nominated zig version integration could be interesting. Not sure exactly what that looks like though?

Mach provides a modified index.json at https://machengine.org/zig/index.json that is a superset (though updated less often) of the Zig project's index.json, it is described here.

I think it will be relatively simple to support Mach's nominated versions by simply checking both the upstream and Mach index.json file when looking for a compiler version. The only small complication I foresee is if we want to deduplicate zig versions with the same name as the Mach index.json contains entries both named for the nominated version and the Mach version, with the latter being an alias for a nominated version, as well as a mach-latest which is an alias for the latest nominated version.

@dweiller dweiller linked a pull request Jul 23, 2024 that will close this issue
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