Mk/bsd.command.mk: missing CSH tag
Rob Farmer
rfarmer at predatorlabs.net
Thu Nov 18 17:18:41 UTC 2010
On Thu, Nov 18, 2010 at 08:13, Eir Nym <eirnym at gmail.com> wrote:
> On 18 November 2010 19:08, Jason Helfman <jhelfman at e-e.com> wrote:
>> Wrote a quick patch:
>> --- bsd.commands.mk.orig 2010-11-18 08:05:47.000000000 -0800
>> +++ bsd.commands.mk 2010-11-18 08:06:17.000000000 -0800
>> @@ -76,6 +76,7 @@
>> SED?= /usr/bin/sed
>> SETENV?= /usr/bin/env
>> SH?= /bin/sh
>> +CSH?= /bin/csh
>> SORT?= /usr/bin/sort
>> STRIP_CMD?= /usr/bin/strip
>> # Command to run commands as privileged user
>>
The patch should probably retain alphabetical order.
>
> Your patch should check if tcsh is in system. it is optional component.
Why? What do you suggest happens if it isn't found? Nothing will break
except stuff that uses the variable and it would use the variable
because it needs csh. It is part of the base system (/bin/csh and
/bin/tcsh) for all supported releases.
--
Rob Farmer
More information about the freebsd-ports
mailing list