]> Cypherpunks repositories - gostls13.git/log
gostls13.git
13 years agoos: IsNotExist() should also consider ERROR_PATH_NOT_FOUND on Windows
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

13 years agomisc/dist: don't lose mode bits when setting tar permissions
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

13 years agomisc/dist: remove exp and old before building
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

13 years agomisc/vim: restore fileencodings.
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

13 years agomisc/dist: force modes to 0755 or 0644 in tarballs
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

13 years agohtml/template: fix nil pointer bug
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

13 years agoruntime/cgo: linux signal masking
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

13 years agoruntime/cgo: darwin signal masking
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

13 years agocmd/godoc: s/ignore/appengine/ in appinit.go +build constraint
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

13 years agogodoc: fix vet warnings for untagged struct literals
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

13 years agodoc: add Go Concurrency Patterns: Timing out, moving on article
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

13 years agoall: various typos
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

13 years agodoc: add Go image/draw package article and convert code snippets to Go1.
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

13 years agohtml/template: fix panic on Clone
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

13 years agotext/template: variables do not take arguments
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

13 years agogo/printer, gofmt: fix multi-line logic
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

13 years agofmt: remove dead code
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

13 years agotext/template: fix a couple of parse bugs around identifiers.
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

13 years agogo/build: fix match
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

13 years agogc: use quoted string format in import error
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

13 years agomake.bat: properly handle directories with spaces
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

13 years agoos: remove document duplication in error predicate functions
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

13 years agotag weekly.2012-03-13
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

13 years agoweekly.2012-03-13 weekly.2012-03-13
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

13 years agodoc: update weekly notes, untag weekly
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

13 years agogc: allow ~ in import paths
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

13 years agomisc/dist: better archive/tar Headers
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

13 years agogc: include full text of import path in error message
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

13 years agomisc/dist: trim space from version string
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

13 years agotag weekly.2012-03-12
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

13 years agoweekly.2012-03-12
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

13 years agodoc: increase h3 size
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

13 years agodoc: windows zip archive installation instructions
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

13 years agodoc: add Go's declaration syntax article
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

13 years agogo spec: fix inaccuracy in type identity definition
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

13 years agodoc: update reference gopher
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

13 years agoruntime: fix windows/amd64 exception handler
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

13 years agocmd/go: fix typo
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

13 years agogodoc: added alt and title attributes to the "pop out" button
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

13 years agobuild: update comment about GO_LDFLAGS.
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

13 years agotest: actually run them on windows
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

13 years agoos, syscall: windows really isn't posix compliant, fix os.IsExist()
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

13 years agodoc/progs: skip cgo1 and cgo2 on freebsd
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

13 years agogodoc: change link styles, add 'pop out' button
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

13 years agogodoc: use elastic layout for most pages
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

13 years agocmd/godoc: remove sync code
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

13 years agogodoc: fix codewalk handler
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

13 years agodoc: add C? Go? Cgo! article
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

13 years agoos/exec: fix typo in documentation
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

13 years agoundo CL 5754088 / cae9a7c0db06
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

13 years agocmd/go: respect $GOBIN always
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

13 years agoruntime: fix windows/amd64
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

13 years agocmd/go: make go get new.code/... work
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

13 years agocmd/go: make net/... match net too
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

13 years agolog: fix doc comment for Ldate
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

13 years agocmd/fix: add rules for net/http -> net/http/httputil renames
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

13 years agoos: document FileInfo.Size as system-dependent for irregular files
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

13 years agoruntime: print error on receipt of signal on non-Go thread
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

13 years agobuild: build correct cmd/dist matching GOHOSTARCH
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

13 years agobuild: update Makefile to track source code dependencies better
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

13 years agogc: correct comment in runtime.go
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

13 years agocmd/go: allow ssh tunnelled bzr, git and svn
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

13 years agocodereview: fix for Mercurial 2.1
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

13 years agohtml/template: doc nit
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

13 years agoA+C: add Ingo Oeser (individual CLA)
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

13 years agonet/http: return appropriate errors from ReadRequest
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

13 years agocmd/godoc: fix directory read
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

13 years agocmd/go: stop using $GOROOT and $GOBIN in script output
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

13 years agoarchive/tar: catch short writes.
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

13 years agomisc/dist: use archive/tar to generate tarballs
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

13 years agocmd/godoc: always include /doc files in union filesystems
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

13 years agomisc/dist: use archive/zip, seek out windows deps, add --upload flag
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

13 years agocrypto/tls, fmt: print fixes
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

13 years agocmd/vet: little tweak to tagged literal script to get it closer to drop-in format.
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

13 years agomisc/dist: minimum target requirement is 10.6 for Darwin
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

13 years agocmd/api: work on Windows again, and make gccgo files work a bit more
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

13 years agocmd/api: set compiler for all build contexts.
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

13 years agocmd/go: local import fixes
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

13 years agodatabase/sql: ensure Stmts are correctly closed.
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

13 years agodoc: fix typo in contribute.html
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

13 years agoA+C: Add Gwenael Treguier (Individual CLA)
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

13 years agodatabase/sql: fix double connection free on Stmt.Query error
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

13 years agonet/mail: close minor TODO that was waiting on a 6g bug fix.
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

13 years agonet/http: fix crash with Transport.CloseIdleConnections
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

13 years agoarchive/zip: verify CRC32s in non-streamed files
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

13 years agocrypto/x509: do not forget to free cert context
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

13 years agoarchive/zip: write data descriptor signature for OS X; fix bugs reading it
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

13 years agogo/printer: example for Fprint
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

13 years agocmd/gc: import path cannot start with slash on Windows
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

13 years agocmd/dist, cmd/go: move CGO_ENABLED from 'go tool dist env' to 'go env'
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

13 years agocmd/vet: fix typo in documentation
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

13 years agoC+A: added Jeremy Jackins (Individual CLA)
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

13 years agogo/printer, gofmt: nicer formatting of multi-line returns
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

13 years agoruntime: move runtime.write back to C
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

13 years agonet: fix typo
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

13 years agomisc/dist: produce a zip file under windows
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

13 years agogo/build: add NoGoError
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

13 years agoruntime: fix windows build
Russ Cox [Thu, 8 Mar 2012 20:53:11 +0000 (15:53 -0500)]
runtime: fix windows build

Implement runtime·write, like on the other systems,
and also runtime·badcallback, in assembly to reduce
stack footprint.

TBR=golang-dev
CC=golang-dev
https://golang.org/cl/5785055

13 years agocmd/go: document import path meta tag discovery in go help remote
Brad Fitzpatrick [Thu, 8 Mar 2012 19:31:52 +0000 (11:31 -0800)]
cmd/go: document import path meta tag discovery in go help remote

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5787059

13 years agocmd/go: add env command, use to fix misc/cgo/testso
Russ Cox [Thu, 8 Mar 2012 19:28:44 +0000 (14:28 -0500)]
cmd/go: add env command, use to fix misc/cgo/testso

Fixes 386 build on 64-bit machines.

R=golang-dev, bradfitz, minux.ma
CC=golang-dev
https://golang.org/cl/5785053