svn commit: r386737 - in head/security: . tmux-cssh

[Mathieu Arnold]

+--On 19 mai 2015 10:15:36 -0600 Adam Weinberger <adamw at adamw.org> wrote:
|> On 19 May, 2015, at 10:04, Baptiste Daroussin <bapt at FreeBSD.org> wrote:
|> 
|> On Tue, May 19, 2015 at 03:33:07AM +0000, Vanilla I. Shu wrote:
|>> Author: vanilla
|>> Date: Tue May 19 03:33:06 2015
|>> New Revision: 386737
|>> URL: https://svnweb.freebsd.org/changeset/ports/386737
|>> 
|>> 
|>> +RUN_DEPENDS=	${LOCALBASE}/bin/tmux:${PORTSDIR}/sysutils/tmux
|> The dependency is wrong it should be tmux:${PORTSDIR}/sysutils/tmux
|> 
|> Best regards,
|> Bapt
| 
| I've seen that construct popping up a lot lately. I suspect that the
| problem is that
| https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/makefi
| le-depend.html lists ${LOCALBASE}/.../ as the first example.
| 
| Perhaps it would be helpful to add something saying "Only include an
| absolute path if the binary is outside of $PATH, or if the binary also
| exists in base and you specifically want the ports tree version."

There's a note afterwards saying it, but you're right, there should be an
example with only a binary in the path first.

-- 
Mathieu Arnold