You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@trafficserver.apache.org by Jan-Frode Myklebust <ja...@tanso.net> on 2011/07/19 10:00:09 UTC
[PATCH] Make condrestart do the right thing, not the opposite.
---
rc/trafficserver.in | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/rc/trafficserver.in b/rc/trafficserver.in
index 6582984..fc892eb 100644
--- a/rc/trafficserver.in
+++ b/rc/trafficserver.in
@@ -398,7 +398,7 @@ case "$1" in
;;
condrestart)
if [ "$DISTRIB_ID" = "fedora" -o "$DISTRIB_ID" = "redhat" ]; then
- if ! $0 'status'
+ if $0 'status'
then
sh $0 'stop'
sh $0 'start'
--
1.7.1
Re: [PATCH] Make condrestart do the right thing, not the opposite.
Posted by Jan-Frode Myklebust <ja...@tanso.net>.
On Tue, Jul 19, 2011 at 08:35:16AM -0600, Leif Hedstrom wrote:
> Is there a bug filed for this?
There is now:
https://issues.apache.org/jira/browse/TS-885
-jf
Re: [PATCH] Make condrestart do the right thing, not the opposite.
Posted by Leif Hedstrom <zw...@apache.org>.
Is there a bug filed for this?
Leif
On Jul 19, 2011, at 2:00 AM, Jan-Frode Myklebust <ja...@tanso.net> wrote:
> ---
> rc/trafficserver.in | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/rc/trafficserver.in b/rc/trafficserver.in
> index 6582984..fc892eb 100644
> --- a/rc/trafficserver.in
> +++ b/rc/trafficserver.in
> @@ -398,7 +398,7 @@ case "$1" in
> ;;
> condrestart)
> if [ "$DISTRIB_ID" = "fedora" -o "$DISTRIB_ID" = "redhat" ]; then
> - if ! $0 'status'
> + if $0 'status'
> then
> sh $0 'stop'
> sh $0 'start'
> --
> 1.7.1
>