code review 5700081: cmd/goman: Adds a new command goman which generates and...

cmd/goman: Adds a new command goman which generates and installs man pages
based on official command documentation in godoc.

Fixes issue 101.

[rcostu, 2012-02-27 13:22:47]

Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com,
iant@golang.org, rsc@golang.org),

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

[rcostu, 2012-03-05 20:51:45]

Ping?

On Mon, Feb 27, 2012 at 14:22, <rcostu@gmail.com> wrote:

> Reviewers: golang-dev_googlegroups.com,
>
> Message:
> Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com,
> iant@golang.org, rsc@golang.org),
>
> I'd like you to review this change to
> https://go.googlecode.com/hg/
>
>
> Description:
> cmd/goman: Adds a new command goman which generates and installs man
> pages
> based on official command documentation in godoc.
>
> Fixes issue 101.
>
> Please review this at
http://codereview.appspot.com/**5700081/<http://codereview.appspot.com/5700081/>
>
> Affected files:
>  A src/cmd/goman/goman.go
>
>
>

[adg, 2012-03-05 23:03:10]

Why not use text/template to do this?

The command should be called 'man', not 'goman'.

[r, 2012-03-06 23:14:26]

This program should just take a package and write a manual page to standard
output. People can script it to do the rest.

[rcostu, 2012-03-07 19:11:35]

On 2012/03/06 23:14:26, r wrote:
> This program should just take a package and write a manual page to standard
> output. People can script it to do the rest.

I'll change the name to man and try to use text/template.

I don't now if it should generate manual pages for packages too. If it is
intended I could do a workaround to get packages and commands working to get the
whole set of manual pages for the language.

I'll submit an update as soon as I have it finished.

[r2, 2012-03-07 20:12:06]

On Mar 8, 2012, at 6:11 AM, rcostu@gmail.com wrote:

> On 2012/03/06 23:14:26, r wrote:
>> This program should just take a package and write a manual page to
> standard
>> output. People can script it to do the rest.
> 
> I'll change the name to man and try to use text/template.
> 
> I don't now if it should generate manual pages for packages too. If it
> is intended I could do a workaround to get packages and commands working
> to get the whole set of manual pages for the language.
> 
> I'll submit an update as soon as I have it finished.
> 
> http://codereview.appspot.com/5700081/

I misspoke. By package, I meant directory. To be able to do this for a package
is a separate question.

-rob