]>
Cypherpunks repositories - gostls13.git/log
Russ Cox [Wed, 14 Mar 2012 18:47:13 +0000 (14:47 -0400)]
io/ioutil: fix crash when Stat fails
Fixes #3320.
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/
5824051
Shenghou Ma [Wed, 14 Mar 2012 18:24:49 +0000 (02:24 +0800)]
runtime: manage stack by ourselves for badcallback on windows/amd64
This function uses 48-byte of precious non-split stack for every callback
function, and without this CL, it can easily overflow the non-split stack.
I encountered this when trying to enable misc/cgo/test on windows/amd64.
R=rsc
CC=golang-dev
https://golang.org/cl/
5784075
Volker Dobler [Wed, 14 Mar 2012 16:49:57 +0000 (12:49 -0400)]
cmd/go: trivial help message fix for go help get
Direct reference to go help build where the flags are described.
R=golang-dev, adg, rsc
CC=golang-dev
https://golang.org/cl/
5825046
Shenghou Ma [Wed, 14 Mar 2012 15:54:40 +0000 (23:54 +0800)]
os: IsNotExist() should also consider ERROR_PATH_NOT_FOUND on Windows
Also update documentation about IsExist() and IsNotExist(), they are not
about files only.
R=rsc
CC=golang-dev
https://golang.org/cl/
5794073
Brad Fitzpatrick [Wed, 14 Mar 2012 15:24:11 +0000 (08:24 -0700)]
misc/dist: don't lose mode bits when setting tar permissions
R=golang-dev, bsiegert, rsc
CC=golang-dev
https://golang.org/cl/
5822046
Andrew Gerrand [Wed, 14 Mar 2012 12:47:34 +0000 (23:47 +1100)]
misc/dist: remove exp and old before building
Fixes #3317.
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5784074
Yasuhiro Matsumoto [Wed, 14 Mar 2012 07:43:01 +0000 (18:43 +1100)]
misc/vim: restore fileencodings.
Currently, ftdetect/gofiletype.vim set fileencodings to open the file as
utf-8 encoding event if the file does not contain multibyte characters.
But fileencodings is global option.
$ vim foo.txt
:set fileencodings
utf-8,ucs-bom,cp932
$ vim foo.go
:set fileencodings
utf-8
This change restore fileencodings before opening the file.
Also added specify fileformats=unix.
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5718045
Andrew Gerrand [Wed, 14 Mar 2012 06:09:15 +0000 (17:09 +1100)]
misc/dist: force modes to 0755 or 0644 in tarballs
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5823045
Rob Pike [Wed, 14 Mar 2012 04:08:54 +0000 (15:08 +1100)]
html/template: fix nil pointer bug
Fixes #3272.
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/
5819046
Mikio Hara [Wed, 14 Mar 2012 04:07:25 +0000 (13:07 +0900)]
runtime/cgo: linux signal masking
Fixes #3314.
Fixes #3101 (again).
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/
5820047
Mikio Hara [Wed, 14 Mar 2012 03:49:59 +0000 (12:49 +0900)]
runtime/cgo: darwin signal masking
Fixes #3101 (again).
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/
5825043
Andrew Gerrand [Wed, 14 Mar 2012 03:02:59 +0000 (14:02 +1100)]
cmd/godoc: s/ignore/appengine/ in appinit.go +build constraint
R=golang-dev
CC=golang-dev
https://golang.org/cl/
5784072
Andrew Gerrand [Wed, 14 Mar 2012 02:19:39 +0000 (13:19 +1100)]
godoc: fix vet warnings for untagged struct literals
src/cmd/godoc/filesystem.go:337:10: os.PathError struct literal uses untagged fields
src/cmd/godoc/filesystem.go:355:10: os.PathError struct literal uses untagged fields
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5820046
Francisco Souza [Wed, 14 Mar 2012 02:03:11 +0000 (13:03 +1100)]
doc: add Go Concurrency Patterns: Timing out, moving on article
Originally published on The Go Programming Language Blog, September 23, 2010.
http://blog.golang.org/2010/09/go-concurrency-patterns-timing-out-and.html
Update #2547.
R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/
5815044
Robert Griesemer [Wed, 14 Mar 2012 00:29:07 +0000 (17:29 -0700)]
all: various typos
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5820045
Johan Euphrosine [Wed, 14 Mar 2012 00:27:41 +0000 (11:27 +1100)]
doc: add Go image/draw package article and convert code snippets to Go1.
Originally published on The Go Programming Language Blog, September 29, 2011.
http://blog.golang.org/2011/09/go-imagedraw-package.html
Update #2547.
R=golang-dev, adg, rsc
CC=golang-dev
https://golang.org/cl/
5755057
Brad Fitzpatrick [Tue, 13 Mar 2012 23:55:43 +0000 (16:55 -0700)]
html/template: fix panic on Clone
Fixes #3281
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/
5819044
Rob Pike [Tue, 13 Mar 2012 23:46:21 +0000 (10:46 +1100)]
text/template: variables do not take arguments
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5821044
Robert Griesemer [Tue, 13 Mar 2012 23:15:58 +0000 (16:15 -0700)]
go/printer, gofmt: fix multi-line logic
A node spans multiple lines if the line difference
between start and end point is > 0 (rather than > 1).
Fixes some odd cases introduced by CL
5706055 ;
pointed out by dsymonds.
Added corresponding test case. The other change
in the .golden file reverts to the status before
the CL mentioned above and is correct.
gofmt -w src misc changes godoc.go back to where
it was before the CL mentioned above.
Fixes #3304.
R=dsymonds, rsc
CC=golang-dev
https://golang.org/cl/
5820044
Shenghou Ma [Tue, 13 Mar 2012 20:59:17 +0000 (04:59 +0800)]
fmt: remove dead code
Also fix a typo in comment.
R=golang-dev, r, r
CC=golang-dev
https://golang.org/cl/
5808043
Rob Pike [Tue, 13 Mar 2012 20:03:11 +0000 (07:03 +1100)]
text/template: fix a couple of parse bugs around identifiers.
1) Poor error checking in variable declarations admitted
$x=2 or even $x%2.
2) Need white space or suitable termination character
after identifiers, so $x+2 doesn't parse, in case we want it
to mean something one day.
Number 2 in particular prevents mistakes that we will have
to honor later and so is necessary for Go 1.
Fixes #3270.
Fixes #3271.
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/
5795073
Maxim Pimenov [Tue, 13 Mar 2012 14:00:43 +0000 (10:00 -0400)]
go/build: fix match
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/
5801043
Russ Cox [Tue, 13 Mar 2012 13:33:54 +0000 (09:33 -0400)]
gc: use quoted string format in import error
R=ken2
CC=golang-dev
https://golang.org/cl/
5794077
Alex Brainman [Tue, 13 Mar 2012 05:50:44 +0000 (16:50 +1100)]
make.bat: properly handle directories with spaces
R=golang-dev, minux.ma
CC=golang-dev
https://golang.org/cl/
5797079
Shenghou Ma [Tue, 13 Mar 2012 05:48:07 +0000 (13:48 +0800)]
os: remove document duplication in error predicate functions
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/
5783092
Andrew Gerrand [Tue, 13 Mar 2012 05:45:08 +0000 (16:45 +1100)]
tag weekly.2012-03-13
R=dsymonds
CC=golang-dev
https://golang.org/cl/
5795072
Andrew Gerrand [Tue, 13 Mar 2012 05:39:52 +0000 (16:39 +1100)]
weekly.2012-03-13
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5754097
Andrew Gerrand [Tue, 13 Mar 2012 05:32:23 +0000 (16:32 +1100)]
doc: update weekly notes, untag weekly
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5783093
Rob Pike [Tue, 13 Mar 2012 05:03:19 +0000 (16:03 +1100)]
gc: allow ~ in import paths
Windows has paths like C:/Users/ADMIN~1. Also, it so happens
that go/parser allows ~ in import paths. So does the spec.
Fixes the build too.
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5777073
Brad Fitzpatrick [Tue, 13 Mar 2012 04:49:43 +0000 (21:49 -0700)]
misc/dist: better archive/tar Headers
This should live in archive/tar later (CL
5796073 ) but we
can always do that after Go 1 and stick it here for now.
R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/
5754096
Rob Pike [Tue, 13 Mar 2012 04:35:08 +0000 (15:35 +1100)]
gc: include full text of import path in error message
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5783091
Andrew Gerrand [Tue, 13 Mar 2012 04:31:39 +0000 (15:31 +1100)]
misc/dist: trim space from version string
This prevents us from writing filenames like "weekly.2012-03-12\n.foo.bar.tar.gz".
R=bradfitz
CC=golang-dev
https://golang.org/cl/
5783090
Andrew Gerrand [Tue, 13 Mar 2012 04:12:05 +0000 (15:12 +1100)]
tag weekly.2012-03-12
R=golang-dev
CC=golang-dev
https://golang.org/cl/
5796074
Andrew Gerrand [Tue, 13 Mar 2012 04:09:51 +0000 (15:09 +1100)]
weekly.2012-03-12
R=golang-dev, nigeltao, bradfitz
CC=golang-dev
https://golang.org/cl/
5783077
Andrew Gerrand [Tue, 13 Mar 2012 04:02:36 +0000 (15:02 +1100)]
doc: increase h3 size
This makes h3 and h4 more distinct, and h2 and h3 the same size.
The h2 and h3 styles may be distinguished h2's background styles.
This means that almost all text on the site is either 16px, 20px,
or 24px. (with a smattering of 14px)
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/
5754095
Andrew Gerrand [Tue, 13 Mar 2012 04:02:14 +0000 (15:02 +1100)]
doc: windows zip archive installation instructions
Fixes #3254.
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/
5794071
Francisco Souza [Tue, 13 Mar 2012 03:46:08 +0000 (14:46 +1100)]
doc: add Go's declaration syntax article
Originally published on The Go Programming Language Blog, July 7, 2010.
I did not put any go file in doc/progs. Shoul I include Go files
with those declarations?
http://blog.golang.org/2010/07/gos-declaration-syntax.html
Update #2547.
R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/
5795068
Robert Griesemer [Tue, 13 Mar 2012 03:27:27 +0000 (20:27 -0700)]
go spec: fix inaccuracy in type identity definition
Pointed out by Steven Blenkinsop (steven099@gmail.com)
via golang-nuts.
R=r, dsymonds
CC=golang-dev
https://golang.org/cl/
5783087
Andrew Gerrand [Tue, 13 Mar 2012 02:43:41 +0000 (13:43 +1100)]
doc: update reference gopher
R=golang-dev
CC=golang-dev
https://golang.org/cl/
5754094
Alex Brainman [Tue, 13 Mar 2012 02:42:55 +0000 (22:42 -0400)]
runtime: fix windows/amd64 exception handler
R=golang-dev, rsc
CC=golang-dev, hectorchu
https://golang.org/cl/
5797077
Robert Hencke [Tue, 13 Mar 2012 02:41:15 +0000 (19:41 -0700)]
cmd/go: fix typo
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/
5777072
Francisco Souza [Tue, 13 Mar 2012 01:57:48 +0000 (12:57 +1100)]
godoc: added alt and title attributes to the "pop out" button
R=adg
CC=golang-dev
https://golang.org/cl/
5777071
David Symonds [Tue, 13 Mar 2012 01:52:15 +0000 (12:52 +1100)]
build: update comment about GO_LDFLAGS.
(5l/6l/8l aren't involved in building packages)
R=golang-dev, robert.hencke
CC=golang-dev
https://golang.org/cl/
5798071
Alex Brainman [Tue, 13 Mar 2012 01:51:28 +0000 (12:51 +1100)]
test: actually run them on windows
R=golang-dev, r, rsc, bradfitz, minux.ma
CC=golang-dev
https://golang.org/cl/
5756065
Shenghou Ma [Tue, 13 Mar 2012 01:50:04 +0000 (12:50 +1100)]
os, syscall: windows really isn't posix compliant, fix os.IsExist()
R=golang-dev, rsc, bradfitz, alex.brainman
CC=golang-dev
https://golang.org/cl/
5754083
Andrew Gerrand [Tue, 13 Mar 2012 00:55:16 +0000 (11:55 +1100)]
doc/progs: skip cgo1 and cgo2 on freebsd
FreeBSD's srandom has a different signature to darwin/linux.
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5795067
Andrew Gerrand [Tue, 13 Mar 2012 00:00:11 +0000 (11:00 +1100)]
godoc: change link styles, add 'pop out' button
Remove underlines from all links, show underline on link hover.
Change all non-link headings to h4, a slight visual cue.
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/
5783088
Andrew Gerrand [Mon, 12 Mar 2012 23:17:10 +0000 (10:17 +1100)]
godoc: use elastic layout for most pages
We use the absence of a Title to detect the front page.
I can't find other pages without titles, so this seems reasonable.
R=golang-dev, bradfitz, gri
CC=golang-dev
https://golang.org/cl/
5797076
Brad Fitzpatrick [Mon, 12 Mar 2012 22:57:38 +0000 (15:57 -0700)]
cmd/godoc: remove sync code
Fixes #3273
R=gri
CC=golang-dev
https://golang.org/cl/
5795065
Francisco Souza [Mon, 12 Mar 2012 22:08:04 +0000 (09:08 +1100)]
godoc: fix codewalk handler
For URLs ending with /, the handler did not work, trying to append
".xml" to the path.
For instance, the "Share Memory by Communicating" returned the
following error:
open /Users/francisco.souza/lib/go/doc/codewalk/sharemem/.xml: no such file or directory
R=adg, minux.ma
CC=golang-dev
https://golang.org/cl/
5797065
Francisco Souza [Mon, 12 Mar 2012 22:07:37 +0000 (09:07 +1100)]
doc: add C? Go? Cgo! article
Originally published on The Go Programming Language Blog, March 17, 2011.
http://blog.golang.org/2011/03/c-go-cgo.html
Update #2547.
R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/
5777054
Fazlul Shahriar [Mon, 12 Mar 2012 21:08:00 +0000 (14:08 -0700)]
os/exec: fix typo in documentation
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/
5797073
Russ Cox [Mon, 12 Mar 2012 21:03:29 +0000 (17:03 -0400)]
undo CL
5754088 /
cae9a7c0db06
broke builders
««« original CL description
cmd/go: respect $GOBIN always
Before, we only consulted $GOBIN for source code
found in $GOROOT, but that's confusing to explain
and less useful. The new behavior lets users set
GOBIN=$HOME/bin and have all go-compiled binaries
installed there.
Fixes #3269.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5754088
»»»
TBR=bradfitz
CC=golang-dev
https://golang.org/cl/
5794065
Russ Cox [Mon, 12 Mar 2012 20:49:12 +0000 (16:49 -0400)]
cmd/go: respect $GOBIN always
Before, we only consulted $GOBIN for source code
found in $GOROOT, but that's confusing to explain
and less useful. The new behavior lets users set
GOBIN=$HOME/bin and have all go-compiled binaries
installed there.
Fixes #3269.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5754088
Russ Cox [Mon, 12 Mar 2012 20:48:16 +0000 (16:48 -0400)]
runtime: fix windows/amd64
Maybe.
TBR=bradfitz
CC=golang-dev
https://golang.org/cl/
5754091
Russ Cox [Mon, 12 Mar 2012 20:35:15 +0000 (16:35 -0400)]
cmd/go: make go get new.code/... work
Fixes #2909.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5796072
Russ Cox [Mon, 12 Mar 2012 20:34:24 +0000 (16:34 -0400)]
cmd/go: make net/... match net too
Otherwise there's no good way to get both, and it comes up often.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5794064
Russ Cox [Mon, 12 Mar 2012 20:29:33 +0000 (16:29 -0400)]
log: fix doc comment for Ldate
Fixes #3303.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5795062
Brad Fitzpatrick [Mon, 12 Mar 2012 20:25:48 +0000 (13:25 -0700)]
cmd/fix: add rules for net/http -> net/http/httputil renames
And merge the httputil fix into go1rename.
R=golang-dev, r, dsymonds, r, rsc
CC=golang-dev
https://golang.org/cl/
5696084
Brad Fitzpatrick [Mon, 12 Mar 2012 20:17:48 +0000 (13:17 -0700)]
os: document FileInfo.Size as system-dependent for irregular files
R=rsc, r
CC=golang-dev
https://golang.org/cl/
5795059
Russ Cox [Mon, 12 Mar 2012 19:55:18 +0000 (15:55 -0400)]
runtime: print error on receipt of signal on non-Go thread
It's the best we can do before Go 1.
For issue 3250; not a fix but at least less mysterious.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5797068
Shenghou Ma [Mon, 12 Mar 2012 19:34:22 +0000 (03:34 +0800)]
build: build correct cmd/dist matching GOHOSTARCH
Fix for issue 3210 comment #1.
R=adg, rsc
CC=golang-dev
https://golang.org/cl/
5794057
Shenghou Ma [Mon, 12 Mar 2012 19:31:11 +0000 (03:31 +0800)]
build: update Makefile to track source code dependencies better
Also update .hgignore to ignore y.output.
R=rsc
CC=golang-dev
https://golang.org/cl/
5797061
Shenghou Ma [Mon, 12 Mar 2012 19:24:13 +0000 (03:24 +0800)]
gc: correct comment in runtime.go
R=rsc, iant
CC=golang-dev
https://golang.org/cl/
5777057
Ingo Oeser [Mon, 12 Mar 2012 18:41:01 +0000 (14:41 -0400)]
cmd/go: allow ssh tunnelled bzr, git and svn
This is often used in private hosting and collaborating environments.
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/
5753063
Russ Cox [Mon, 12 Mar 2012 18:39:44 +0000 (14:39 -0400)]
codereview: fix for Mercurial 2.1
Mercurial: the Python of version control systems.
Python: the Mercurial of programming languages.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5777066
Russ Cox [Mon, 12 Mar 2012 18:26:10 +0000 (14:26 -0400)]
html/template: doc nit
Execute's data is untrusted regardless of package.
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/
5797062
Russ Cox [Mon, 12 Mar 2012 18:25:52 +0000 (14:25 -0400)]
A+C: add Ingo Oeser (individual CLA)
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5794061
Brad Fitzpatrick [Mon, 12 Mar 2012 17:42:25 +0000 (10:42 -0700)]
net/http: return appropriate errors from ReadRequest
Fixes #3298
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/
5783080
Russ Cox [Mon, 12 Mar 2012 17:10:37 +0000 (13:10 -0400)]
cmd/godoc: fix directory read
Undo CL
5783076 and apply correct fix.
The /doc hack is wrong. The code to handle this case was
already there and just needs a simple fix:
// We didn't find any directories containing Go files.
// If some directory returned successfully, use that.
- if len(all) == 0 && first != nil {
+ if !haveGo {
for _, d := range first {
haveName[d.Name()] = true
all = append(all, d)
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5783079
Russ Cox [Mon, 12 Mar 2012 16:39:31 +0000 (12:39 -0400)]
cmd/go: stop using $GOROOT and $GOBIN in script output
They were necessary to produce a canonical script
when we checked in build scripts, but now they're just
getting in the way.
Fixes #3279.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5796068
David Symonds [Mon, 12 Mar 2012 06:33:35 +0000 (17:33 +1100)]
archive/tar: catch short writes.
Also make error messages consistent throughout.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5777064
Brad Fitzpatrick [Mon, 12 Mar 2012 06:07:38 +0000 (23:07 -0700)]
misc/dist: use archive/tar to generate tarballs
For people untarring with -p or as root, preserving file permissions.
This way we don't make tars owned by adg/eng or adg/staff or whatever
machine Andrew was on. Instead, we always build tarballs owned by predictable
users.
Except archive/tar doesn't seem to work.
Updates #3209.
R=golang-dev, adg
CC=dsymonds, golang-dev
https://golang.org/cl/
5796064
Andrew Gerrand [Mon, 12 Mar 2012 04:55:39 +0000 (15:55 +1100)]
cmd/godoc: always include /doc files in union filesystems
Makes Path rewrites work, as the metadata was never being scanned.
Fixes #3282.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5783076
Brad Fitzpatrick [Mon, 12 Mar 2012 04:02:40 +0000 (21:02 -0700)]
misc/dist: use archive/zip, seek out windows deps, add --upload flag
Use archive/zip instead of 7z on Windows.
Look for all Windows deps before starting build, and include looking
for them in their common locations instead of making users update
their PATHs.
Add an --upload flag that, if set to false, doesn't require credential
files.
R=golang-dev, alex.brainman, adg
CC=golang-dev
https://golang.org/cl/
5794046
Robert Hencke [Mon, 12 Mar 2012 03:04:45 +0000 (12:04 +0900)]
crypto/tls, fmt: print fixes
R=golang-dev, bradfitz, minux.ma, rsc, bradfitz
CC=golang-dev
https://golang.org/cl/
5787069
David Symonds [Mon, 12 Mar 2012 02:50:25 +0000 (13:50 +1100)]
cmd/vet: little tweak to tagged literal script to get it closer to drop-in format.
This is a minor change that made it easier for me; if you hate it I can drop it.
R=nigeltao
CC=golang-dev
https://golang.org/cl/
5798060
Shenghou Ma [Mon, 12 Mar 2012 02:20:25 +0000 (13:20 +1100)]
misc/dist: minimum target requirement is 10.6 for Darwin
As we've dropped support for Mac OS X 10.5, I think the generated .pkg
should reflect that decision.
But this CL make it impossible to generate pkg on Mac OS X 10.6, at least
for me.
R=adg, bradfitz
CC=golang-dev
https://golang.org/cl/
5798051
Brad Fitzpatrick [Mon, 12 Mar 2012 00:55:15 +0000 (17:55 -0700)]
cmd/api: work on Windows again, and make gccgo files work a bit more
handle string and []byte conversions.
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/
5754082
David Symonds [Mon, 12 Mar 2012 00:25:38 +0000 (11:25 +1100)]
cmd/api: set compiler for all build contexts.
The generated syscall files for Windows are still breaking "go tool api"
(unknown function []byte); I'll look at fixing that separately.
Fixes #3285.
R=bradfitz
CC=golang-dev
https://golang.org/cl/
5777062
Russ Cox [Sun, 11 Mar 2012 19:53:42 +0000 (15:53 -0400)]
cmd/go: local import fixes
1) The -D argument should always be a pseudo-import path,
like _/Users/rsc/foo/bar, never a standard import path,
because we want local imports to always resolve to pseudo-paths.
2) Disallow local imports in non-local packages. Otherwise
everything works but you get two copies of a package
(the real one and the "local" one) in your binary.
R=golang-dev, bradfitz, yiyu.jgl
CC=golang-dev
https://golang.org/cl/
5787055
Gwenael Treguier [Sat, 10 Mar 2012 23:21:44 +0000 (15:21 -0800)]
database/sql: ensure Stmts are correctly closed.
To make sure that there is no resource leak,
I suggest to fix the 'fakedb' driver such as it fails when any
Stmt is not closed.
First, add a check in fakeConn.Close().
Then, fix all missing Stmt.Close()/Rows.Close().
I am not sure that the strategy choose in fakeConn.Prepare/prepare* is ok.
The weak point in this patch is the change in Tx.Query:
- Tests pass without this change,
- I found it by manually analyzing the code,
- I just try to make Tx.Query look like DB.Query.
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5759050
Jeremy Jackins [Sat, 10 Mar 2012 23:13:34 +0000 (15:13 -0800)]
doc: fix typo in contribute.html
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5795050
Brad Fitzpatrick [Sat, 10 Mar 2012 22:48:00 +0000 (14:48 -0800)]
A+C: Add Gwenael Treguier (Individual CLA)
R=golang-dev, wcn
CC=golang-dev
https://golang.org/cl/
5777052
Brad Fitzpatrick [Sat, 10 Mar 2012 18:00:02 +0000 (10:00 -0800)]
database/sql: fix double connection free on Stmt.Query error
In a transaction, on a Stmt.Query error, it was possible for a
connection to be added to a db's freelist twice. Should use
the local releaseConn function instead.
Thanks to Gwenael Treguier for the failing test.
Also in this CL: propagate driver errors through releaseConn
into *DB.putConn, which conditionally ignores the freelist
addition if the driver signaled ErrBadConn, introduced in a
previous CL.
R=golang-dev, gary.burd
CC=golang-dev
https://golang.org/cl/
5798049
David Symonds [Sat, 10 Mar 2012 08:02:52 +0000 (19:02 +1100)]
net/mail: close minor TODO that was waiting on a 6g bug fix.
R=golang-dev, mikioh.mikioh
CC=golang-dev
https://golang.org/cl/
5796050
Brad Fitzpatrick [Sat, 10 Mar 2012 00:27:32 +0000 (16:27 -0800)]
net/http: fix crash with Transport.CloseIdleConnections
Thanks Michael Lore for the bug report!
Fixes #3266
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/
5754068
Brad Fitzpatrick [Fri, 9 Mar 2012 22:45:40 +0000 (14:45 -0800)]
archive/zip: verify CRC32s in non-streamed files
We should check the CRC32s of files on EOF, even if there's no
data descriptor (in streamed files), as long as there's a non-zero
CRC32 in the file header / TOC.
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/
5794045
Alex Brainman [Fri, 9 Mar 2012 22:35:56 +0000 (09:35 +1100)]
crypto/x509: do not forget to free cert context
R=golang-dev, krautz, rsc
CC=golang-dev
https://golang.org/cl/
5783059
Brad Fitzpatrick [Fri, 9 Mar 2012 22:12:02 +0000 (14:12 -0800)]
archive/zip: write data descriptor signature for OS X; fix bugs reading it
We now always write the "optional" streaming data descriptor
signature, which turns out to be required for OS X.
Also, handle reading the data descriptor with or without the
signature, per the spec's recommendation. Fix data descriptor
reading bugs found in the process.
Fixes #3252
R=golang-dev, alex.brainman, nigeltao, rsc
CC=golang-dev
https://golang.org/cl/
5787062
Robert Griesemer [Fri, 9 Mar 2012 21:53:25 +0000 (13:53 -0800)]
go/printer: example for Fprint
R=golang-dev, dsymonds, rsc
CC=golang-dev
https://golang.org/cl/
5785057
Shenghou Ma [Fri, 9 Mar 2012 21:11:51 +0000 (05:11 +0800)]
cmd/gc: import path cannot start with slash on Windows
For CL
5756065 .
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/
5784065
Shenghou Ma [Fri, 9 Mar 2012 19:42:23 +0000 (03:42 +0800)]
cmd/dist, cmd/go: move CGO_ENABLED from 'go tool dist env' to 'go env'
So that we don't duplicate knowledge about which OS/ARCH combination
supports cgo.
Also updated src/run.bash and src/sudo.bash to use 'go env'.
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/
5792055
Jeremy Jackins [Fri, 9 Mar 2012 19:21:01 +0000 (11:21 -0800)]
cmd/vet: fix typo in documentation
familiy -> family
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/
5783061
Robert Griesemer [Fri, 9 Mar 2012 19:19:10 +0000 (11:19 -0800)]
C+A: added Jeremy Jackins (Individual CLA)
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5783063
Robert Griesemer [Fri, 9 Mar 2012 19:05:50 +0000 (11:05 -0800)]
go/printer, gofmt: nicer formatting of multi-line returns
This affects corner (test) cases only; gofmt -w src misc
doesn't cause any changes.
- added additional test cases
- removed doIndent parameter from printer.valueSpec
(was always false)
- gofmt -w src misc causes no changes
Fixes #1207.
R=dsymonds, rsc
CC=golang-dev
https://golang.org/cl/
5786060
Russ Cox [Fri, 9 Mar 2012 05:10:34 +0000 (00:10 -0500)]
runtime: move runtime.write back to C
It may have to switch stacks, since we are calling
a DLL instead of a system call.
badcallback says where it is, because it is being called
on a Windows stack already.
R=golang-dev, alex.brainman
CC=golang-dev
https://golang.org/cl/
5782060
Mikio Hara [Fri, 9 Mar 2012 02:50:38 +0000 (11:50 +0900)]
net: fix typo
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/
5782063
Andrew Gerrand [Fri, 9 Mar 2012 01:57:38 +0000 (12:57 +1100)]
misc/dist: produce a zip file under windows
Updates #3254.
R=golang-dev, bradfitz, alex.brainman
CC=golang-dev
https://golang.org/cl/
5783058
Russ Cox [Thu, 8 Mar 2012 22:30:45 +0000 (17:30 -0500)]
go/build: add NoGoError
R=dsymonds
CC=golang-dev
https://golang.org/cl/
5781063