Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#41656 closed update (fixed)

Update Go Portfile to 1.2

Reported by: b@… Owned by: ci42
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc:
Port: go

Description

Go 1.2 is officially released: http://blog.golang.org/go12 Go 1.2 fixes several bugs with Clang compilation (confirmed on OS X 10.9), so clang should not be blacklisted anymore.

Attachments (1)

Portfile-go.diff (1.2 KB) - added by b@… 10 years ago.

Download all attachments as: .zip

Change History (11)

Changed 10 years ago by b@…

Attachment: Portfile-go.diff added

comment:1 Changed 10 years ago by mf2k (Frank Schima)

Keywords: haspatch added; go removed
Owner: changed from macports-tickets@… to ciserlohn@…
Version: 2.2.1

Thanks. In the future, please Cc the port maintainers (port info --maintainers go).

comment:2 Changed 10 years ago by bmhatfield@…

Silly question - but how long until this is merged / released?

comment:3 in reply to:  description Changed 10 years ago by ci42

Replying to b@…:

Go 1.2 is officially released: http://blog.golang.org/go12

Yes, I know. I'm on the golang mailing list(s).

Go 1.2 fixes several bugs with Clang compilation (confirmed on OS X 10.9), so clang should not be blacklisted anymore.

Thanks for the patch but just changing the checksums and removing the clang blacklisting is not enough. The godoc and vet commands have been moved to another repository (see http://golang.org/doc/go1.2#go_tools_godoc). Including them in the port requires some extra work.

Last edited 10 years ago by ci42 (previous) (diff)

comment:4 in reply to:  2 Changed 10 years ago by ci42

Replying to bmhatfield@…:

Silly question - but how long until this is merged / released?

Expect the update within the next few days.

comment:5 Changed 10 years ago by ci42

Resolution: fixed
Status: newclosed

Updated to 1.2 in r114986. Currently without the godoc and vet command since the upstream developers refuse to properly tag them (https://code.google.com/p/go/issues/detail?id=6905&can=1&q=godoc%201.2%20tag&colspec=ID%20Status%20Stars%20Release%20Owner%20Repo%20Summary).

comment:6 Changed 10 years ago by zquestz (Josh Ellithorpe)

Seems more like they are using a different way of tracking than the port maintainer wishes... they are in their own branch and could easily be provided. Why hurt all the users that need a complete port for golang because of politics? Not including those other tools makes this port pretty worthless since it is incomplete and most Go editors include go doc and go vet hooks that won't work with this build.

Version 1, edited 10 years ago by zquestz (Josh Ellithorpe) (previous) (next) (diff)

comment:7 Changed 10 years ago by b@…

I would prefer to have separate ports for go-vet and go-doc.

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

Replying to b@…:

I would prefer to have separate ports for go-vet and go-doc.

A separate port or subport that tracks the go.tools subrepository seems like the best solution. Maybe “go-tools”.

comment:9 Changed 10 years ago by zquestz (Josh Ellithorpe)

That makes sense, but if you do it as a separate port, would it be locked on a particular SHA, or would it just build the latest within the go 1.2 branch? Seems the issue the maintainer brought up on code.google.com still wouldn't be addressed. The package could change whenever someone commits to that branch.

comment:10 in reply to:  9 Changed 10 years ago by larryv (Lawrence Velázquez)

Replying to quest@…:

That makes sense, but if you do it as a separate port, would it be locked on a particular SHA, or would it just build the latest within the go 1.2 branch? Seems the issue the maintainer brought up on code.google.com still wouldn't be addressed. The package could change whenever someone commits to that branch.

The new port would have to be fixed to a specific changeset. The point would be to divorce Go’s versioning from the tools’ (nonexistent) “versioning”.

Note: See TracTickets for help on using tickets.