On Saturday 05 May 2012, Tobias Sarnowski wrote:
> On 05/04/12 15:11, Joel Sing wrote:
> > The following diff update Go to version 1.0.1. This release pulls several
> > bug fixes up to the stable branch.
> >
> > ok?
>
> Basically works for me on openbsd-current/amd64 with one exception:
Thanks.
> Using cgo to use system calls does not work. I attached an example
> package which works perfectly on linux/amd64 ("go test") but fails on my
> openbsd machines. The compiled package (.a file) is empty and does not
> contain the function symbols. Seems that the cgo compiler does not get
> triggered correctly.
>
> This is not a specific bug of your patch now (its broken in the original
> mercurial repository too) but maybe you have a clue where to look at.
cgo is not currently supported/enabled on OpenBSD. Now that rthreads is
enabled we are part of the way to being able to enable it. However, we are
still missing support for PT_TLS program headers, which means that the TLS
memory is eventually corrupted.
If you want to play you can use this change, however expect to see random
segfaults (mostly when using go routines):
http://codereview.appspot.com/5819050
> Maybe its the same problem why the syscall[1] package is not implemented
> on openbsd?
What do you mean by "not implemented"?
The syscall package is definately implemented for OpenBSB, since it is an
essential part of the Go runtime packages:
$ cat syscall.go
package main
import (
"fmt"
"syscall"
)
func main() {
fmt.Println(syscall.Sysctl("kern.version"))
}
$ go run syscall.go
OpenBSD 5.1-current (GENERIC.MP) #0: Tue Mar 27 20:25:57 EST 2012
joel@obsdamd64.localdomain:/usr/src/sys/arch/amd64/compile/GENERIC.MP
> [1] http://golang.org/pkg/syscall/
>
> > Index: Makefile
> > ===================================================================
> > RCS file: /cvs/ports/lang/go/Makefile,v
> > retrieving revision 1.2
> > diff -u -p -r1.2 Makefile
> > --- Makefile 15 Apr 2012 13:37:27 -0000 1.2
> > +++ Makefile 4 May 2012 12:50:14 -0000
> > @@ -4,10 +4,9 @@ ONLY_FOR_ARCHS = amd64 i386
> >
> > COMMENT = Go programming language
> >
> > -VERSION = 1
> > -REVISION = 0
> > +VERSION = 1.0.1
> > EXTRACT_SUFX = .src.tar.gz
> > -DISTNAME = go.go${VERSION}
> > +DISTNAME = go${VERSION}
> > PKGNAME = go-${VERSION}
> > CATEGORIES = lang
> >
> > @@ -61,6 +60,8 @@ do-configure:
> >
> > do-build:
> > @cd ${WRKSRC}&& \
> > + GOROOT=${WRKDIST} \
> > + GOBIN=${WRKDIST}/bin \
> > GOROOT_FINAL=${GOROOT} ./make.bash --no-banner
> >
> > do-regress:
> > Index: distinfo
> > ===================================================================
> > RCS file: /cvs/ports/lang/go/distinfo,v
> > retrieving revision 1.1.1.1
> > diff -u -p -r1.1.1.1 distinfo
> > --- distinfo 31 Mar 2012 12:37:16 -0000 1.1.1.1
> > +++ distinfo 4 May 2012 12:50:14 -0000
> > @@ -1,5 +1,5 @@
> > -MD5 (go.go1.src.tar.gz) = eYvPwjm/mC1xlWMMwLQUaw==
> > -RMD160 (go.go1.src.tar.gz) = Rr015SaZ2zNWmPPUQFShJQRQy4U=
> > -SHA1 (go.go1.src.tar.gz) = YCNiPQg9sZgJZTNbisT6i0KPpIQ=
> > -SHA256 (go.go1.src.tar.gz) =
> > RO1cFmVNAuzZGtO9jlVkG4FVYcTsKATSl1PbvO+6dt4= -SIZE (go.go1.src.tar.gz) =
> > 9637785
> > +MD5 (go1.0.1.src.tar.gz) = NM1dWSYklvWamIROlMi0tw==
> > +RMD160 (go1.0.1.src.tar.gz) = Ti2OM1GbL9NWKkwGdi8C7lAU+mY=
> > +SHA1 (go1.0.1.src.tar.gz) = /IptZyX38r98lGhcX9CIDJt/Z/Y=
> > +SHA256 (go1.0.1.src.tar.gz) =
> > Kc26e8kJ33CR2B9SBJ3gI1ArWzNRzSBglPLC2ZYcAxU= +SIZE (go1.0.1.src.tar.gz) =
> > 9617092
> > Index: pkg/PLIST
> > ===================================================================
> > RCS file: /cvs/ports/lang/go/pkg/PLIST,v
> > retrieving revision 1.2
> > diff -u -p -r1.2 PLIST
> > --- pkg/PLIST 15 Apr 2012 13:19:15 -0000 1.2
> > +++ pkg/PLIST 4 May 2012 12:50:15 -0000
> > @@ -628,12 +628,6 @@ go/src/cmd/dist/buf.c
> > go/src/cmd/dist/build.c
> > go/src/cmd/dist/buildgc.c
> > go/src/cmd/dist/buildruntime.c
> > -go/src/cmd/dist/dist.dSYM/
> > -go/src/cmd/dist/dist.dSYM/Contents/
> > -go/src/cmd/dist/dist.dSYM/Contents/Info.plist
> > -go/src/cmd/dist/dist.dSYM/Contents/Resources/
> > -go/src/cmd/dist/dist.dSYM/Contents/Resources/DWARF/
> > -go/src/cmd/dist/dist.dSYM/Contents/Resources/DWARF/dist
> > go/src/cmd/dist/goc2c.c
> > go/src/cmd/dist/main.c
> > go/src/cmd/dist/unix.c
> > @@ -1882,6 +1876,8 @@ go/src/pkg/mime/multipart/formdata.go
> > go/src/pkg/mime/multipart/formdata_test.go
> > go/src/pkg/mime/multipart/multipart.go
> > go/src/pkg/mime/multipart/multipart_test.go
> > +go/src/pkg/mime/multipart/testdata/
> > +go/src/pkg/mime/multipart/testdata/nested-mime
> > go/src/pkg/mime/multipart/writer.go
> > go/src/pkg/mime/multipart/writer_test.go
> > go/src/pkg/mime/test.types
--
"Reason is not automatic. Those who deny it cannot be conquered by it.
Do not count on them. Leave them alone." -- Ayn Rand
