[libvirt] [libvirt-python] Use user requested python binary for setup.py
Daniel P. Berrange
berrange at redhat.com
Tue Dec 3 17:26:25 UTC 2013
On Tue, Dec 03, 2013 at 11:11:18AM -0600, Doug Goldstein wrote:
> When setup.py is kicked off with a python interpreter other than the
> system 'python', (e.g. python2.7 setup.py build) the build process would
> switch to 'python' and not use python2.7 as requested by the user. We
> should always respect the user requested python interpreter and use it.
> ---
> setup.py | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/setup.py b/setup.py
> index ecbce1f..24d4cf2 100755
> --- a/setup.py
> +++ b/setup.py
> @@ -122,10 +122,10 @@ class my_build(build):
> def run(self):
> apis = get_api_xml_files()
>
> - self.spawn(["python", "generator.py", "libvirt", apis[0]])
> - self.spawn(["python", "generator.py", "libvirt-qemu", apis[1]])
> + self.spawn([sys.executable, "generator.py", "libvirt", apis[0]])
> + self.spawn([sys.executable, "generator.py", "libvirt-qemu", apis[1]])
> if have_libvirt_lxc:
> - self.spawn(["python", "generator.py", "libvirt-lxc", apis[2]])
> + self.spawn([sys.executable, "generator.py", "libvirt-lxc", apis[2]])
>
> build.run(self)
>
> @@ -267,7 +267,7 @@ class my_test(Command):
>
> apis = get_api_xml_files()
>
> - self.spawn(["python", "sanitytest.py", self.build_platlib, apis[0]])
> + self.spawn([sys.executable, "sanitytest.py", self.build_platlib, apis[0]])
ACK.
I didn't notice this since I've been running generator.py directly in
testing :-)
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the libvir-list
mailing list