Closed j-lanson closed 1 month ago
Thanks @j-lanson! I think this is part of what we need, and I'm happy to merge as-is. The part that's missing is expanding the SupportedArch
enum to reflect more architectures we may not auto-detect, but which we can allow users to specify. In my specific case that would be aarch64-unknown-linux-gnu
but there are certainly others we could reflect in the enum.
Resolves #415 .
We had previously encountered problems with cross-compilation to niche-but-valid architectures, where the compile-time target detection would fail, causing
CURRENT_ARCH
to be()
instead of a valid arch enum variant. We changed this field to be typeOption<SupportedArch>
and also support a user-provided--arch
flag to override the detected arch at runtime if the user has good reason to do so.