Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#52041 closed defect (fixed)

go-1.4 @1.4.3_0: fatal error: MSpanList_Insert

Reported by: kode54 (Christopher Snowhill) Owned by: ci42
Priority: Normal Milestone:
Component: ports Version: 2.3.4
Keywords: sierra haspatch Cc: mkae (Marko Käning), e-t-h-a-n, ryandesign (Ryan Carsten Schmidt), breiter (Brian Reiter), elventear (Pepe Barbe), eric@…, macports@…, ethan@…, andre.david@…, martijnverburg@…
Port: go-1.4

Description

Attempting to build go-1.4 fails during the testing phase, both with the current release build of Xcode, and with Xcode 8 beta 6.

Attachments (6)

main.log (10.3 KB) - added by kode54 (Christopher Snowhill) 8 years ago.
go-1.4 build log
main.2.log (13.1 KB) - added by kode54 (Christopher Snowhill) 8 years ago.
Clean log of go-1.4
Portfile.diff (320 bytes) - added by e-t-h-a-n 8 years ago.
Patch the Portfile to use the patch.
sierra-clock_gettime.patch (1.1 KB) - added by e-t-h-a-n 8 years ago.
Patch for go-1.4.3 on Sierra
Portfile-1.7.1.patch (1.2 KB) - added by lbschenkel (Leonardo Brondani Schenkel) 8 years ago.
go 1.7.1 with self-bootstrap
sierra-clock_gettime copy.patch (1.4 KB) - added by e-t-h-a-n 8 years ago.

Download all attachments as: .zip

Change History (37)

Changed 8 years ago by kode54 (Christopher Snowhill)

Attachment: main.log added

go-1.4 build log

comment:1 Changed 8 years ago by larryv (Lawrence Velázquez)

Keywords: sierra added
Owner: changed from macports-tickets@… to ciserlohn@…
Port: go-1.4 added
Summary: Building go-1.4 fails on Sierrago-1.4: build fails on Sierra

That log isn’t clean. Could you run sudo port clean go-1.4 and try installing again, and attach the new log if it fails.

Changed 8 years ago by kode54 (Christopher Snowhill)

Attachment: main.2.log added

Clean log of go-1.4

comment:2 Changed 8 years ago by kode54 (Christopher Snowhill)

Still fails, so I attached a clean build log.

comment:3 Changed 8 years ago by larryv (Lawrence Velázquez)

Summary: go-1.4: build fails on Sierrago-1.4 @1.4.3_0: build fails on Sierra

comment:4 Changed 8 years ago by mkae (Marko Käning)

Cc: mk@… added

Cc Me!

comment:5 Changed 8 years ago by e-t-h-a-n

Cc: ethansherriff@… added

Cc Me!

comment:6 Changed 8 years ago by e-t-h-a-n

Okay, so since 10.12, there's now clock_gettime in the C Standard Library, which broke Go. A commit was made to their master branch (https://github.com/golang/go/commit/2da5633eb9091608047881953f75b489a3134cdc) in response to this issue (https://github.com/golang/go/issues/16570). I have backported the fix to go-1.4.3 and created sierra-clock_gettime.patch.

Changed 8 years ago by e-t-h-a-n

Attachment: Portfile.diff added

Patch the Portfile to use the patch.

Changed 8 years ago by e-t-h-a-n

Attachment: sierra-clock_gettime.patch added

Patch for go-1.4.3 on Sierra

comment:7 Changed 8 years ago by e-t-h-a-n

Working for me on 10.12 Public Beta 7 (16A313a).

comment:8 Changed 8 years ago by lbschenkel (Leonardo Brondani Schenkel)

I fixed it in my local environment in a different way: I upgraded Go to 1.7.1 and bootstrapped it with itself (although that works with 1.7.0 was well). Go 1.4 is officially used by upstream to bootstrap Go since they want to prevent newer versions depending on Go features introduced after 1.4, however that is less important for downstream users — and this has the advantage of not requiring download/building Go 1.4 which results in faster builds. This was the same approach taken by Homebrew as well, since the fix for a change of gettimeofday(2) introduced in OS X 10.12 is not backported to 1.4 yet. I have attached my updated Portfile.

