[OE-core] [PATCH 2/6] sysvinit: add init.d/bootlogd status command for LSB compliance
jhuang0
jackie.huang at windriver.com
Tue Aug 20 09:33:46 UTC 2013
On 8/20/2013 2:39 PM, Phil Blundell wrote:
> On Tue, 2013-08-20 at 11:25 +0800, jackie.huang at windriver.com wrote:
>> @@ -37,6 +40,19 @@ case "$0" in
>> ;;
>> esac
>>
>> +# default /var/log/boot is not an option since /var/log becomes
>> +# /var/volatile/log
>> +logfile=/var/local/bootlogd.log
>> +
>> +#
>> +# Create initial log files
>> +#
>> +if [ ! -f "$logfile" ] && touch "$logfile" >/dev/null 2>&1; then
>> + echo "(Nothing has been logged yet.)" >| "$logfile"
>> + chown root:adm "$logfile"
>> + chmod 640 "$logfile"
>> +fi
>> +
>> case "$ACTION" in
>> start)
>> echo -n "Starting $DESC: "
>> @@ -44,9 +60,9 @@ case "$ACTION" in
>> then
>> umask 027
>> start-stop-daemon --start --quiet \
>> - --exec $DAEMON -- -r
>> + --exec $DAEMON -- -r -l $logfile
>> else
>> - $DAEMON -r
>> + $DAEMON -r -l $logfile
>> fi
>> echo "$NAME."
>> ;;
>
> It's not totally obvious that the changes are related to the "status"
> command. If you're creating new logfiles that didn't exist previously
> then I think the commit message ought to explain why.
>
Sorry, my mistake to involve some other patch when I got the patch from
our layer, I will remake the patches.
Thanks,
Jackie
> p.
>
>
>
>
--
Jackie Huang
WIND RIVER | China Development Center
MSN:jackielily at hotmail.com
Tel: +86 8477 8594
Mobile: +86 138 1027 4745
More information about the Openembedded-core
mailing list