Date: Thu, 13 Apr 2017 13:33:15 -0700 From: Maxim Sobolev <sobomax@freebsd.org> To: Jan Beich <jbeich@freebsd.org> Cc: Alexey Dokuchaev <danfe@freebsd.org>, svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org Subject: Re: svn commit: r438413 - in head/devel: . debugedit debugedit/files Message-ID: <CAH7qZfv%2B6m1a2d7DMo2gqabHXY0o_ANFeOxvWoZbSbCVzTpeTw@mail.gmail.com> In-Reply-To: <8tn4-6sgl-wny@FreeBSD.org> References: <201704122049.v3CKnQS9032104@repo.freebsd.org> <20170413150057.GB86375@FreeBSD.org> <8tn4-6sgl-wny@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Yes, also I think use of this option should be reasonable. If it's just
1.5k file just let it be, don't need to prompt poor user if he wants it or
not. Just install the damn thing.
-Max
On Thu, Apr 13, 2017 at 12:03 PM, Jan Beich <jbeich@freebsd.org> wrote:
> Alexey Dokuchaev <danfe@FreeBSD.org> writes:
>
> >> + ${INSTALL_MAN} ${WRKSRC}/README.txt ${STAGEDIR}${DOCSDIR}
> >
> > INSTALL_MAN is not for docs, it's for manpages. Also, when installing
> docs,
> > it's a good idea to OPTIONS_DEFINE+=DOCS and use
> {do|post}-install-DOCS-on
> > special helper target.
>
> PHB disagrees with you:
>
> - INSTALL_MAN is a command to install manpages and other documentation
> (it does not compress anything).
>
> https://www.freebsd.org/doc/en/books/porters-handbook/install.html
>
>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAH7qZfv%2B6m1a2d7DMo2gqabHXY0o_ANFeOxvWoZbSbCVzTpeTw>