I don''t know python well, but when I compile at my platform, compile error as: ======================... Parsing libxl.idl cmp: _libxl_paths.h.tmp: No such file or directory cmp: _libxl_paths.h: No such file or directory cmp: xen/util/path.py: No such file or directory Parsing /otc/source/vtd/xen-unstable/tools/python/../../tools/libxl/libxl.idl File "/usr/lib64/python2.4/site-packages/xen/remus/save.py", line 169 finally: ^ SyntaxError: invalid syntax ... ===================== This kind of compile error introduced from c/s 23195. This patch revert it. Thanks, Jinsong _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Adding Remus maintainer. You can consult the MAINTAINERS file at the top level to fix this. On Fri, 2011-05-13 at 15:26 +0100, Liu, Jinsong wrote:> I don''t know python well, but when I compile at my platform, compile error as: > > > ======================> ... > Parsing libxl.idl > cmp: _libxl_paths.h.tmp: No such file or directory > cmp: _libxl_paths.h: No such file or directory > cmp: xen/util/path.py: No such file or directory > Parsing /otc/source/vtd/xen-unstable/tools/python/../../tools/libxl/libxl.idl > File "/usr/lib64/python2.4/site-packages/xen/remus/save.py", line 169 > finally: > ^ > SyntaxError: invalid syntax > ... > > =====================> > > This kind of compile error introduced from c/s 23195. > This patch revert it.That seems rather extreme in the first instance.>From a quick google It seems that python 2.4 did not properly supportthe "try: ... except: ... finally: ... " syntax. It appears that as a workaround one can do try: try: ... except: ... finally: ... But I didn''t try this myself. Ian. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Ian Campbell wrote:> Adding Remus maintainer. You can consult the MAINTAINERS file at the > top level to fix this. > > On Fri, 2011-05-13 at 15:26 +0100, Liu, Jinsong wrote: >> I don''t know python well, but when I compile at my platform, compile >> error as: >> >> >> ======================>> ... >> Parsing libxl.idl >> cmp: _libxl_paths.h.tmp: No such file or directory >> cmp: _libxl_paths.h: No such file or directory >> cmp: xen/util/path.py: No such file or directory >> Parsing >> >> >> /otc/source/vtd/xen-unstable/tools/python/../../tools/libxl/libxl.idl >> File "/usr/lib64/python2.4/site-packages/xen/remus/save.py", line >> 169 finally: ^ >> SyntaxError: invalid syntax >> ... >> >> =====================>> >> >> This kind of compile error introduced from c/s 23195. >> This patch revert it. > > That seems rather extreme in the first instance. > > From a quick google It seems that python 2.4 did not properly support > the "try: ... except: ... finally: ... " syntax. It appears that as a > workaround one can do > try: > try: > ... > except: > ... > finally: > ... > > But I didn''t try this myself. > > Ian.Shriram and Ian, I update the patch according to Ian''s comments, as attached. Test result shows compiling OK now. Thanks, Jinsong _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
On Fri, May 13, 2011 at 1:33 PM, Liu, Jinsong <jinsong.liu@intel.com> wrote:> Ian Campbell wrote: > > Adding Remus maintainer. You can consult the MAINTAINERS file at the > > top level to fix this. > > > > On Fri, 2011-05-13 at 15:26 +0100, Liu, Jinsong wrote: > >> I don''t know python well, but when I compile at my platform, compile > >> error as: > >> > >> > >> ======================> >> ... > >> Parsing libxl.idl > >> cmp: _libxl_paths.h.tmp: No such file or directory > >> cmp: _libxl_paths.h: No such file or directory > >> cmp: xen/util/path.py: No such file or directory > >> Parsing > >> > >> > >> /otc/source/vtd/xen-unstable/tools/python/../../tools/libxl/libxl.idl > >> File "/usr/lib64/python2.4/site-packages/xen/remus/save.py", line > >> 169 finally: ^ > >> SyntaxError: invalid syntax > >> ... > >> > >> =====================> >> > >> > >> This kind of compile error introduced from c/s 23195. > >> This patch revert it. > > > > That seems rather extreme in the first instance. > > > > From a quick google It seems that python 2.4 did not properly support > > the "try: ... except: ... finally: ... " syntax. It appears that as a > > workaround one can do > > try: > > try: > > ... > > except: > > ... > > finally: > > ... > > > > But I didn''t try this myself. > > > > Ian. > > Shriram and Ian, > > I update the patch according to Ian''s comments, as attached. > Test result shows compiling OK now. > > Hi,sorry for the delay but I was waiting for the xen side bug to get resolved before testing your patch. It works properly in remus installation. Acked-by: Shriram Rajagopalan <rshriram@cs.ubc.ca> Thanks,> Jinsong_______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Resend it, it has been acked by Ian and Shriram, but seems to be ignored and didn''t in unstable tree :) Thanks, Jinsong ===============Fix save.py bug Fix save.py bug, to avoid compiling error under python2.4 Signed-off-by: Liu, Jinsong <jinsong.liu@intel.com> Acked-by: Ian Campbell <Ian.Campbell@eu.citrix.com> Acked-by: Shriram Rajagopalan <rshriram@cs.ubc.ca> diff -r f9bb0bbea7c2 tools/python/xen/remus/save.py --- a/tools/python/xen/remus/save.py Thu May 12 16:42:54 2011 +0100 +++ b/tools/python/xen/remus/save.py Fri May 13 22:45:21 2011 +0800 @@ -161,11 +161,12 @@ class Saver(object): self.checkpointer = xen.lowlevel.checkpoint.checkpointer() try: - self.checkpointer.open(self.vm.domid) - self.checkpointer.start(self.fd, self.suspendcb, self.resumecb, - self.checkpointcb, self.interval) - except xen.lowlevel.checkpoint.error, e: - raise CheckpointError(e) + try: + self.checkpointer.open(self.vm.domid) + self.checkpointer.start(self.fd, self.suspendcb, self.resumecb, + self.checkpointcb, self.interval) + except xen.lowlevel.checkpoint.error, e: + raise CheckpointError(e) finally: try: #errors in checkpoint close are not critical atm. self.checkpointer.close() _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
On Wed, May 18, 2011 at 4:30 AM, Liu, Jinsong <jinsong.liu@intel.com> wrote:> Resend it, it has been acked by Ian and Shriram, but seems to be ignored > and didn''t in unstable tree :) > > Thanks, > Jinsong > > ===============> Fix save.py bug > > Fix save.py bug, to avoid compiling error under python2.4 > > Signed-off-by: Liu, Jinsong <jinsong.liu@intel.com> > Acked-by: Ian Campbell <Ian.Campbell@eu.citrix.com> > Acked-by: Shriram Rajagopalan <rshriram@cs.ubc.ca> > > diff -r f9bb0bbea7c2 tools/python/xen/remus/save.py > --- a/tools/python/xen/remus/save.py Thu May 12 16:42:54 2011 +0100 > +++ b/tools/python/xen/remus/save.py Fri May 13 22:45:21 2011 +0800 > @@ -161,11 +161,12 @@ class Saver(object): > > self.checkpointer = xen.lowlevel.checkpoint.checkpointer() > try: > - self.checkpointer.open(self.vm.domid) > - self.checkpointer.start(self.fd, self.suspendcb, > self.resumecb, > - self.checkpointcb, self.interval) > - except xen.lowlevel.checkpoint.error, e: > - raise CheckpointError(e) > + try: > + self.checkpointer.open(self.vm.domid) > + self.checkpointer.start(self.fd, self.suspendcb, > self.resumecb, > + self.checkpointcb, self.interval) > + except xen.lowlevel.checkpoint.error, e: > + raise CheckpointError(e) > finally: > try: #errors in checkpoint close are not critical atm. > self.checkpointer.close()>From my experience it takes couple of days for the patches to appear in theunstable tree, after they have been sent to the mailing list. No point resending it. :) shriram _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Thanks for reminding :) Jinsong ________________________________ From: Shriram Rajagopalan [mailto:rshriram@cs.ubc.ca] Sent: Wednesday, May 18, 2011 9:00 PM To: Liu, Jinsong Cc: Keir Fraser; xen-devel@lists.xensource.com; Ian Campbell; Li, Xin; Ian Jackson Subject: Re: [PATCH] Fix save.py bug On Wed, May 18, 2011 at 4:30 AM, Liu, Jinsong <jinsong.liu@intel.com<mailto:jinsong.liu@intel.com>> wrote: Resend it, it has been acked by Ian and Shriram, but seems to be ignored and didn''t in unstable tree :) Thanks, Jinsong ===============Fix save.py bug Fix save.py bug, to avoid compiling error under python2.4 Signed-off-by: Liu, Jinsong <jinsong.liu@intel.com<mailto:jinsong.liu@intel.com>> Acked-by: Ian Campbell <Ian.Campbell@eu.citrix.com<mailto:Ian.Campbell@eu.citrix.com>> Acked-by: Shriram Rajagopalan <rshriram@cs.ubc.ca<mailto:rshriram@cs.ubc.ca>> diff -r f9bb0bbea7c2 tools/python/xen/remus/save.py --- a/tools/python/xen/remus/save.py Thu May 12 16:42:54 2011 +0100 +++ b/tools/python/xen/remus/save.py Fri May 13 22:45:21 2011 +0800 @@ -161,11 +161,12 @@ class Saver(object): self.checkpointer = xen.lowlevel.checkpoint.checkpointer() try: - self.checkpointer.open(self.vm.domid) - self.checkpointer.start(self.fd, self.suspendcb, self.resumecb, - self.checkpointcb, self.interval) - except xen.lowlevel.checkpoint.error, e: - raise CheckpointError(e) + try: + self.checkpointer.open(self.vm.domid) + self.checkpointer.start(self.fd, self.suspendcb, self.resumecb, + self.checkpointcb, self.interval) + except xen.lowlevel.checkpoint.error, e: + raise CheckpointError(e) finally: try: #errors in checkpoint close are not critical atm. self.checkpointer.close()>From my experience it takes couple of days for the patches to appear in the unstable tree,after they have been sent to the mailing list. No point resending it. :) shriram _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Shriram Rajagopalan writes ("Re: [PATCH] Fix save.py bug"):> Signed-off-by: Liu, Jinsong <jinsong.liu@intel.com<mailto:jinsong.liu@intel.com>> > Acked-by: Ian Campbell <Ian.Campbell@eu.citrix.com<mailto:Ian.Campbell@eu.citrix.com>> > Acked-by: Shriram Rajagopalan <rshriram@cs.ubc.ca<mailto:rshriram@cs.ubc.ca>>patching file tools/python/xen/remus/save.py Hunk #1 FAILED at 161. 1 out of 1 hunk FAILED -- saving rejects to file tools/python/xen/remus/save.py.rej Ian. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Ian Jackson wrote:> Shriram Rajagopalan writes ("Re: [PATCH] Fix save.py bug"): >> Signed-off-by: Liu, Jinsong >> <jinsong.liu@intel.com<mailto:jinsong.liu@intel.com>> Acked-by: Ian >> Campbell >> <Ian.Campbell@eu.citrix.com<mailto:Ian.Campbell@eu.citrix.com>> >> Acked-by: Shriram Rajagopalan >> <rshriram@cs.ubc.ca<mailto:rshriram@cs.ubc.ca>> > > patching file tools/python/xen/remus/save.py > Hunk #1 FAILED at 161. > 1 out of 1 hunk FAILED -- saving rejects to file > tools/python/xen/remus/save.py.rej > > Ian.?? some strange, I patched based on a recent tree :( Shriram, is there big change recently for save.py? Anyway, I will pull latest xen tree next Monday and resend a patch to avoid hunk. Thanks, Jinsong _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
On 2011-05-20, at 11:30 PM, "Liu, Jinsong" <jinsong.liu@intel.com> wrote:> Ian Jackson wrote: >> Shriram Rajagopalan writes ("Re: [PATCH] Fix save.py bug"): >>> Signed-off-by: Liu, Jinsong >>> <jinsong.liu@intel.com<mailto:jinsong.liu@intel.com>> Acked-by: Ian >>> Campbell >>> <Ian.Campbell@eu.citrix.com<mailto:Ian.Campbell@eu.citrix.com>> >>> Acked-by: Shriram Rajagopalan >>> <rshriram@cs.ubc.ca<mailto:rshriram@cs.ubc.ca>> >> >> patching file tools/python/xen/remus/save.py >> Hunk #1 FAILED at 161. >> 1 out of 1 hunk FAILED -- saving rejects to file >> tools/python/xen/remus/save.py.rej >> >> Ian. > > ?? some strange, I patched based on a recent tree :( > > Shriram, is there big change recently for save.py?None at all!> Anyway, I will pull latest xen tree next Monday and resend a patch to avoid hunk. > > Thanks, > JinsongShriram _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Ian Jackson wrote:> Shriram Rajagopalan writes ("Re: [PATCH] Fix save.py bug"): >> Signed-off-by: Liu, Jinsong >> <jinsong.liu@intel.com<mailto:jinsong.liu@intel.com>> Acked-by: Ian >> Campbell >> <Ian.Campbell@eu.citrix.com<mailto:Ian.Campbell@eu.citrix.com>> >> Acked-by: Shriram Rajagopalan >> <rshriram@cs.ubc.ca<mailto:rshriram@cs.ubc.ca>> > > patching file tools/python/xen/remus/save.py > Hunk #1 FAILED at 161. > 1 out of 1 hunk FAILED -- saving rejects to file > tools/python/xen/remus/save.py.rej > > Ian.Ian, I pull and try at latest xen (c/s 23368), no problem to patch it. Some strange :( Thanks, Jinsong _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Liu, Jinsong writes ("RE: [PATCH] Fix save.py bug"):> I pull and try at latest xen (c/s 23368), no problem to patch it. > Some strange :(Thanks. Your resend works for me. I think it''s possible that the problem was caused by some difficulties we are having with the Citrix email system. In this case I used the copy that came via the xen-devel list (which doesn''t come through the Citrix email system for me). Acked-by: Ian Jackson <ian.jackson@eu.citrix.com> Committed-by: Ian Jackson <ian.jackson@eu.citrix.com> Ian. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Ian Jackson wrote:> Liu, Jinsong writes ("RE: [PATCH] Fix save.py bug"): >> I pull and try at latest xen (c/s 23368), no problem to patch it. >> Some strange :( > > Thanks. Your resend works for me. I think it''s possible that the > problem was caused by some difficulties we are having with the Citrix > email system. In this case I used the copy that came via the > xen-devel list (which doesn''t come through the Citrix email system > for me). > > Acked-by: Ian Jackson <ian.jackson@eu.citrix.com> > Committed-by: Ian Jackson <ian.jackson@eu.citrix.com> > > Ian.Good, thanks! _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel