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

[Mathieu Arnold]

+--On 19 mai 2015 15:23:54 -0600 Adam Weinberger <adamw at adamw.org> wrote:
|> 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/mak
|> | efi 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 :-)

Yes, yes, I get it, I'll change the example so that the one using a full
path is *not* an executable :-)

-- 
Mathieu Arnold