Matthew Burgess wrote:
> On 02/08/2011 03:15, bdu...@linuxfromscratch.org wrote:
>> Author: bdubbs
>> Date: 2011-08-01 20:15:42 -0600 (Mon, 01 Aug 2011)
>> New Revision: 9574
> 
> <snip>
> 
>> Modified: trunk/BOOK/bootscripts/lfs/init.d/sysklogd
> 
> This hunk...
> 
>>   case "${1}" in
>> -    start)
>> -            boot_mesg "Starting system log daemon..."
>> -            loadproc syslogd -m 0
>> +   start)
>> +      boot_mesg "Starting system log daemon..."
>> +      PARMS=${SYSKLOGD_PARMS=-'-m 0'}
>> +      loadproc syslogd $PARMS
> 
> ...appears to cause a failure for me:
> 
> syslogd: invalid option -- '-'
> 
> I think what you're trying to do here (but have possibly made a typo of 
> '=' instead of ':') is:

Actually it should be PARMS=${SYSKLOGD_PARMS-'-m 0'}

Try this short script:

#!/bin/bash
SYSKLOGD_PARMS=''
PARMS=${SYSKLOGD_PARMS-'-m 0'}
echo $PARMS


> 1) Set PARMS = $SYSKLOGD_PARMS, if that variable exists, regardless of 
> whether it is null.
> 2) Set PARMS to '-m 0' if $SYSKLOGD_PARMS is not set at all

Exactly.  I've fixed it, but will commit tomorrow.

   -- Bruce
-- 
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page

Reply via email to