Changed 8 years ago by lbschenkel (Leonardo Brondani Schenkel)

Attachment: Portfile-1.7.1.patch added

go 1.7.1 with self-bootstrap

comment:9 Changed 8 years ago by lbschenkel (Leonardo Brondani Schenkel)

Disregard my patch from now; it works if you had Go 1.7 previously built (I had it from before the Sierra upgrade) however it does not work in a clean environment. Once I make it self-bootstrap I will update the patch.

comment:10 Changed 8 years ago by lbschenkel (Leonardo Brondani Schenkel)

Please disregard Portfile-1.7.1.patch. I misunderstood how Go bootstrapping mechanism works and I assumed it could self-bootstrap but it can't — you need 1.4 (Homebrew downloads a pre-built binary to bootstrap 1.7 and deletes it aftwerwards.)

I tried the other patch and it works for me on macOS Sierra GM + XCode 8 GM.

Last edited 8 years ago by lbschenkel (Leonardo Brondani Schenkel) (previous) (diff)

comment:11 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added
Keywords: haspatch added

It looks like your proposed patch includes all of the changes from https://github.com/golang/go/commit/2da5633eb9091608047881953f75b489a3134cdc and also includes the sys_darwin_386.s changes from https://github.com/golang/go/commit/fad2bbdc6a686a20174d2e73cf78f1659722bb39 but not the sys_darwin_amd64.s changes. Don't we need the sys_darwin_amd64.s changes as well?

Your portfile patch is not correct because >= is used to compare numbers but 16.0.0 is a string. We want to apply the patch unconditionally anyway.

comment:12 Changed 8 years ago by e-t-h-a-n

The change's I made for sys_darwin_386.s were purely to make the systime and inreg routine match those upstream. I didn't see inreg in GitHub's changelog for sys_darwin_amd64.s for 2da5633 and missed those out. Thank you for pointing that out, I will update the (portfile) patch shortly.

comment:13 Changed 8 years ago by e-t-h-a-n

Also, if any Portfile wizards are reading this (unlike me, :P) could we have go build itself against go-1.4 (the last version that can be built with a C compiler) if there isn't a go already installed?

Changed 8 years ago by e-t-h-a-n

comment:14 Changed 8 years ago by e-t-h-a-n

Sorry, I added the patch with the wrong name. Please see the attachment "sierra-clock_gettime copy.patch".

comment:15 in reply to:  13 ; Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to ethansherriff@…:

could we have go build itself against go-1.4 (the last version that can be built with a C compiler) if there isn't a go already installed?

Doesn't it already do that? It has a runtime dependency on go-1.4, so if go-1.4 is not yet installed, it will be installed first. (Granted, it should be a library dependency, not a runtime dependency, since it is also needed at build time, not just at runtime).

comment:16 Changed 8 years ago by breiter (Brian Reiter)

Cc: brian.reiter@… added

Cc Me!

comment:17 Changed 8 years ago by elventear (Pepe Barbe)

Cc: elventear@… added

Cc Me!

comment:18 Changed 8 years ago by eric@…

Cc: eric@… added

Cc Me!

comment:19 Changed 8 years ago by macports@…

Cc: macports@… added

Cc Me!

comment:20 Changed 8 years ago by carlos.abalde@…

Cc Me!

comment:21 in reply to:  20 Changed 8 years ago by larryv (Lawrence Velázquez)

Cc: carlos.abalde@… added

You have to actually click the “CcMe!” button near the top of the ticket.

comment:22 Changed 8 years ago by carlos.abalde@…

Cc: carlos.abalde@… removed

Cc Me!

comment:23 Changed 8 years ago by ethan@…

Cc: ethan@… added

Cc Me!

comment:24 Changed 8 years ago by andre.david@…

Cc: andre.david@… added

Cc Me!

comment:25 Changed 8 years ago by andre.david@…

Please note that this is blocking docker-machine and docker that both depend on go that, in turn, depends on go-1.4.

comment:26 in reply to:  15 ; Changed 8 years ago by e-t-h-a-n

