I think the bot could be taught to at least compile all the tests under gccgo (with go test -compiler=gcc -c). That would at least let us detect compile failures.
John =:-> On Fri, Aug 29, 2014 at 2:45 PM, David Cheney <[email protected]> wrote: > nah, we have a fix upstream, we just need to get that backported to > trusty then this becomes a non issue. > > On Fri, Aug 29, 2014 at 7:44 PM, Matthew Williams > <[email protected]> wrote: > > As it's something we need to be doing for a while yet is there value in > > adding this as a task that gets run by the landing bot? > > > > Thanks > > > > Matty > > > > > > On Thu, Aug 28, 2014 at 11:48 PM, Tim Penhey <[email protected]> > > wrote: > >> > >> Hi folks, > >> > >> I spent some time this morning looking at > >> https://bugs.launchpad.net/juju-core/+bug/1362636 > >> > >> A critical regression that was breaking CI on power. > >> > >> There is a bug in gccgo where we hit an internal compiler error when > >> comparing an interface to a concrete type that implements the interface > >> (as opposed to a pointer to the concrete type implementing the > interface). > >> > >> This impacts some of the names.Tag rework that is going on. > >> > >> If you try to compare: > >> var tag names.Tag = names.NewMachineTag("1") > >> > >> if names.NewUnitTag("1") == tag { > >> // BOOM!!! > >> } > >> > >> This is entirely valid Go, and works fine with gc, but gccgo barfs > >> horribly. > >> > >> My fix is here: https://github.com/juju/juju/pull/633 > >> > >> This is just a warning. > >> > >> Remember folks that we need to support gccgo still (for at least another > >> year until we have power and arm64 using gc). > >> > >> You can test locally by doing this: > >> go test -compiler gccgo > >> > >> If you install the gccgo packages, which I don't remember, but hopefully > >> someone will follow up with. > >> > >> Cheers, > >> Tim > >> > >> -- > >> Juju-dev mailing list > >> [email protected] > >> Modify settings or unsubscribe at: > >> https://lists.ubuntu.com/mailman/listinfo/juju-dev > > > > > > > > -- > > Juju-dev mailing list > > [email protected] > > Modify settings or unsubscribe at: > > https://lists.ubuntu.com/mailman/listinfo/juju-dev > > > > -- > Juju-dev mailing list > [email protected] > Modify settings or unsubscribe at: > https://lists.ubuntu.com/mailman/listinfo/juju-dev >
-- Juju-dev mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/juju-dev
