patches for cgo on arm64

[Michael Hudson-Doyle]

Hi,

I am attaching 5 patches that together with https://bugs.launchpad.net/ubuntu/+source/gcc-4.9/+bug/1361940 enable cgo to work on arm64 with gccgo. The patches are all rather small and could probably be rolled up into fewer, but well.

Cheers,
mwh

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "0001-cmdcgo.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

[Michael Hudson-Doyle (mwhudson)]

that which is not tested is broken etc. here's a fixed version of the makefile

[James Page (james-page)]

Hi Michael

Does arm64 need to be added to the pointer size map in src/cmd/cgo/main.go?

[Michael Hudson-Doyle (mwhudson)]

Yes. How did I not include that patch, I wonder...

[Michael Hudson-Doyle (mwhudson)]

Here's that patch. Are you waiting on anything else? Would be nice to get these changes into utopic.

[Michael Hudson-Doyle (mwhudson)]

Ah, we need one more patch. This one is a bit of a hack but with all of these (as seen at the packages in https://launchpad.net/~mwhudson/+archive/ubuntu/gccgo-cgo/+packages) it does actually work (enough to build docker with some hacks to docker).

[Matthias Klose (doko)]

Michael, please check and submit these patches for gccgo-5 as well.

[Matthias Klose (doko)]

now fixed in gccgo-5 in vivid.

[Matthias Klose (doko)]

gccgo-go is removed in vivid, superseded by gccgo-5

[Michael Hudson-Doyle (mwhudson)]

Turns out these changes are required to do SRU verification for 1361940, so I've reapplied them and made a debdiff.

[Michael Hudson-Doyle (mwhudson)]

Better debdiff (better formatting, deleted two hunks that confusingly had no effect)

[Chris J Arges (arges)]

Before doing the SRU can you verify this works on Xenial?

[Matthias Klose (doko)]

no, not directly. gccgo-go isn't in the distro anymore, intgrated into gccgo-5. confirmed to work.

[Chris J Arges (arges)]

Hello Michael, or anyone else affected,

Accepted gccgo-go into trusty-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/gccgo-go/1.2.1-0ubuntu1.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

[Launchpad Janitor (janitor)]

This bug was fixed in the package gccgo-go - 1.2.1-0ubuntu1.2

---------------
gccgo-go (1.2.1-0ubuntu1.2) trusty-proposed; urgency=medium

  * SRU: #1361940.
  * Backport some changes from pre-release vivid (LP: #1361946):
    - Build and install the cgo tool
    - In the go tool, look for the cgo tool where it is installed (and do not
      attempt to rebuild it, ever) and enable cgo by default on ppc64le.
    - In the cgo tool itself, support arm64 and ppc64le.

 -- Michael Hudson-Doyle <email address hidden> Fri, 27 Nov 2015 09:39:19 +1300

[Chris J Arges (arges)]

The verification of the Stable Release Update for gccgo-go has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

[Robie Basak (racb)]

Claimed regression caused by this update in bug 1527020.

[Michael Hudson-Doyle (mwhudson)]

I think the regression will be fixed when gccgo from bug 1361940 migrates.