Replying to ryandesign@…:

Doesn't it already do that? It has a runtime dependency on go-1.4, so if go-1.4 is not yet installed, it will be installed first. (Granted, it should be a library dependency, not a runtime dependency, since it is also needed at build time, not just at runtime).

go (from versions 1.5 upwards) only depends on go-1.4 if being built from source, and this is only a build time dependancy. What I meant to say was that, if someone installs go and removes go-1.4, go-1.4 would be (unnecessarily) reinstalled if the go port needed to be upgraded.

Either way, this is beyond the scope of this ticket and my patch is working, so I'd be very grateful you would check (and hopefully) approve it.

comment:27 Changed 8 years ago by seanfarley (Sean Farley)

I just installed Sierra and can confirm these patches more-or-less fix things. Ryan, I think go-1.4 is just needed for bootstrapping go 1.7. If I don't hear any objections, I can clean up the patches and push them (I left go-1.4 as a build dep).

comment:28 in reply to:  26 ; Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to ethansherriff@…:

Replying to ryandesign@…:

Doesn't it already do that? It has a runtime dependency on go-1.4, so if go-1.4 is not yet installed, it will be installed first. (Granted, it should be a library dependency, not a runtime dependency, since it is also needed at build time, not just at runtime).

go (from versions 1.5 upwards) only depends on go-1.4 if being built from source, and this is only a build time dependancy.

Christoph, do you remember why, in closing #48940 via r140801, you made go-1.4 a runtime dependency, rather than build-time dependency as had been proposed in attachment:ticket:48940:Portfile-go.diff?

What I meant to say was that, if someone installs go and removes go-1.4, go-1.4 would be (unnecessarily) reinstalled if the go port needed to be upgraded.

In the go port's current state, a user who tries to uninstall go-1.4 cannot do so:

$ sudo port uninstall go-1.4
--->  Unable to uninstall go-1.4 @1.4.3_0, the following ports depend on it:
--->  	go @1.7_0
Error: Failed to uninstall go-1.4: Please uninstall the ports that depend on go-1.4 first.
Error: See /opt/local/var/macports/logs/_opt_local_var_macports_registry_portfiles_go-1.4-1.4.3_0_86ab4a8314cc0f60fb001e57c331230040cdff8c6728f9b181bb077d6f3a1fbd-2906/go-1.4/main.log for details.
Warning: Failed to execute portfile from registry for go-1.4 @1.4.3_0
--->  Unable to uninstall go-1.4 @1.4.3_0, the following ports depend on it:
--->  	go @1.7_0
Error: port uninstall failed: Please uninstall the ports that depend on go-1.4 first.

Once we change it from a runtime to a build-time dependency, that should become possible. MacPorts should not reinstall go-1.4 unless you are building from source; if you instead receive a binary from our build server, you don't need build dependencies.

If you are suggesting that we should only build go using go-1.4 if go is not installed, and should build go with go if go is already installed, then we don't want to do that, because that would be contrary to the goal of ReproducibleBuilds.

Last edited 8 years ago by ryandesign (Ryan Carsten Schmidt) (previous) (diff)

comment:29 Changed 8 years ago by martijnverburg@…

Cc: martijnverburg@… added

Cc Me!

comment:30 in reply to:  28 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed
Summary: go-1.4 @1.4.3_0: build fails on Sierrago-1.4 @1.4.3_0: fatal error: MSpanList_Insert

Replying to ryandesign@…:

Christoph, do you remember why, in closing #48940 via r140801, you made go-1.4 a runtime dependency, rather than build-time dependency as had been proposed in attachment:ticket:48940:Portfile-go.diff?

Christoph changed it to a build dependency in r153179 and fixed the build failure of go-1.4 on Sierra in r153178.

comment:31 in reply to:  28 Changed 8 years ago by ci42

Replying to ryandesign@…:

Christoph, do you remember why, in closing #48940 via r140801, you made go-1.4 a runtime dependency, rather than build-time dependency as had been proposed in attachment:ticket:48940:Portfile-go.diff?

I can't remember. I think I just made a mistake.

Note: See TracTickets for help on using tickets.