Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 1360388 - tests fail in currency_test.go:155
Summary: tests fail in currency_test.go:155
Alias: None
Product: Fedora
Classification: Fedora
Component: golang-googlecode-text
Version: rawhide
Hardware: s390x
OS: Unspecified
Target Milestone: ---
Assignee: Fridolín Pokorný
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: ZedoraTracker
TreeView+ depends on / blocked
Reported: 2016-07-26 14:52 UTC by Dan Horák
Modified: 2016-07-27 15:33 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed: 2016-07-27 15:33:07 UTC

Attachments (Terms of Use)

Description Dan Horák 2016-07-26 14:52:39 UTC
The build of golang-googlecode-text package fails in the test phase with

Executing(%check): /bin/sh -e /var/tmp/rpm-tmp.vjXA3A
+ umask 022
+ cd /builddir/build/BUILD
+ cd text-6fc2e00a0d64b1f7fc1212dae5b0c939cf6d9ac4
+ export GOPATH=/builddir/build/BUILDROOT/golang-googlecode-text-0-0.12.git6fc2e00.fc25.s390x//usr/share/gocode:/usr/share/gocode
+ GOPATH=/builddir/build/BUILDROOT/golang-googlecode-text-0-0.12.git6fc2e00.fc25.s390x//usr/share/gocode:/usr/share/gocode
+ go test -compiler gc -ldflags ''
ok	0.012s
+ go test -compiler gc -ldflags ''
ok	0.004s
+ go test -compiler gc -ldflags ''
ok	1.260s
+ go test -compiler gc -ldflags ''
--- FAIL: TestLinking (1.48s)
	currency_test.go:155: size(symbols)-size(base) was 902; want > 2K
FAIL	1.482s
error: Bad exit status from /var/tmp/rpm-tmp.vjXA3A (%check)
    Bad exit status from /var/tmp/rpm-tmp.vjXA3A (%check)

for full logs please see

This build is using golang.

Version-Release number of selected component (if applicable):

Comment 1 Jan Chaloupka 2016-07-26 15:41:55 UTC
Unless, it is a blocker, it does not get fixed. There are other go projects with its own set of failing tests. It would be more beneficial to fix more failing tests at once.

Keeping this bug opened but not guarantee any progress on it.

Comment 2 Jan Chaloupka 2016-07-26 15:42:42 UTC
Meaning, it is better to comment out the test rather than fix it.

Comment 3 Dan Horák 2016-07-27 08:22:50 UTC
I mean the question should be what is more likely wrong - golang-googlecode-text itself, the test case or even the compiler?

Comment 4 Jan Chaloupka 2016-07-27 11:05:24 UTC
The TestLinking compares size of two compiled binaries. First binary comes from this source code (e.g. as main1.go):

package main
import (
func main() {

The second one from (e.g. as main2.go):

package main
import (
func main() {

Compiling the code on x86_64 with golang-1.6.2-1.fc25.x86_64 I have:
main1: 2971752
main2: 2992416

Evaluated condition from currency_test.go:154 that must hold:
size(main2) - size(main1) >= 2*1024

For x86_64 case: 2992416 - 2971752 = 20664 > 2048

Comment 5 Jan Chaloupka 2016-07-27 11:32:38 UTC
Compiling the code on s390x with golang-1.7-0.rc1.fc25.s390x I have:
main1: 2391609
main2: 2392511

The difference:
2392511 - 2391609 = 902 < 2048

Answering your question, the test case is not applicable on s390x.

Comment 7 Dan Horák 2016-07-27 12:47:23 UTC
s390 f25 build =

Note You need to log in before you can comment on or make changes to this bug.