cmd/go: fix two bugs

Issue 3207 was caused by setting GOPATH=GOROOT.
This is a common mistake, so diagnose it at command start
and also correct the bug that it caused in get (downloading
to GOROOT/src/foo instead of GOROOT/src/pkg/foo).

Issue 3268 was caused by recognizing 'packages' that
had installed binaries but no source.  This behavior is not
documented and causes trouble, so remove it.  We can
revisit the concept of binary-only packages after Go 1.

Fixes #3207.
Fixes #3268.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5930044

doc: use relative links in Laws of Reflection article

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5924050

doc: use relative links in draw package article

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5924051

doc: update wiki tutorial templates, and template discussion

Fixes #3384.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5915044

path/filepath: use windows GetShortPathName api to force GetLongPathName to do its work

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

cmd/go: update for go1 tag format

R=golang-dev, r, r
CC=golang-dev
https://golang.org/cl/5919048

cmd/dist: fix detection of go1 version

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5924044

doc: update licensing text one more time

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5916046

net: ignore ECONNABORTED from syscall.Accept

Fixes #3395.

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

undo CL 5844051 / 5d0322034aa8

Breaks closure test when GOMAXPROCS=2 or more.

««« original CL description
runtime: restore deadlock detection in the simplest case.

Fixes #3342.

R=iant, r, dave, rsc
CC=golang-dev, remy
https://golang.org/cl/5844051

»»»

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

cmd/go: copy tag_test.go from goinstall

hg cat -r 11846 src/cmd/goinstall/tag_test.go >tag_test.go
No changes.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5919047

misc/emacs: fix overindentation caused by mis-parsing lines ending with special chars

Fixes #3313

go-mode-backward-skip-comments is probably due for a more ambitious refactoring --- it repeats guard conditions after every nearly every movement of point.

R=sameer, r
CC=golang-dev
https://golang.org/cl/5844063

C: add Ben Fried, Googler

R=golang-dev, dsymonds
CC=ben.fried, golang-dev
https://golang.org/cl/5869043

runtime: restore deadlock detection in the simplest case.

Fixes #3342.

R=iant, r, dave, rsc
CC=golang-dev, remy
https://golang.org/cl/5844051

doc: fix concurrency patterns rawhtml generation

The Makefile target was broken due to the rename of the HTML file.

R=golang-dev
CC=golang-dev
https://golang.org/cl/5918045

doc: move /ref/cmd to /doc/cmd

Fixes #3400.

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

doc: add JSON-RPC: a tale of interfaces article

Originally published on The Go Programming Language Blog, Abril 27, 2010.

http://blog.golang.org/2010/04/json-rpc-tale-of-interfaces.html

R=adg, r
CC=golang-dev
https://golang.org/cl/5920044

doc: update footer text

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

path/filepath: convert drive letter to upper case in windows EvalSymlinks

Fixes #3347.

R=golang-dev, aram, r, rsc
CC=golang-dev
https://golang.org/cl/5918043

doc: remove cov and prof from /ref/cmd

Update #3400

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5921044

cmd/godoc: canonicalize custom path redirects

For example, /ref and /doc/reference.html now both redirect to /ref/.

Fixes #3401.

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

doc: move gdb doc back to /doc/ and de-emphasize on references page

Fixes #3399.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5919044

doc: add doc/articles/index.html

Fixes #3402.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5923043

doc: describe the Windows MSI installer as experimental

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5921043

misc/goplay: fix error on IE8.
        use cancelBubble=true instead of preventDefault().

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

misc/dist: don't ship codereview either

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5917043

cmd/godoc: fix app engine version

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

misc/dist: don't include old python dashboard in releases

I'd prefer not to ship a bunch of bad Python code to everyone who uses
Go 1.

R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5915043

doc: add Terms of Service

R=r
CC=golang-dev
https://golang.org/cl/5913044

exp/types: generalized GCImporter API.

- Renamed ExportData -> FindGcExportData
  and base it on an a bufio.Reader rather
  than a filename so it can be used in
  environments where object files are
  stored elsewhere.

- Factor former GcImporter into GcImportData
  and GcImport. Implementations with different
  storage locations for object files can build
  a customized GcImport using GcImportData.

This is pkg/exp only - no impact on Go 1.

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

doc: Change id of footer

