Re: [libvirt-python PATCH] setup: require python >= 3.5 to build

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

 



On Mon, Apr 20, 2020 at 02:59:38PM +0200, Philipp Hahn wrote:
> Hello Daniel.
> 
> 2 nits:
> 
> Am 20.04.20 um 14:47 schrieb Daniel P. Berrangé:
> > Pytjon 3.5 is the oldest Python version available across our supported
>      ^ h
> > build platforms.
> > 
> > Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> > ---
> >  setup.py | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/setup.py b/setup.py
> > index 56b6eea..b3e418e 100755
> > --- a/setup.py
> > +++ b/setup.py
> > @@ -17,8 +17,8 @@ import re
> >  import shutil
> >  import time
> >  
> > -if sys.version_info[0] != 3:
> > -    print("libvirt-python requires Python 3.x to build")
> > +if sys.version_info[0] != 3 or sys.version_info[1] < 5:
> 
> if sys.version_info < (3, 5):

IIUC, that would not reject  Python 4.x, which I presume would be
a backwards incompatible version bump.  

> 
> > +    print("libvirt-python requires Python 3 >= 3.5 to build")
> >      sys.exit(1)
> >  
> >  MIN_LIBVIRT = "0.9.11"
> > 
> 
> plus maybe this:
> 
> > diff --git a/setup.py b/setup.py
> > index 56b6eea..e20b7b3 100755
> > --- a/setup.py
> > +++ b/setup.py
> > @@ -368,5 +368,9 @@ of recent versions of Linux (and other OSes).''',
> >            "License :: OSI Approved :: GNU Lesser General Public License v2 or later (LGPLv2+)",
> >            "Programming Language :: Python",
> >            "Programming Language :: Python :: 3",
> > +          "Programming Language :: Python :: 3.5",
> > +          "Programming Language :: Python :: 3.6",
> > +          "Programming Language :: Python :: 3.7",
> > +          "Programming Language :: Python :: 3.8",

Are these used for any functional purpose, or just really a documentation
thing ?  This does introduce the issue of someone having to remember to
add  3.9, 3.10, etc as they arrive. I'm confident we'll forget that.


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 :|





[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