All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] tools/hotplug/Linux/init.d: fix wrong path in xendomains init
@ 2014-08-08 10:46 Fabio Fantoni
  2014-08-19  7:48 ` Fabio Fantoni
  2014-08-21 22:50 ` Ian Campbell
  0 siblings, 2 replies; 3+ messages in thread
From: Fabio Fantoni @ 2014-08-08 10:46 UTC (permalink / raw
  To: xen-devel; +Cc: Fabio Fantoni, Ian.Jackson, Ian.Campbell

Fixed wrong path in xendomains init that make it not working, introduced
in commit d8bad9df2544291a0fdc8e4d826b16f9f5394462 (tools/xendomains:
move to libexec and use a smaller init helper).

Signed-off-by: Fabio Fantoni <fabio.fantoni@m2r.biz>
Reviewed-by: Wei Liu <wei.liu2@citrix.com>

---

Changes in v2:
- changed path also in Descrition field
---
 tools/hotplug/Linux/init.d/xendomains | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/tools/hotplug/Linux/init.d/xendomains b/tools/hotplug/Linux/init.d/xendomains
index d0122fc..3857602 100644
--- a/tools/hotplug/Linux/init.d/xendomains
+++ b/tools/hotplug/Linux/init.d/xendomains
@@ -20,7 +20,7 @@
 # Short-Description: Wrapper to start/stop secondary xen domains
 # Description:       Wrapper for starting / stopping domains automatically
 #                    when domain 0 boots / shuts down on systems using init.
-#                    The $SBINDIR/xendomains helper is shared between init and
+#                    The $LIBEXEC/xendomains helper is shared between init and
 #                    systemd systems.
 ### END INIT INFO
 
@@ -28,19 +28,19 @@
 
 case "$1" in
     start)
-	$SBINDIR/xendomains start
+	$LIBEXEC/xendomains start
 	;;
     stop)
-	$SBINDIR/xendomains stop
+	$LIBEXEC/xendomains stop
 	;;
     restart)
-	$SBINDIR/xendomains restart
+	$LIBEXEC/xendomains restart
 	;;
     reload)
-	$SBINDIR/xendomains reload
+	$LIBEXEC/xendomains reload
 	;;
     status)
-	$SBINDIR/xendomains status
+	$LIBEXEC/xendomains status
 	;;
     *)
 	echo "Usage: $0 {start|stop|restart|reload|status}"
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] tools/hotplug/Linux/init.d: fix wrong path in xendomains init
  2014-08-08 10:46 [PATCH v2] tools/hotplug/Linux/init.d: fix wrong path in xendomains init Fabio Fantoni
@ 2014-08-19  7:48 ` Fabio Fantoni
  2014-08-21 22:50 ` Ian Campbell
  1 sibling, 0 replies; 3+ messages in thread
From: Fabio Fantoni @ 2014-08-19  7:48 UTC (permalink / raw
  To: xen-devel; +Cc: Ian.Jackson, Ian.Campbell

Il 08/08/2014 12:46, Fabio Fantoni ha scritto:
> Fixed wrong path in xendomains init that make it not working, introduced
> in commit d8bad9df2544291a0fdc8e4d826b16f9f5394462 (tools/xendomains:
> move to libexec and use a smaller init helper).
>
> Signed-off-by: Fabio Fantoni <fabio.fantoni@m2r.biz>
> Reviewed-by: Wei Liu <wei.liu2@citrix.com>

Ping...

>
> ---
>
> Changes in v2:
> - changed path also in Descrition field
> ---
>   tools/hotplug/Linux/init.d/xendomains | 12 ++++++------
>   1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/tools/hotplug/Linux/init.d/xendomains b/tools/hotplug/Linux/init.d/xendomains
> index d0122fc..3857602 100644
> --- a/tools/hotplug/Linux/init.d/xendomains
> +++ b/tools/hotplug/Linux/init.d/xendomains
> @@ -20,7 +20,7 @@
>   # Short-Description: Wrapper to start/stop secondary xen domains
>   # Description:       Wrapper for starting / stopping domains automatically
>   #                    when domain 0 boots / shuts down on systems using init.
> -#                    The $SBINDIR/xendomains helper is shared between init and
> +#                    The $LIBEXEC/xendomains helper is shared between init and
>   #                    systemd systems.
>   ### END INIT INFO
>   
> @@ -28,19 +28,19 @@
>   
>   case "$1" in
>       start)
> -	$SBINDIR/xendomains start
> +	$LIBEXEC/xendomains start
>   	;;
>       stop)
> -	$SBINDIR/xendomains stop
> +	$LIBEXEC/xendomains stop
>   	;;
>       restart)
> -	$SBINDIR/xendomains restart
> +	$LIBEXEC/xendomains restart
>   	;;
>       reload)
> -	$SBINDIR/xendomains reload
> +	$LIBEXEC/xendomains reload
>   	;;
>       status)
> -	$SBINDIR/xendomains status
> +	$LIBEXEC/xendomains status
>   	;;
>       *)
>   	echo "Usage: $0 {start|stop|restart|reload|status}"

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] tools/hotplug/Linux/init.d: fix wrong path in xendomains init
  2014-08-08 10:46 [PATCH v2] tools/hotplug/Linux/init.d: fix wrong path in xendomains init Fabio Fantoni
  2014-08-19  7:48 ` Fabio Fantoni
@ 2014-08-21 22:50 ` Ian Campbell
  1 sibling, 0 replies; 3+ messages in thread
From: Ian Campbell @ 2014-08-21 22:50 UTC (permalink / raw
  To: Fabio Fantoni; +Cc: xen-devel, Ian.Jackson

On Fri, 2014-08-08 at 12:46 +0200, Fabio Fantoni wrote:
> Fixed wrong path in xendomains init that make it not working, introduced
> in commit d8bad9df2544291a0fdc8e4d826b16f9f5394462 (tools/xendomains:
> move to libexec and use a smaller init helper).
> 
> Signed-off-by: Fabio Fantoni <fabio.fantoni@m2r.biz>
> Reviewed-by: Wei Liu <wei.liu2@citrix.com>

Applied.

Ian.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-08-21 22:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-08-08 10:46 [PATCH v2] tools/hotplug/Linux/init.d: fix wrong path in xendomains init Fabio Fantoni
2014-08-19  7:48 ` Fabio Fantoni
2014-08-21 22:50 ` Ian Campbell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.