svn commit: r443599 - head/Mk
Adam Weinberger
adamw at adamw.org
Thu Jun 15 01:44:53 UTC 2017
> On 14 Jun, 2017, at 18:55, Alexey Dokuchaev <danfe at FreeBSD.org> wrote:
>
> On Wed, Jun 14, 2017 at 08:18:38PM +0000, Tobias C. Berner wrote:
>> New Revision: 443599
>> URL: https://svnweb.freebsd.org/changeset/ports/443599
>>
>> Log:
>> Add note, that WRKDIRPREFIX must not end in '/'.
>>
>> PR: 199297
>
> This is stupid, fragile, error-prone appoach that does not even cover all
> cases (like shown in the PR) when no paths end in slash but the problem
> still occurs. Did you read the analysis of my original problem carefully
> enough? It was not about slashes, it was symlinks and realpath.
>
> You might not like proposed solution but then think of something better;
> computers are smart enough to handle slashes automagically rather than
> depend on that unrealiable human beings will follow the rules and notes.
Okay, but WRKDIRPREFIX still shouldn't end in a slash though, and a comment saying not to put a slash at the end is a pretty reasonable move.
# Adam
--
Adam Weinberger
adamw at adamw.org
https://www.adamw.org
More information about the svn-ports-all
mailing list