5080 2004-08-27 03:22:02 +0400 /sbin/service incorrectly intercepts command line arguments passed to service 2007-04-21 16:22:51 +0400 1 1 4 Development Sisyphus service unstable all Linux CLOSED FIXED P2 normal --- 1 evseev placeholder glebfm ldv legion placeholder vt qa-sisyphus oldest_to_newest 18386 0 evseev 2004-08-27 03:22:02 +0400 /sbin/service looks for command line switches even after service name. So, "service testd -huh" is processed like "service -huh testd", not like "/etc/init.d/testd -huh". My little patch fixes this. --- service.orig 2004-01-25 20:44:48 +0300 +++ service 2004-08-27 03:06:05 +0400 @@ -50,7 +50,9 @@ exit 0 ;; -*) - usage 1 + test -z "$SERVICE" && usage 1 + OPTIONS="$OPTIONS $1" + shift ;; *) if [ $# -eq 2 -a "$2" = "--full-restart" ]; then Steps to Reproduce: /sbin/service anything -anything Actual Results: /sbin/service displays usage and fails. Expected Results: Tail of command line should be passed directly to called service. 18532 1 ldv 2004-09-01 19:38:14 +0400 First, startup scripts usually have no option parser, so your script is probably first one. Second, your patch doesn't make possible to pass .e.g --help option to startup script. 22435 2 767 ldv 2005-03-09 15:46:17 +0300 Created attachment 767 Proposed change I'm going to apply this change to /sbin/service. Objections? 22438 3 ldv 2005-03-09 16:07:20 +0300 Applied in 0.5.7-alt1 767 2005-03-09 15:46:17 +0300 2005-03-09 15:46:17 +0300 Proposed change service.diff text/plain 1266 ldv SW5kZXg6IHNlcnZpY2UKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PQpSQ1MgZmlsZTogL2hvbWUvY3ZzL2N2c3Jvb3Qvc2Vy dmljZS9zZXJ2aWNlLHYKcmV0cmlldmluZyByZXZpc2lvbiAxLjkKZGlmZiAtdSAtcCAtcjEuOSBz ZXJ2aWNlCi0tLSBzZXJ2aWNlCTI1IEphbiAyMDA0IDE3OjQ0OjQ4IC0wMDAwCTEuOQorKysgc2Vy dmljZQk5IE1hciAyMDA1IDEyOjQxOjI3IC0wMDAwCkBAIC00MCwzNiArNDAsMzIgQEAgaWYgWyAk IyAtZXEgMCBdOyB0aGVuCiAJdXNhZ2UgMQogZmkKIAotd2hpbGUgWyAkIyAtZ3QgMCBdOyBkbwot CWNhc2UgIiQxIiBpbgotCQktLWhlbHB8LWgpCi0JCQl1c2FnZSAwCi0JCQk7OwotCQktLXZlcnNp b258LVYgKQotCQkJZWNobyAiJFZFUlNJT04iCi0JCQlleGl0IDAKLQkJCTs7Ci0JCS0qKQotCQkJ dXNhZ2UgMQotCQkJOzsKLQkJKikKLQkJCWlmIFsgJCMgLWVxIDIgLWEgIiQyIiA9ICItLWZ1bGwt cmVzdGFydCIgXTsgdGhlbgotCQkJCVNFUlZJQ0U9IiQxIgotCQkJCWNoZWNrX3NlcnZpY2UKLQkJ CQljZCAvIHx8IGV4aXQKLQkJCQkiJFNFUlZJQ0VESVIvJFNFUlZJQ0UiIHN0b3AKLQkJCQkiJFNF UlZJQ0VESVIvJFNFUlZJQ0UiIHN0YXJ0Ci0JCQkJZXhpdCAkPwotCQkJZWxpZiBbIC16ICIkU0VS VklDRSIgXTsgdGhlbgotCQkJCVNFUlZJQ0U9IiQxIgotCQkJCWNoZWNrX3NlcnZpY2UKLQkJCWVs c2UKLQkJCQlPUFRJT05TPSIkT1BUSU9OUyAkMSIKLQkJCWZpCi0JCQlzaGlmdAotCQkJOzsKLQll c2FjCi1kb25lCitpZiBbICQjIC1lcSAyIC1hICIkMiIgPSAiLS1mdWxsLXJlc3RhcnQiIF07IHRo ZW4KKwlTRVJWSUNFPSIkMSIKKwljaGVja19zZXJ2aWNlCisJY2QgLyB8fCBleGl0CisJIiRTRVJW SUNFRElSLyRTRVJWSUNFIiBzdG9wCisJIiRTRVJWSUNFRElSLyRTRVJWSUNFIiBzdGFydAorCWV4 aXQgJD8KK2ZpCiAKLWNkIC8gfHwgZXhpdAotIiRTRVJWSUNFRElSLyRTRVJWSUNFIiAke09QVElP TlN9CitjYXNlICIkMSIgaW4KKwktLWhlbHB8LWgpCisJCXVzYWdlIDAKKwkJOzsKKwktLXZlcnNp b258LVYgKQorCQllY2hvICIkVkVSU0lPTiIKKwkJZXhpdCAwCisJCTs7CisJLSopCisJCXVzYWdl IDEKKwkJOzsKKwkqKQorCQlTRVJWSUNFPSIkMSIKKwkJc2hpZnQKKwkJY2hlY2tfc2VydmljZQor CQljZCAvIHx8IGV4aXQKKwkJIiRTRVJWSUNFRElSLyRTRVJWSUNFIiAiJEAiCisJCWV4aXQgJD8K KwkJOzsKK2VzYWMK