go-curl does not compile on precise builders

Bug #1196811 reported by Dave Cheney
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Go Windows Azure Client Library
Invalid
Undecided
Unassigned
juju-core
Won't Fix
Undecided
Unassigned
golang (Ubuntu)
Fix Released
Undecided
Unassigned
Precise
Won't Fix
Undecided
Unassigned

Bug Description

Precise ships with Go version 1.0.0 (technically the tag is go1, but that is unhelpful)

This version of Go contains a bug in the cgo preprocessor which makes it incompatible with the go-curl library used by gwacl.

ubuntu@ip-10-248-67-205:~$ go version
go version go1
ubuntu@ip-10-248-67-205:~$ go get -v launchpad.net/juju-core/...
github.com/andelf/go-curl
# github.com/andelf/go-curl
src/github.com/andelf/go-curl/c-callback.c: In function ‘header_function’:
src/github.com/andelf/go-curl/c-callback.c:8:52: error: ‘GoUintptr’ undeclared (first use in this function)
src/github.com/andelf/go-curl/c-callback.c:8:52: note: each undeclared identifier is reported only once for each function it appears in
src/github.com/andelf/go-curl/c-callback.c:8:62: error: expected ‘)’ before ‘ctx’
src/github.com/andelf/go-curl/c-callback.c:8:62: error: too few arguments to function ‘goGetCurlField’
callback.go:23:16: note: declared here
src/github.com/andelf/go-curl/c-callback.c:9:70: error: expected ‘)’ before ‘ctx’
src/github.com/andelf/go-curl/c-callback.c:9:70: error: too few arguments to function ‘goGetCurlField’
callback.go:23:16: note: declared here
src/github.com/andelf/go-curl/c-callback.c: In function ‘write_function’:
src/github.com/andelf/go-curl/c-callback.c:24:51: error: ‘GoUintptr’ undeclared (first use in this function)
src/github.com/andelf/go-curl/c-callback.c:24:61: error: expected ‘)’ before ‘ctx’
src/github.com/andelf/go-curl/c-callback.c:24:61: error: too few arguments to function ‘goGetCurlField’
callback.go:23:16: note: declared here
src/github.com/andelf/go-curl/c-callback.c:25:70: error: expected ‘)’ before ‘ctx’
src/github.com/andelf/go-curl/c-callback.c:25:70: error: too few arguments to function ‘goGetCurlField’
callback.go:23:16: note: declared here
src/github.com/andelf/go-curl/c-callback.c: In function ‘read_function’:
src/github.com/andelf/go-curl/c-callback.c:39:50: error: ‘GoUintptr’ undeclared (first use in this function)
src/github.com/andelf/go-curl/c-callback.c:39:60: error: expected ‘)’ before ‘ctx’
src/github.com/andelf/go-curl/c-callback.c:39:60: error: too few arguments to function ‘goGetCurlField’
callback.go:23:16: note: declared here
src/github.com/andelf/go-curl/c-callback.c:40:70: error: expected ‘)’ before ‘ctx’
src/github.com/andelf/go-curl/c-callback.c:40:70: error: too few arguments to function ‘goGetCurlField’
callback.go:23:16: note: declared here
src/github.com/andelf/go-curl/c-callback.c: In function ‘progress_function’:
src/github.com/andelf/go-curl/c-callback.c:55:54: error: ‘GoUintptr’ undeclared (first use in this function)
src/github.com/andelf/go-curl/c-callback.c:55:64: error: expected ‘)’ before ‘ctx’
src/github.com/andelf/go-curl/c-callback.c:55:64: error: too few arguments to function ‘goGetCurlField’
callback.go:23:16: note: declared here
src/github.com/andelf/go-curl/c-callback.c:56:69: error: expected ‘)’ before ‘ctx’
src/github.com/andelf/go-curl/c-callback.c:56:69: error: too few arguments to function ‘goGetCurlField’
callback.go:23:16: note: declared here

This bug is not present when compiling on Q and R which ship with a version of Go derived from go1.0.2

Tags: golang-mre
affects: gwacl → juju-core
Revision history for this message
Dave Cheney (dave-cheney) wrote :

I have confirmed that this bug is present when compiling with Go versions up to 1.0.1. This bug was fixed when compiling with Go 1.0.2 or later.

affects: juju-core → gwacl
James Page (james-page)
Changed in golang (Ubuntu):
status: New → Fix Released
James Page (james-page)
tags: added: golang-mre
Revision history for this message
Julian Edwards (julian-edwards) wrote :

We've remove go-curl as a dependency from gwacl now.

Changed in gwacl:
status: New → Invalid
Curtis Hovey (sinzui)
Changed in juju-core:
status: New → Won't Fix
Revision history for this message
Steve Langasek (vorlon) wrote :

The Precise Pangolin has reached end of life, so this bug will not be fixed for that release

Changed in golang (Ubuntu Precise):
status: New → Won't Fix
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.