The heading "Copyright" uses id="copyright" as the anchor name.
However, there is also a <div id="copyright"> at the bottom.
Using the same ID value twice in the same file is illegal
(http://www.w3.org/TR/html401/struct/links.html#h-12.2.3).

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

cmd/go: explain versions better
Summarize the desiderata for selecting versions of remote packages to get.
Fixes #3394.

R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/5902058

misc/dist: add -wxs flag to provide custom installer.wxs file

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5908055

run.bash: set -e in new codewalk block
Otherwise we won't fail if something goes wrong.
This shell programming stuff is tricky.

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5905062

windows: install fixes

* set default installation drive to C:\
* remove Win64 component property

R=golang-dev, bradfitz, aram
CC=golang-dev
https://golang.org/cl/5901044

run.bash: compile the codewalks
They could be tested but that requires more than seems wise right now.

Update #2648.

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5908054

godoc: make 'Overview' section collapsable

This makes packages with lengthly package comments easier to browse.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5901055

doc: make installation instructions more skim-friendly

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5908052

cmd/go: allow underscores in tool name
        Otherwise we can't invoke go_bootstrap directly.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5900061

go/build: cgoEnabled is not known to cmd/dist anymore

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5901051

cmd/go: add missing error check

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

doc/reference-cmd: use vet as example, not fmt
"go tool fmt" doesn't work, "go tool vet" does.

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

effective_go: cleanups and fixes
Also explain the situation with recursive String methods more accurately,
and clean up the code now that the fmt package is more careful.

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

docs/articles/wiki: minor fixes

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5900046

doc/go1: add mention of packaged releases
Fixes #3245.

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5905045

doc/reference-cmd.html: new introduction
Compare and explain go, go cmd, and go tool cmd.
The situation is not concisely explained elsewhere in the main docs.

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5906043

os: add missing byte to FileMode buffer

32 bytes is enough for all FileMode bits.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5853044

codewalk/sharemem.xml: fix references to files

R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/5877064

text/template: fix typo in package comment

Fixes #3383.

R=iant, bradfitz
CC=golang-dev
https://golang.org/cl/5891045

doc/reference-cmd: doc/fmt can be invoked by cmd/go but also available as independent command

R=golang-dev, iant, r
CC=golang-dev
https://golang.org/cl/5877050

crypto/tls: don't select ECC ciphersuites with no mutual curve.

The existing code that tried to prevent ECC ciphersuites from being
selected when there were no mutual curves still left |suite| set.
This lead to a panic on a nil pointer when there were no acceptable
ciphersuites at all.

Thanks to George Kadianakis for pointing it out.

R=golang-dev, r, bradfitz
CC=golang-dev
https://golang.org/cl/5857043

doc/articles/defer_panic_recover.html: minor tweaks
Delete () from function names and change the reference to some
functions to the correct term, methods.

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5874063

doc/articles/image_draw.html: fix circle example
It was showing the same snippet twice instead of the type definition and snippet.

R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/5875045

path/filepath: windows drive letter cannot be a digit

R=golang-dev, r
CC=golang-dev, mattn.jp
https://golang.org/cl/5885056

tag weekly.2012-03-22

R=golang-dev
CC=golang-dev
https://golang.org/cl/5877063

misc/dist: make godoc shortcut work

R=golang-dev
CC=golang-dev
https://golang.org/cl/5877062

doc/godoc.js: fix error on IE8.
        * implement simple getElementsByClassName for IE8.
        * remove some lint warnings.

Fixes #3318.

R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/5881054

weekly.2012-03-22

R=golang-dev, r, dave
CC=golang-dev
https://golang.org/cl/5876068

misc/dist: updates to installer script

Now sets GOROOT.

Fixes #3287.
Fixes #3361.

R=golang-dev
CC=golang-dev
https://golang.org/cl/5877059

doc/articles/c_go_cgo.html: correct "C" comment to mention #cgo

Also fix invalid apostrophe characters.

R=r, gri, adg
CC=golang-dev
https://golang.org/cl/5874058

doc/articles: rename concurrency patterns article
The old name, automatically generated, was ludicrously verbose.
Also clean up an example to use time.Second.

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5876051

test: fix run.bash by spelling out the commands to use for bug424.go

R=golang-dev, gri, iant
CC=golang-dev
https://golang.org/cl/5882046

cmd/gc: fix comment typo, assignment spacing

R=bradfitz, gri, lvd
CC=golang-dev
https://golang.org/cl/5874053

cmd/gc: when expanding append inline, preserve arguments

Fixes #3369.

R=golang-dev, gri, lvd, r
CC=golang-dev
https://golang.org/cl/5876044

sort: add time complexity to doc

Let's tell the world that Go's sort is O(n log n).
Surely this is a feature we intend to keep.

R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5867045

doc/articles/gobs_of_data.html: delete extra word
Putt putt putt our way towards felicity.

R=golang-dev, bsiegert
CC=golang-dev
https://golang.org/cl/5874048

doc: add JSON and Go article

Originally published on The Go Programming Language Blog, January 25, 2011.

http://blog.golang.org/2011/01/json-and-go.html

R=adg
CC=golang-dev
https://golang.org/cl/5846044

doc/articles/go_command.html: nits
Fix some English mistakes and minor inaccuracies.

R=golang-dev, jsing
CC=golang-dev
https://golang.org/cl/5885046

flag: add examples

R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5867049

doc: fix typo in The Laws of Reflection article

R=golang-dev, gri, r
CC=golang-dev
https://golang.org/cl/5876047

exp/norm/normalize.go: fix typo

R=golang-dev, r, dsymonds
CC=golang-dev
https://golang.org/cl/5874045

doc/effective_go.html: undo local font change

R=r
CC=golang-dev
https://golang.org/cl/5876043

go/build: clarify why we exclude files starting with '_' or '.'

R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5864053

godoc: use shorter titles for tabs

In a browser with many open tabs, the tab titles become short
and uninformative because they all start with the same prefix
("Package ", "Directory ", etc.).

Permit use of shorter tab titles that start with the relevant
information first.

Fixes #3365.

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5865056

test: use testlib in a few more cases
        Introduce a new skip cmd.

R=golang-dev, bradfitz, iant, iant
CC=golang-dev
https://golang.org/cl/5868048

cmd/cgo: fix handling of errno for gccgo

Fixes #3332.

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5868047

doc/effective_go: minor corrections

R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5848063

build: lengthen timeout for the lengthy runtime test
        To fix build on (overheated) Linux/ARM builder.

R=golang-dev, dave
CC=golang-dev
https://golang.org/cl/5865051

doc: remove reference to deleted os.ENOSPC

R=golang-dev, gri, r, r
CC=golang-dev
https://golang.org/cl/5866046

html/template: fix typo and make grammar consistent in comments.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5866044

sort: fix computation of maxDepth to avoid infinite loop

The current computation loops indefinitely if n > 1<<30 (for 32-bit ints).

R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5848067

cmd/go: use .o, not .{5,6,8}, for gccgo created object files

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5861044

doc: general update of gccgo_install

Fixes #3325.

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

sort: document two undocumented functions

They looked out of place in godoc.
Includes documenting sort stability.

Fixes #3356

R=golang-dev, gri, trolleriprofessorn
CC=golang-dev
https://golang.org/cl/5855044

misc/cgo/gmp: update for Go 1
1. make the program go buildable
2. update os.EINVAL and runtime.Cgocalls()
3. wrap mpz_div_2exp() and mpz_mul_2exp to support both
   pre-5.0 and post-5.0 gmp (we really have no reason to
   restrict ourselves to gmp 5.0+)

R=golang-dev, remyoudompheng, iant
CC=golang-dev
https://golang.org/cl/5847061

runtime: remove unused goc2c.c
        Its functionality has been moved into cmd/dist.

R=golang-dev, r, iant
CC=golang-dev
https://golang.org/cl/5843062

build: unset GOPATH before tests
This is because we disallow local import for non-local packages, if
GOROOT happens to be under one of GOPATH, then some tests will fail
to build.
Fixes #3337.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5852043

doc/debugging_with_gdb: format & content update

R=adg, lvd
CC=golang-dev
https://golang.org/cl/5845065

run.bat: disable test in test\bench\go1 to fix build

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5848066

api: update go1.txt after adding functions to html/template
Fixes build, teaches lesson.

R=golang-dev, r
TBR=bradfitz
CC=golang-dev
https://golang.org/cl/5853051

html/template: add Templates and *Escape functions
to bring it in line with text/template's interface.
Fixes #3296.

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5843066

cmd/godoc: inform users that the playground doesn't work via local godoc

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5843065

doc: remove defunct playground.html

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5846071

build: do more during windows build

- use GO_GCFLAGS and GO_LDFLAGS if supplied
- build misc\dashboard\builder and misc\goplay
- run tests in test\bench\go1
- check api compatibility

R=golang-dev, r, kardianos, bradfitz
CC=golang-dev
https://golang.org/cl/5847063

doc: replace mentions of 6g with gc or the go command

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5846070

misc/dashboard: remove remnants of package dashboard

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5847070

net: drop unnecessary type assertions and fix leak in test

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5847064

gc: fix struct and array comparisons for new bool rules

The two optimizations for small structs and arrays
were missing the implicit cast from ideal bool.

Fixes #3351.

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

crypto/tls: always send a Certificate message if one was requested.

If a CertificateRequest is received we have to reply with a
Certificate message, even if we don't have a certificate to offer.

Fixes #3339.

R=golang-dev, r, ality
CC=golang-dev
https://golang.org/cl/5845067

doc: update format for "C? Go? Cgo!" article

R=adg
CC=golang-dev
https://golang.org/cl/5841050