Skip to content

Unable to test go 1.6.4 with 1.6.3 on Mageia 5 (trying to package it) #18491

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
bcornec opened this issue Jan 2, 2017 · 5 comments
Closed

Unable to test go 1.6.4 with 1.6.3 on Mageia 5 (trying to package it) #18491

bcornec opened this issue Jan 2, 2017 · 5 comments

Comments

@bcornec
Copy link

bcornec commented Jan 2, 2017

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

go version go1.6.3 linux/amd64

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH=""
GORACE=""
GOROOT="/usr/lib/golang"
GOTOOLDIR="/usr/lib/golang/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?

I'm trying to package the latest go 1.6.4 for Mageia (uptading our current 1.6.3)
I ran an rpmbuild which triggered the following in the %check section (after the build succeed):

  • umask 022
  • cd /users-ssd/bruno/prj/mageia/golang/BUILD
  • cd go
  • '[' 1 -eq 1 ']'
    ++ pwd -P
  • export GOROOT=/users-ssd/bruno/prj/mageia/golang/BUILD/go
  • GOROOT=/users-ssd/bruno/prj/mageia/golang/BUILD/go
  • export PATH=/users-ssd/bruno/prj/mageia/golang/BUILD/go/bin:/usr/local/bin:/usr/bin:/usr/local/games:/usr/games:/usr/lib64/qt4/bin:/usr/lib64/qt5/bin:/usr/lib64/qt5/bin
  • PATH=/users-ssd/bruno/prj/mageia/golang/BUILD/go/bin:/usr/local/bin:/usr/bin:/usr/local/games:/usr/games:/usr/lib64/qt4/bin:/usr/lib64/qt5/bin:/usr/lib64/qt5/bin
  • cd src
  • export CC=gcc
  • CC=gcc
  • export 'CFLAGS=-O2 -g -pipe -Wformat -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fstack-protector --param=ssp-buffer-size=4 -fPIC'
  • CFLAGS='-O2 -g -pipe -Wformat -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fstack-protector --param=ssp-buffer-size=4 -fPIC'
  • export LDFLAGS=
  • LDFLAGS=
  • export GO_TEST_TIMEOUT_SCALE=2
  • GO_TEST_TIMEOUT_SCALE=2
  • ./run.bash --no-rebuild -v -v -v -k
Testing packages.

go tool dist test -run=^go_test:archive/tar$

ok archive/tar 0.029s
ok archive/zip 0.144s
ok bufio 0.120s
[...]
ok text/template 0.044s
ok text/template/parse 0.009s
--- FAIL: TestLoadFixed (0.00s)
time_test.go:943: Now().In(loc).Zone() = "-01", -3600, want "GMT+1", -3600
FAIL
FAIL time 2.735s
ok unicode 0.041s

I can reproduce this by doing:
$ go test time
--- FAIL: TestLoadFixed (0.00s)
time_test.go:943: Now().In(loc).Zone() = "-01", -3600, want "GMT+1", -3600
FAIL
FAIL time 7.561s

@quentinmit
Copy link
Contributor

Go 1.6 is no longer supported. The tests fail when run with a current timezone database. Either use a newer Go or don't run the tests.

@ALTree
Copy link
Member

ALTree commented Jan 2, 2017

@bcornec if for some reason you can't package a newer version, and you want to be able to run the tests, you can cherrypick the patch for this. Ref is c5434f2

@bcornec
Copy link
Author

bcornec commented Jan 2, 2017

Thanks Alberto. Will see whether we should update to 1.7.4 instead.
The update is necessary for a security wrt 1.6.3 that is shiped with Mageia 5 anyway so your git id is very handy !

@ALTree
Copy link
Member

ALTree commented Jan 2, 2017

That test will fail in 1.7.4 too, unfortunately. You'll need go1.8 (scheduled for February 1, 2017).

The fix was pushed after the go1.7 release, and it wasn't cherry-picked for 1.7.3 or 1.7.4. People has been complaining about this (see this comment). Then #18163 was filed, proposing we allow test-only changes to be included in minor releases. It was accepted, and in the future this will hopefully make life a little easier for Go package maintainers when a test-only problem arises.

@bcornec
Copy link
Author

bcornec commented Jan 2, 2017

I've been able to package 1.6.4 + your patch for mageia 5 as an update.
I'll probably provide that one for mga6, while I'm working on packaging 1.7.4 (which requires modification of our patches to build). Thanks a lot for your help.

@golang golang locked and limited conversation to collaborators Jan 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants