On Fri, Dec 8, 2017 at 12:43 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Jonathan Nieder <jrnieder@xxxxxxxxx> writes: >>> @@ -413,6 +414,7 @@ int cmd_version(int argc, const char **argv, const char *prefix) >>> >>> if (build_options) { >>> printf("sizeof-long: %d\n", (int)sizeof(long)); >>> + printf("machine: %s\n", build_platform); >> >> Can this use GIT_BUILD_PLATFORM directly instead of going via the indirection >> of a mutable static string? That is, something like >> >> printf("machine: %s\n", GIT_BUILD_PLATFORM); > > Good point. And if this is externally identified as "machine", > probably the macro should also use the same word, not "platform". > We can go either way, as long as we are consistent, though. In Autoconf parlance, this would be called "host architecture" (GIT_HOST_ARCH).