diff --git a/extra/go/0001-time-make-the-ParseInLocation-test-more-robust.patch b/extra/go/0001-time-make-the-ParseInLocation-test-more-robust.patch deleted file mode 100644 index 533c2d8dc..000000000 --- a/extra/go/0001-time-make-the-ParseInLocation-test-more-robust.patch +++ /dev/null @@ -1,93 +0,0 @@ -From 91563ced5897faf729a34be7081568efcfedda31 Mon Sep 17 00:00:00 2001 -From: Alberto Donizetti -Date: Thu, 9 Mar 2017 13:20:54 +0100 -Subject: [PATCH] time: make the ParseInLocation test more robust - -The tzdata 2017a update (2017-02-28) changed the abbreviation of the -Asia/Baghdad time zone (used in TestParseInLocation) from 'AST' to the -numeric '+03'. - -Update the test so that it skips the checks if we're using a recent -tzdata release. - -Fixes #19457 - -Change-Id: I45d705a5520743a611bdd194dc8f8d618679980c -Reviewed-on: https://go-review.googlesource.com/37964 -Reviewed-by: Ian Lance Taylor -Run-TryBot: Ian Lance Taylor -TryBot-Result: Gobot Gobot ---- - src/time/format_test.go | 41 +++++++++++++++++++++++++++++++---------- - 1 file changed, 31 insertions(+), 10 deletions(-) - -diff --git a/src/time/format_test.go b/src/time/format_test.go -index 219c2ca..d0013bc 100644 ---- a/src/time/format_test.go -+++ b/src/time/format_test.go -@@ -245,27 +245,45 @@ func TestParseDayOutOfRange(t *testing.T) { - } - } - -+// TestParseInLocation checks that the Parse and ParseInLocation -+// functions do not get confused by the fact that AST (Arabia Standard -+// Time) and AST (Atlantic Standard Time) are different time zones, -+// even though they have the same abbreviation. -+// -+// ICANN has been slowly phasing out invented abbreviation in favor of -+// numeric time zones (for example, the Asia/Baghdad time zone -+// abbreviation got changed from AST to +03 in the 2017a tzdata -+// release); but we still want to make sure that the time package does -+// not get confused on systems with slightly older tzdata packages. - func TestParseInLocation(t *testing.T) { -- // Check that Parse (and ParseInLocation) understand that -- // Feb 01 AST (Arabia Standard Time) and Feb 01 AST (Atlantic Standard Time) -- // are in different time zones even though both are called AST - - baghdad, err := LoadLocation("Asia/Baghdad") - if err != nil { - t.Fatal(err) - } - -- t1, err := ParseInLocation("Jan 02 2006 MST", "Feb 01 2013 AST", baghdad) -+ var t1, t2 Time -+ -+ t1, err = ParseInLocation("Jan 02 2006 MST", "Feb 01 2013 AST", baghdad) - if err != nil { - t.Fatal(err) - } -- t2 := Date(2013, February, 1, 00, 00, 00, 0, baghdad) -- if t1 != t2 { -- t.Fatalf("ParseInLocation(Feb 01 2013 AST, Baghdad) = %v, want %v", t1, t2) -- } -+ - _, offset := t1.Zone() -- if offset != 3*60*60 { -- t.Fatalf("ParseInLocation(Feb 01 2013 AST, Baghdad).Zone = _, %d, want _, %d", offset, 3*60*60) -+ -+ // A zero offset means that ParseInLocation did not recognize the -+ // 'AST' abbreviation as matching the current location (Baghdad, -+ // where we'd expect a +03 hrs offset); likely because we're using -+ // a recent tzdata release (2017a or newer). -+ // If it happens, skip the Baghdad test. -+ if offset != 0 { -+ t2 = Date(2013, February, 1, 00, 00, 00, 0, baghdad) -+ if t1 != t2 { -+ t.Fatalf("ParseInLocation(Feb 01 2013 AST, Baghdad) = %v, want %v", t1, t2) -+ } -+ if offset != 3*60*60 { -+ t.Fatalf("ParseInLocation(Feb 01 2013 AST, Baghdad).Zone = _, %d, want _, %d", offset, 3*60*60) -+ } - } - - blancSablon, err := LoadLocation("America/Blanc-Sablon") -@@ -273,6 +291,9 @@ func TestParseInLocation(t *testing.T) { - t.Fatal(err) - } - -+ // In this case 'AST' means 'Atlantic Standard Time', and we -+ // expect the abbreviation to correctly match the american -+ // location. - t1, err = ParseInLocation("Jan 02 2006 MST", "Feb 01 2013 AST", blancSablon) - if err != nil { - t.Fatal(err) diff --git a/extra/go/PKGBUILD b/extra/go/PKGBUILD index a070a7733..aa0533f28 100644 --- a/extra/go/PKGBUILD +++ b/extra/go/PKGBUILD @@ -7,19 +7,17 @@ pkgname=go epoch=2 -pkgver=1.8 -pkgrel=3 +pkgver=1.8.1 +pkgrel=1 pkgdesc='Core compiler tools for the Go programming language' arch=(i686 x86_64) url='http://golang.org/' license=(BSD) makedepends=(git go) options=(!strip staticlibs) -_commit=cd6b6202dd1559b3ac63179b45f1833fcfbe7eca # go1.8 -source=(git+https://go.googlesource.com/go#commit=$_commit - 0001-time-make-the-ParseInLocation-test-more-robust.patch) -md5sums=('SKIP' - 'a9031d2fdd5d0951980db7ffe20afe41') +_commit=a4c18f063b6659079ca2848ca217a0587dabc001 +source=(git+https://go.googlesource.com/go#commit=$_commit) +md5sums=('SKIP') export GOOS=linux case "$CARCH" in @@ -36,9 +34,6 @@ export GOROOT_BOOTSTRAP=/usr/lib/go prepare() { cd $pkgname - # fixes time/format test with tzdata >=2017a - patch -p1 -i "$srcdir/0001-time-make-the-ParseInLocation-test-more-robust.patch" - # upstream patch for AArch64 git cherry-pick -n 3557d546090c7fedd69562c88d20767397de835d }