Minor "BAKE_ARCHITECTURE" vs. "BAKE_ARCH" inconsistency
heybdj opened this issue · 1 comments
heybdj commented
See:
Line 1063 in ef8b880
Bake's README lists "BAKE_ARCHITECTURE" as the environment variable used to specify processor architecture, but the line referenced above stores the initialized value in "BAKE_ARCH".
Would it be okay to submit a PR that corrects this? If you suspect that user code uses both "BAKE_ARCHITECTURE" and "BAKE_ARCH", we could maintain compatibility like this:
ut_setenv("BAKE_ARCH", UT_ARCH);
ut_setenv("BAKE_ARCHITECTURE", UT_ARCH);
...but it would be nice to simply replace "BAKE_ARCH" and have one definitive variable, if that sounds safe.
Cheers!
SanderMertens commented
Sounds good to me!