Re: [jenkins-ci PATCH 4/7] jobs: projects: be explicit about python2 binary

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Nov 01, 2017 at 02:26:56PM +0100, Pavel Hrdina wrote:
> Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> ---
>  jobs/python-distutils.yaml | 8 ++++----
>  projects/virt-manager.yaml | 2 +-
>  2 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/jobs/python-distutils.yaml b/jobs/python-distutils.yaml
> index b7dbb1d..7055d26 100644
> --- a/jobs/python-distutils.yaml
> +++ b/jobs/python-distutils.yaml
> @@ -41,8 +41,8 @@
>      builders:
>        - shell: |
>            {command_pre_build}
> -          python setup.py build
> -          python setup.py install --prefix=$VIRT_PREFIX
> +          python2 setup.py build
> +          python2 setup.py install --prefix=$VIRT_PREFIX
>      publishers:
>        - email:
>            recipients: '{obj:spam}'
> @@ -79,7 +79,7 @@
>            values: '{obj:machines}'
>      builders:
>        - shell: |
> -          python setup.py test
> +          python2 setup.py test
>      publishers:
>        - email:
>            recipients: '{obj:spam}'
> @@ -116,7 +116,7 @@
>      builders:
>        - shell: |
>            sed -i -e 's/BuildRequires: libvirt.*devel.*//' *.spec.in
> -          python setup.py rpm
> +          python2 setup.py rpm
>      publishers:
>        - email:
>            recipients: '{obj:spam}'


Hmm, for libvirt-python bindings we ought to make sure we have two sets
of build jobs - once for py2 and once for py3. So, we ought to make the
choice of python binary a variable we pass in.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux