From: Alberto Donizetti Date: Thu, 2 Nov 2017 09:22:19 +0000 (+0100) Subject: cmd/dist: fix bad LC_TIME assumption in 'date' invocation X-Git-Tag: go1.10beta1~427 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=6de53832ae7678f18e1b4077134764b11687992d;p=gostls13.git cmd/dist: fix bad LC_TIME assumption in 'date' invocation With GOBUILDTIMELOGFILE set, make.bash logs the starting time using $ echo $(date) > file and expects to be able to read the date back with time.Parse(time.UnixDate) but in some locales the default date format is not the same as time.UnixDate; for example on LC_TIME="en_GB.UTF-8" $ locale date_fmt %a %e %b %H:%M:%S %Z %Y Fix this by setting LC_TIME=C before the date command invocation. Fixes #22541 Change-Id: I59bf944bb868e2acdd816c7e35134780cdbfc6a6 Reviewed-on: https://go-review.googlesource.com/75370 Run-TryBot: Alberto Donizetti TryBot-Result: Gobot Gobot Reviewed-by: Russ Cox Reviewed-by: Brad Fitzpatrick --- diff --git a/src/make.bash b/src/make.bash index e1c3d92610..93a5c43d11 100755 --- a/src/make.bash +++ b/src/make.bash @@ -70,7 +70,7 @@ if [ ! -f run.bash ]; then fi if [ "$GOBUILDTIMELOGFILE" != "" ]; then - echo $(date) start make.bash >"$GOBUILDTIMELOGFILE" + echo $(LC_TIME=C date) start make.bash >"$GOBUILDTIMELOGFILE" fi # Test for Windows.