marler8997 / zigup

Download and manage zig compilers.
MIT No Attribution
793 stars 61 forks source link

Support for ZIGUP_INSTALL_DIR and ZIGUP_PATH_LINK #92

Closed yohannd1 closed 1 year ago

yohannd1 commented 1 year ago

Hello! Here it is, as I mentioned yesterday in https://github.com/marler8997/zigup/issues/91 I wouldn't say this is fully done, though. Currently it doesn't support windows, since I'm using std.os.getenv, but on that note I'd like to ask: should I conditionally use std.os.getenv and std.process.getEnvVarOwned conditionally or just the latter? Using the owned variant would have a performance penalty on non-windows systems but I don't think it's too much to worry about, and it might make the code simpler.

yohannd1 commented 1 year ago

Cancelled (see #91 for rationale)