SanderMertens/bake

Minor "BAKE_ARCHITECTURE" vs. "BAKE_ARCH" inconsistency

heybdj opened this issue · 1 comments

See:

ut_setenv("BAKE_ARCH", UT_ARCH);

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! ☺️

Sounds good to me!