launchpad.net/juju-core/environs/imagemetadata fails with gccgo

Bug #1252122 reported by Michael Hudson-Doyle
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-core
Fix Released
High
Ian Booth

Bug Description

Like so:

$ go test -compiler gccgo launchpad.net/juju-core/environs/imagemetadata
...
simplestreams_test.go:275:
    c.Check(images, gc.DeepEquals, t.images)
... obtained []*imagemetadata.ImageMetadata = []*imagemetadata.ImageMetadata{(*imagemetadata.ImageMetadata)(0xc21008ef80), (*imagemetadata.ImageMetadata)(0xc21008ed80), (*imagemetadata.ImageMetadata)(0xc21008ea80), (*imagemetadata.ImageMetadata)(0xc213eafc00), (*imagemetadata.ImageMetadata)(0xc213eafb80)}
... expected []*imagemetadata.ImageMetadata = []*imagemetadata.ImageMetadata{(*imagemetadata.ImageMetadata)(0xc213eaf880), (*imagemetadata.ImageMetadata)(0xc213eaf900), (*imagemetadata.ImageMetadata)(0xc213eaf980), (*imagemetadata.ImageMetadata)(0xc213eafa00), (*imagemetadata.ImageMetadata)(0xc213eafa80)}

instinct says this is hash-order dependent

(if you get link errors you should try again (it's erratic) or apply the patch from https://codereview.appspot.com/28050043/ to your go tool)

Related branches

Curtis Hovey (sinzui)
tags: added: gccgo test-failure
Changed in juju-core:
status: New → Triaged
importance: Undecided → High
Revision history for this message
Dave Cheney (dave-cheney) wrote :

This test won't even compile under ppc64el

tags: added: ppc64el
Revision history for this message
Dave Cheney (dave-cheney) wrote :
Download full text (3.6 KiB)

$ go test launchpad.net/juju-core/environs/imagemetadata

...

/tmp/go-build496951398/launchpad.net/juju-core/environs/imagemetadata/_test/launchpad.net/juju-
core/environs/libimagemetadata.a(imagemetadata.o):(.rodata.launchpad.net_juju_core_environs_imagemetadata.ImageMetadataURL$descriptor+0x0): multiple definition of `launchpad.net_juju_core_environs_imagemetadata.ImageMetadataURL$descriptor'
/tmp/go-build496951398/launchpad.net/juju-core/environs/libimagemetadata.a(imagemetadata.o):(.rodata.launchpad.net_juju_core_environs_imagemetadata.ImageMetadataURL$descriptor+0x0): first defined here
/tmp/go-build496951398/launchpad.net/juju-core/environs/imagemetadata/_test/launchpad.net/juju-core/environs/libimagemetadata.a(imagemetadata.o):(.rodata.launchpad.net_juju_core_environs_imagemetadata.ValidateImageMetadata$descriptor+0x0): multiple definition of `launchpad.net_juju_core_environs_imagemetadata.ValidateImageMetadata$descriptor'
/tmp/go-build496951398/launchpad.net/juju-core/environs/libimagemetadata.a(imagemetadata.o):(.rodata.launchpad.net_juju_core_environs_imagemetadata.ValidateImageMetadata$descriptor+0x0): first defined here
/tmp/go-build496951398/launchpad.net/juju-core/environs/imagemetadata/_test/launchpad.net/juju-core/environs/libimagemetadata.a(imagemetadata.o): In function `launchpad.net_juju_core_environs_imagemetadata.ValidateImageMetadata':
/home/ubuntu/src/launchpad.net/juju-core/environs/imagemetadata/validation.go:14: multiple definition of `launchpad.net_juju_core_environs_imagemetadata.ValidateImageMetadata'
/tmp/go-build496951398/launchpad.net/juju-core/environs/libimagemetadata.a(imagemetadata.o):/home/ubuntu/src/launchpad.net/juju-core/environs/imagemetadata/validation.go:14: first defined here
/tmp/go-build496951398/launchpad.net/juju-core/environs/imagemetadata/_test/launchpad.net/juju-core/environs/libimagemetadata.a(imagemetadata.o): In function `launchpad.net_juju_core_environs_imagemetadata..import':
/home/ubuntu/src/launchpad.net/juju-core/environs/imagemetadata/generate.go:4: multiple definition of `launchpad.net_juju_core_environs_imagemetadata..import'
/tmp/go-build496951398/launchpad.net/juju-core/environs/libimagemetadata.a(imagemetadata.o):/home/ubuntu/src/launchpad.net/juju-core/environs/imagemetadata/generate.go:4: first defined here
collect2: error: ld returned 1 exit status
FAIL launchpad.net/juju-core/environs/imagemetadata [build failed]

$ go version
go version xgcc (Ubuntu 4.9-20140222-0ubuntu1) 4.9.0 20140222 (experimental) [trunk revision 208043] linux/ppc64

$ gccgo -v
Using built-in specs.
COLLECT_GCC=gccgo
COLLECT_LTO_WRAPPER=/usr/lib/gcc/powerpc64le-linux-gnu/4.9/lto-wrapper
Target: powerpc64le-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 4.9-20140222-0ubuntu1' --with-bugurl=file:///usr/share/doc/gccgo-4.9/README.Bugs --enable-languages=c,c++,go --prefix=/usr --program-suffix=-4.9 --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --with-gxx-include-dir=/usr/include/c++/4.9 --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-time=yes --enable-gnu-unique-object --d...

Read more...

Ian Booth (wallyworld)
Changed in juju-core:
assignee: nobody → Ian Booth (wallyworld)
status: Triaged → In Progress
Ian Booth (wallyworld)
Changed in juju-core:
milestone: none → 1.17.5
Ian Booth (wallyworld)
Changed in juju-core:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.