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

[Adam Weinberger]

> On 19 May, 2015, at 15:15, Mathieu Arnold <mat at FreeBSD.org> wrote:
> 
> 
> 
> +--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.

It says to use full path when it's not in $PATH. What it doesn't say is to use full path ONLY when it's not in $PATH :-)

# Adam


-- 
Adam Weinberger
adamw at adamw.org
http://www.adamw.org