All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] xm list ERROR_INVALID_DOMAIN
@ 2008-09-17 18:25 Shaun R.
  2008-09-19 19:32 ` Shaun R.
  0 siblings, 1 reply; 3+ messages in thread
From: Shaun R. @ 2008-09-17 18:25 UTC (permalink / raw)
  To: xen-devel

This patch will make xm return a exit code of 3 if `xm list 
<non_existant_domain>` is done rather than the generic code of 1.  I used 3 
because XendClient had a macro setup pointing ERROR_INVALID_DOMAIN to 3.

~Shaun Reitan



diff -r 536475e491cc tools/python/xen/xm/main.py
--- a/tools/python/xen/xm/main.py       Mon Sep 15 11:42:31 2008 +0100
+++ b/tools/python/xen/xm/main.py       Wed Sep 17 06:20:38 2008 -0700
@@ -2926,6 +2926,7 @@
     except xmlrpclib.Fault, ex:
         if ex.faultCode == XendClient.ERROR_INVALID_DOMAIN:
             err("Domain '%s' does not exist." % ex.faultString)
+            return False, ex.faultCode
         else:
             err(ex.faultString)
             _usage(cmd_name)

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

* Re: [PATCH] xm list ERROR_INVALID_DOMAIN
  2008-09-17 18:25 [PATCH] xm list ERROR_INVALID_DOMAIN Shaun R.
@ 2008-09-19 19:32 ` Shaun R.
  2008-09-20  8:06   ` Keir Fraser
  0 siblings, 1 reply; 3+ messages in thread
From: Shaun R. @ 2008-09-19 19:32 UTC (permalink / raw)
  To: xen-devel

just curious, did i go about this the right way to get it applied?

~Shaun

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

* Re: Re: [PATCH] xm list ERROR_INVALID_DOMAIN
  2008-09-19 19:32 ` Shaun R.
@ 2008-09-20  8:06   ` Keir Fraser
  0 siblings, 0 replies; 3+ messages in thread
From: Keir Fraser @ 2008-09-20  8:06 UTC (permalink / raw)
  To: Shaun R., xen-devel

Yes, it's in the staging tree now
(http://xenbits.xensource.com/staging/xen-unstable.hg).

 -- Keir

On 19/9/08 20:32, "Shaun R." <mailinglists@unix-scripts.com> wrote:

> just curious, did i go about this the right way to get it applied?
> 
> ~Shaun
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel

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

end of thread, other threads:[~2008-09-20  8:06 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-09-17 18:25 [PATCH] xm list ERROR_INVALID_DOMAIN Shaun R.
2008-09-19 19:32 ` Shaun R.
2008-09-20  8:06   ` Keir Fraser

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.