RE: [Xen-devel] [PATCH] unifying error message of migrate and sysrq

["Yoshinori Katase" <y_katase@xxxxxxxxxxxxxxxx>]

Hi,Takebe

Yes,you're right.Please replace it this.
# I have sent a reviced mail to myself !! (><)

Best Regards,
Yoshinori Katase

-----Original Message-----
From: Akio Takebe [mailto:takebe_akio@xxxxxxxxxxxxxx] 
Sent: Saturday, January 14, 2006 1:49 AM
To: Yoshinori Katase; xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Xen-devel] [PATCH] unifying error message of migrate and sysrq

Hi, Katase

I found mistake in your patch.
>diff -r bfcdf4099d23 tools/python/xen/xm/sysrq.py
>--- a/tools/python/xen/xm/sysrq.py      Thu Jan 12 13:27:55 2006
>+++ b/tools/python/xen/xm/sysrq.py      Fri Jan 13 15:26:14 2006
>@@ -24,8 +24,9 @@
>         return
>
>     # no options for the moment
>-    if len(args) < 1: opts.err('Missing domain')
>-    if len(args) < 2: opts.err('Missing sysrq character')
>+    from xen.xm.main import arg_check
>+    arg_check(args, "migrate", 2)
>+
I think this is arg_check(args, "sysrq", 2).
Am I correct?

Best Regards,

Akio Takebe

>This patch unifys error messag of migrate and sysrq.
>
>Signed-off-by: Yoshinori Katase <y_katase@xxxxxxxxxxxxxxxx> Best 
>Regards, Yoshinori Katase
>
>diff -r bfcdf4099d23 tools/python/xen/xm/migrate.py
>--- a/tools/python/xen/xm/migrate.py    Thu Jan 12 13:27:55 2006
>+++ b/tools/python/xen/xm/migrate.py    Fri Jan 13 15:26:14 2006
>@@ -56,8 +56,8 @@
>     if opts.vals.help:
>         opts.usage()
>         return
>-    if len(args) != 2:
>-        opts.err('Invalid arguments: ' + str(args))
>+    from xen.xm.main import arg_check
>+    arg_check(args, "migrate", 2)
>     dom = args[0]
>     dst = args[1]
>     server.xend_domain_migrate(dom, dst, opts.vals.live, 
>opts.vals.resource, opts.vals.port) diff -r bfcdf4099d23 
>tools/python/xen/xm/sysrq.py
>--- a/tools/python/xen/xm/sysrq.py      Thu Jan 12 13:27:55 2006
>+++ b/tools/python/xen/xm/sysrq.py      Fri Jan 13 15:26:14 2006
>@@ -24,8 +24,9 @@
>         return
>
>     # no options for the moment
>-    if len(args) < 1: opts.err('Missing domain')
>-    if len(args) < 2: opts.err('Missing sysrq character')
>+    from xen.xm.main import arg_check
>+    arg_check(args, "migrate", 2)
>+
>     dom = args[0]
>     req = ord(args[1][0])
>     server.xend_domain_sysrq(dom, req) 
>_______________________________________________
>Xen-devel mailing list
>Xen-devel@xxxxxxxxxxxxxxxxxxx
>http://lists.xensource.com/xen-devel

Attachment: migrate.sysrq.patch
Description: Binary data

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel