code review 6846119: godoc: remove main package if main and once more packag...

godoc: remove main package if main and once more package exists.
Below should work.
$ godoc net/http

[mattn, 2012-11-28 02:45:14]

Hello golang-dev@googlecode.com (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
http://go.googlecode.com/hg/

[rsc, 2012-11-29 03:20:53]

NOT LGTM

'godoc net/http' works fine for me already.
We used to have logic like this in godoc and it was very confusing,
because if you were in, say, cmd/gofmt, and you accidentally created
a file saying

    package p

then it took precedence over gofmt (package main) and then 'godoc
gofmt' stopped working entirely.

If you want to put 'package main' helpers in a non-main package
directory, that's fine: just put
    // +build ignore
at the top of the file.

Russ

[minux1, 2012-11-29 04:53:51]

On Thu, Nov 29, 2012 at 11:20 AM, Russ Cox <rsc@golang.org> wrote:

> 'godoc net/http' works fine for me already.
>
i think he talks about godoc on windows.
C:\go\>godoc net/http
2012/11/29 12:34:54 /target contains more than one package: http, main

This is strange as `godoc net/http` work perfectly under Unix.
I guess the real cause is that somehow godoc doesn't recognize the "//
+build ignore"
in src/pkg/net/http/triv.go; this will be a real bug to fix.

[minux1, 2012-11-29 06:04:58]

Found real problem, sent http://codereview.appspot.com/6852100

[khr, 2013-07-22 19:45:13]

R=close