Re: [LTP] [PATCH v2] cron02: fix a bug of /etc/init.d/crond is nonexistent

[Wanlong Gao <gaowanlong@cn.fujitsu.com>]

On 03/30/2012 09:07 AM, Garrett Cooper wrote:

> On Thu, Mar 29, 2012 at 6:05 PM, Peng Haitao <penght@cn.fujitsu.com> wrote:
>> /etc/init.d/crond is nonexistent in fedora16, so should use systemctl.
>>
>> Signed-off-by: Peng Haitao <penght@cn.fujitsu.com>
>> ---
>>  testcases/commands/cron/cron02 |   14 ++++++++++++++
>>  1 file changed, 14 insertions(+)
>>
>> diff --git a/testcases/commands/cron/cron02 b/testcases/commands/cron/cron02
>> index 0c05c95..d348936 100755
>> --- a/testcases/commands/cron/cron02
>> +++ b/testcases/commands/cron/cron02
>> @@ -52,12 +52,26 @@ do_setup(){
>>         }
>>         fi
>>
>> +       # running under systemd?
>> +       if command -v systemctl >/dev/null 2>&1; then
>> +               HAVE_SYSTEMCTL=true
>> +       else
>> +               HAVE_SYSTEMCTL=false
>> +       fi
>> +
>>        # restart cron daemon
>>        # Red Hat uses crond, SuSE/Other uses cron.
>>        if [ -x /etc/init.d/crond ]; then
>>                /etc/init.d/crond restart
>>        elif [ -x /etc/init.d/cron ]; then
>>                /etc/init.d/cron restart
>> +       elif $HAVE_SYSTEMCTL; then
>> +               for crond in "crond" "cron"; do
>> +                       if systemctl is-enabled $crond.service >/dev/null 2>&1; then
>> +                               systemctl restart $crond.service
>> +                               break
>> +                       fi
>> +               done
>>        else
>>                echo "Could not determine cron init.d script."
>>                exit 1
> 
> Reviewed-by: Garrett Cooper <yanegomi@gmail.com>


pushed, thanks Peng, Garrett,

Wanlong Gao

> 
> ------------------------------------------------------------------------------
> This SF email is sponsosred by:
> Try Windows Azure free for 90 days Click Here 
> http://p.sf.net/sfu/sfd2d-msazure
> _______________________________________________
> Ltp-list mailing list
> Ltp-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ltp-list
> 



------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list