<html> <head> <style></style> </head> <body class='hmmessage'> <div>Patch 5/5 only touches the 3 files listed in your original email.</div><div><br></div><div>The script fence_xenapi.py is the script that does the fencing, not the library.</div><div><br></div><div>XenAPI.py is the underlying library which is now located in lib/ as of patch 4/5</div><div><div>0004-Clean-up-of-fence_xenapi-patches.patch: fence/agents/lib/XenAPI.py.py | 209 +++++++++++++++++++++++++++++++++++++++++</div><div>0004-Clean-up-of-fence_xenapi-patches.patch: fence/agents/xenapi/XenAPI.py | 209 -----------------------------------------</div></div><div><br></div><div>Unless I am missing something... Which is quite possible.</div><div><br><br>> Date: Wed, 20 Apr 2011 17:32:23 +0200<br>> From: fdinitto@redhat.com<br>> To: cluster-devel@redhat.com<br>> Subject: Re: [Cluster-devel] [PATCH 5/5] Cleanup of fence_xenapi patches. Added copyright information to doc/COPYRIGHT. Fixed static reference to lib directory in fence_xenapi.py. Fixed static reference to RELEASE_VERSION and BUILD_DATE in fence_xenapi.py.<br>> <br>> On 04/20/2011 05:12 PM, Matt Clark wrote:<br>> > Patch 4/5 takes care of that. At least it should have...<br>> > <br>> <br>> Then something is wrong with the patch ordering because 4/4 renames but<br>> 5 changes the old path?<br>> <br>> Fabio<br>> <br>> >> Date: Wed, 20 Apr 2011 16:04:46 +0200<br>> >> From: fdinitto@redhat.com<br>> >> To: cluster-devel@redhat.com<br>> >> Subject: Re: [Cluster-devel] [PATCH 5/5] Cleanup of fence_xenapi<br>> > patches. Added copyright information to doc/COPYRIGHT. Fixed static<br>> > reference to lib directory in fence_xenapi.py. Fixed static reference to<br>> > RELEASE_VERSION and BUILD_DATE in fence_xenapi.py.<br>> >><br>> >> On 04/20/2011 01:51 PM, Matt Clark wrote:<br>> >> > ---<br>> >> > doc/COPYRIGHT | 4 ++++<br>> >> > fence/agents/lib/Makefile.am | 4 ++--<br>> >> > fence/agents/xenapi/fence_xenapi.py | 12 ++++++++----<br>> >> > 3 files changed, 14 insertions(+), 6 deletions(-)<br>> >><br>> >> You forgot to move the xenapi file in lib/ ?<br>> >><br>> >> Fabio<br>> >><br>> <br></div> </body> </html>