1.0.2-1ubuntu1~14.04.1 in trusty-proposed needs to depend on gccgo-go not gccgo

Bug #1510288 reported by Michael Hudson-Doyle
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
go-md2man (Ubuntu)
Confirmed
Undecided
dann frazier
Trusty
In Progress
High
dann frazier

Bug Description

[Impact]
This package does not currently build on architectures that do not support golang, but do have gccgo support. Since this package is a build-dependency for docker.io, docker.io is also not available.

[Test Case]
Build docker.io in trusty on an impacted architecture (I'm using arm64).

[Regression Risk]
The changes to enable gccgo are restricted to architectures that currently do not build.

Tags: patch
Revision history for this message
Robie Basak (racb) wrote :

OK, but this is a new package in Trusty and docker.io in Trusty has never built against arm64, powerpc or ppc64el. So this isn't an SRU regression.

I guess we could fix this in the backport, although it's a bit awkward since we're using a script to generate the backports and I'd prefer to not have to special case this one. But does it gain anything if we're not aiming to enable extra architectures for Docker in Trusty anyway?

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

You're right this is not a regression. I didn't know if we are *trying* to fix the failure to build of docker on arm64 etc though -- if we are, this should be fixed (and it's possible that fixing this and getting finally getting bug 1361940 SRUed is enough, although I haven't verified that). If not, then sure, feel free to leave this.

Revision history for this message
dann frazier (dannf) wrote :

Though I agree this is not a regression, it does fall under hardware enablement IMO. The attached patch resolves this by allowing gccgo-go fulfill the build-dep if gccgo > 5 is not available.

Could we avoid future special casing by adding the same to wily iff it is ever updated? If so - maybe we could add a reminder in your backport script so it isn't forgotten.

Changed in go-md2man (Ubuntu):
status: New → Confirmed
tags: added: patch
Revision history for this message
dann frazier (dannf) wrote :

We also need to disable stripping of the resulting gccgo binaries. Updated patch attached.

dann frazier (dannf)
Changed in go-md2man (Ubuntu):
assignee: nobody → dann frazier (dannf)
Changed in go-md2man (Ubuntu Trusty):
status: New → Confirmed
assignee: nobody → dann frazier (dannf)
importance: Undecided → High
dann frazier (dannf)
description: updated
dann frazier (dannf)
Changed in go-md2man (Ubuntu Trusty):
status: Confirmed → In Progress
Revision history for this message
dann frazier (dannf) wrote :

I prepared a docker PPA with this update and asked our QA team to run some tests on it. Even though the binary builds and runs, we're still seeing docker crashes based on the older version of gccgo in trusty, so it still doesn't make sense to enable docker for gccgo in trusty at this time.

This upload is pending in the SRU queue for trusty but, since docker.io is the only package that build-depends upon go-md2man, I'll ask an SRU team member to reject.

Revision history for this message
Chris J Arges (arges) wrote :

To confirm you want go-md2man (1.0.2-1ubuntu1~14.04.2) in trusty rejected?

Revision history for this message
dann frazier (dannf) wrote : Re: [Bug 1510288] Re: 1.0.2-1ubuntu1~14.04.1 in trusty-proposed needs to depend on gccgo-go not gccgo

On Wed, Mar 23, 2016 at 6:02 AM, Chris J Arges
<email address hidden> wrote:
> To confirm you want go-md2man (1.0.2-1ubuntu1~14.04.2) in trusty
> rejected?

Yes, please and thank you! :)

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.