Re: Removing glupy from release 5.7

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

 



Le jeudi 04 juillet 2019 à 16:20 +0200, Niels de Vos a écrit :
> On Wed, Jul 03, 2019 at 04:46:11PM +0200, Michael Scherer wrote:
> > Le mercredi 03 juillet 2019 à 20:03 +0530, Deepshikha Khandelwal a
> > écrit :
> > > Misc, is EPEL got recently installed on the builders?
> > 
> > No, it has been there since september 2016. What got changed is
> > that
> > python3 wasn't installed before.
> > 
> > > Can you please resolve the 'Why EPEL on builders?'. EPEL+python3
> > > on
> > > builders seems not a good option to have.
> > 
> > 
> > Python 3 is pulled by 'mock', cf 
> > 
https://lists.gluster.org/pipermail/gluster-devel/2019-June/056347.html
> > 
> > So sure, I can remove EPEL, but then it will remove mock. Or I can
> > remove python3, and it will remove mock.
> > 
> > But again, the problem is not with the set of installed packages on
> > the
> > builder, that's just showing there is a bug.
> > 
> > The configure script do pick the latest python version:
> > https://github.com/gluster/glusterfs/blob/master/configure.ac#L612
> > 
> > if there is a python3, it take that, if not, it fall back to
> > python2. 
> > 
> > then, later:
> > https://github.com/gluster/glusterfs/blob/master/configure.ac#L639
> > 
> > it verify the presence of what is required to build.
> > 
> > So if there is a runtime version only of python3, it will detect
> > python3, but not build anything, because the -devel subpackage is
> > not h
> > ere. 
> > 
> > There is 2 solutions:
> > - fix that piece of code, so it doesn't just test the presence of
> > python executable, but do that, and test the presence of headers
> > before
> > deciding if we need to build or not glupy.
> > 
> > - use PYTHON env var to force python2, and document that it need to
> > be
> > done.  
> 
> What about option 3:
> 
> - install python3-devel in addition to python3

That's a option, but I think that's a disservice for the users, since
that's fixing our CI to no longer trigger a corner case, which doesn't
mean the corner case no longer exist, just that we do not trigger it. 


-- 
Michael Scherer
Sysadmin, Community Infrastructure



Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________

Community Meeting Calendar:

APAC Schedule -
Every 2nd and 4th Tuesday at 11:30 AM IST
Bridge: https://bluejeans.com/836554017

NA/EMEA Schedule -
Every 1st and 3rd Tuesday at 01:00 PM EDT
Bridge: https://bluejeans.com/486278655

Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
https://lists.gluster.org/mailman/listinfo/gluster-devel


[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux