Kent R. Spillner said:
> Devel is an obvious choice but it's awfully crowded.  Would it make sense
> to put this in textproc instead?

Technically it does text processing, but it is not a processing system
one would expect to find under textproc.  I don't really care that much
though.

> > WANTLIB += c pthread
> 
> Tabs here instead?

It is commonly pasted verbatim from "make port-lib-depends-check"
output.

> > BUILD_DEPENDS =             lang/go
> > RUN_DEPENDS =               lang/go
> 
> Does it require Go at runtime?

Technically it doesn't, but it won't show documentation for go unless it
is installed.  I placed this dependency to reduce surprise factor.

FWIW it is unlikely to be a big deal: go documentation makes little
sense without go itself, and this port makes little sense as server-side
facility, as it will only document installed packages.

> Also, format pkg/DESCR with fmt -72,

Sure.

> and you might want to add an ONLY_FOR_ARCHS = $(GO_ARCHS).

I'm not sure about this.  The port depends on lang/go, so its build
dependencies will only be satisfied on $(GO_ARCHS) only.

-- 
Dmitrij D. Czarkoff

Attachment: godoc.tgz
Description: godoc.tgz

Reply via email to