Show symlinks to compilers in zigup list#144
Show symlinks to compilers in zigup list#144dweiller wants to merge 1 commit intomarler8997:masterfrom
zigup list#144Conversation
ca2067d to
beebdb4
Compare
|
I'm not sure exactly how to get into this state that's newly being handled here but it seems like a reasonable feature, would you mind adding a test for this in P.S. This might be helpful and/or a way to implement #135 ? |
|
To help with an implementation of #135, the logic would want to be a bit different than what is currently implemented. At the moment I have (conservatively) chosen to not list links that resolve to paths outside the installation directory. For #135 (and maybe in general) we would want to list any symlink to a directory. This is simpler to do than the current implementation and allows for more use cases like #135 or linking to a local build of zig.
Will do. |
beebdb4 to
36f9b11
Compare
|
I've simplified the logic as mentioned above. It looks like windows is handling the symlinks in an odd way (or at least unexpected to me) that will need some investigation. |
|
Looks like windows is failing, I think there's a way to test if symlinks are supported you could use to disable the test in that case? |
This PR makes
zigup listinclude symlinks to compilers in the install directory display in the list of installed compilers. This is useful as it is possible to use these symlinks as theCOMPILER_NAMEinzigup default COMPILER_NAME. Currently the only symlink created by zigup is one formaster, but this PR would also support a user who wants to make custom symlinks to name particular compilers.This feature will be particularly helpful if/when mach nominated versions are supported (see #145), as all of the nominated versions should be referenced via a link.