Open cdaguerre opened 2 years ago
hi, this was very informative and helpful. when can we expect this to get merged?
@BrianMusson this is already merged, you can use it like this:
TFENV_ARCH=arm64 tfenv install 1.0.2
Note that 1.0.2 is the first release that was compiled for macOS arm64. Earlier versions will fail with a 403 error.
@danielcompton yea, that works, but it would be nice if install
would automatically know which arch to use :)
Though it seems that that's already the case as of https://github.com/tfutils/tfenv/commit/d22ffb421ec8e6263fe0dd5ba705bd2cc386960e and https://github.com/tfutils/tfenv/commit/abd0f46ca658dc81ec4f253f4883c7b942f8f1dd, but those commits weren't released (yet); so I guess all we have to do is to wait for next release to happen (and set local TFENV_ARCH
env variable in the time being, as you suggest).
Just ran into this, and following the README.md I used arm
at first, until I realized it needs to be arm64
. I just added:
export TFENV_ARCH=arm64
Into my ~/.zshrc
It would be nice if 'tfenv' either didn't show anything older than 1.0.2 when used on Mac arm64 or just displayed a warning about this.
Or better yet, would fetch and compile older versions upon request when it hits a 404 on the precompiled binary.
In case someone else wonders,
tfenv
does support ARM architecture on macOS when setting aTFENV_ARCH
env var toarm64
. As @kamatama41 said in the MR that introduced this env var, it should probably be resolved usinguname -m
rather than defaulting toamd64