]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/dist: fix bad LC_TIME assumption in 'date' invocation
authorAlberto Donizetti <alb.donizetti@gmail.com>
Thu, 2 Nov 2017 09:22:19 +0000 (10:22 +0100)
committerRuss Cox <rsc@golang.org>
Fri, 3 Nov 2017 13:42:33 +0000 (13:42 +0000)
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 <alb.donizetti@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/make.bash

index e1c3d9261088cf751ede03fec7191c1f914577cc..93a5c43d11275e40ccf88829be6fd976bf913d0b 100755 (executable)
@@ -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.