New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/go: remove the doc command #4849

Closed
gopherbot opened this Issue Feb 19, 2013 · 10 comments

Comments

Projects
None yet
7 participants
@gopherbot

gopherbot commented Feb 19, 2013

by jjandroman:

What steps will reproduce the problem?
1.go doc fmt Printf


What is the expected output?
the same as in godoc fmt Printf

What do you see instead?
An error message stating that `can't load package: package Printf: import
"Printf": cannot find package´ and after that, the whole output of fmt
documentation.

Which compiler are you using (5g, 6g, 8g, gccgo)?


Which operating system are you using?
Windows 7 x64

Which version are you using?  (run 'go version')
go1.0.3

Please provide any additional information below.
@dominikh

This comment has been minimized.

Show comment
Hide comment
@dominikh

dominikh Feb 20, 2013

Member

Comment 1:

That'd break backwards compatibility though: 
usage: go doc [-n] [-x] [packages]
Doc runs the godoc command on the packages named by the import paths.
There's probably more than one case where that feature is actively being used.
Member

dominikh commented Feb 20, 2013

Comment 1:

That'd break backwards compatibility though: 
usage: go doc [-n] [-x] [packages]
Doc runs the godoc command on the packages named by the import paths.
There's probably more than one case where that feature is actively being used.
@griesemer

This comment has been minimized.

Show comment
Hide comment
@griesemer

griesemer Feb 20, 2013

Contributor

Comment 2:

We should consider accepting godoc pkg.x (instead of godoc pkg x), and then go doc pkg.x
might work as well.
Contributor

griesemer commented Feb 20, 2013

Comment 2:

We should consider accepting godoc pkg.x (instead of godoc pkg x), and then go doc pkg.x
might work as well.
@minux

This comment has been minimized.

Show comment
Hide comment
@minux

minux Feb 20, 2013

Member

Comment 3:

agree. simple copy and paste will result in that kind of command line.
Member

minux commented Feb 20, 2013

Comment 3:

agree. simple copy and paste will result in that kind of command line.
@rsc

This comment has been minimized.

Show comment
Hide comment
@rsc

rsc Feb 20, 2013

Contributor

Comment 4:

'go doc', like all the other go commands, only accepts package lists (or a
set of files).
it has limited utility. it might be better to delete the 'go doc' command.
russ
Contributor

rsc commented Feb 20, 2013

Comment 4:

'go doc', like all the other go commands, only accepts package lists (or a
set of files).
it has limited utility. it might be better to delete the 'go doc' command.
russ
@rsc

This comment has been minimized.

Show comment
Hide comment
@rsc

rsc Mar 12, 2013

Contributor

Comment 5:

[The time for maybe has passed.]

Labels changed: removed go1.1maybe.

Contributor

rsc commented Mar 12, 2013

Comment 5:

[The time for maybe has passed.]

Labels changed: removed go1.1maybe.

@adg

This comment has been minimized.

Show comment
Hide comment
@adg

adg Mar 18, 2013

Contributor

Comment 6:

Labels changed: added godoc.

Contributor

adg commented Mar 18, 2013

Comment 6:

Labels changed: added godoc.

@adg

This comment has been minimized.

Show comment
Hide comment
@adg

adg Mar 22, 2013

Contributor

Comment 7:

I always thought "go doc" was pretty useless (at least I never use it). I'd be in favor
of removing it.

Labels changed: added priority-later, removed priority-triage.

Status changed to Accepted.

Contributor

adg commented Mar 22, 2013

Comment 7:

I always thought "go doc" was pretty useless (at least I never use it). I'd be in favor
of removing it.

Labels changed: added priority-later, removed priority-triage.

Status changed to Accepted.

@rsc

This comment has been minimized.

Show comment
Hide comment
@rsc

rsc Jul 30, 2013

Contributor

Comment 8:

Labels changed: added go1.2.

Contributor

rsc commented Jul 30, 2013

Comment 8:

Labels changed: added go1.2.

@robpike

This comment has been minimized.

Show comment
Hide comment
@robpike

robpike Aug 15, 2013

Contributor

Comment 9:

Owner changed to @robpike.

Status changed to Started.

Contributor

robpike commented Aug 15, 2013

Comment 9:

Owner changed to @robpike.

Status changed to Started.

@robpike

This comment has been minimized.

Show comment
Hide comment
@robpike

robpike Aug 16, 2013

Contributor

Comment 10:

This issue was closed by revision 71eae5a.

Status changed to Fixed.

Contributor

robpike commented Aug 16, 2013

Comment 10:

This issue was closed by revision 71eae5a.

Status changed to Fixed.

@rsc rsc added this to the Go1.2 milestone Apr 14, 2015

@rsc rsc removed the go1.2 label Apr 14, 2015

@golang golang locked and limited conversation to collaborators Jun 24, 2016

This issue was closed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.