Re: [heketi-devel] Does anyone object to removing the python bindings for the Heketi client?

2018-07-30 Thread Niels de Vos
On Mon, Jul 30, 2018 at 07:12:33AM -0400, John Mulligan wrote:
> Hi Niels,
> 
> The Heketi python bindings should already be Python 3 compatible. I updated 
> them a while back and tried to make sure that they would be regularly tested 
> for both Python 2 and 3 compatibility via tox.
> My guess is that the RPM "doesn't know" this.

Ok, thanks! It seems I can change the .spec to use python3 and at least
it builds just fine (not sure what automated tests there are).

> If you really want to remove the RPM from Fedora because of lack of use I 
> suppose that might be OK, but the binding themselves will continue to exist 
> and I hope to use them more for tests in heketi's own repo.

If the intention is to use it only for testing, then I'll drop it.
Otherwise we can keep the Python bindings. What is your suggestion or
preference?

Niels


> On Sunday, July 29, 2018 2:47:54 PM EDT Niels de Vos wrote:
> > The Fedora package currently fails to build for Rawhide (Fedora 29)
> > because the python bindings under client/api/python have a requirement
> > on Python 2.
> > 
> > I do not know how much work it is to make the bindings compatible with
> > both Python 2 and 3. Before spending time on this, I would like to know
> > if there are any users of the Python bindings at all. In case nobody has
> > a use-case for the bindings, I propose we remove them from the
> > repository.
> > 
> > Thanks,
> > Niels
> 
> 
> 
> 


signature.asc
Description: PGP signature
___
heketi-devel mailing list
heketi-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/heketi-devel


Re: [heketi-devel] Does anyone object to removing the python bindings for the Heketi client?

2018-07-30 Thread John Mulligan
Hi Niels,

The Heketi python bindings should already be Python 3 compatible. I updated 
them a while back and tried to make sure that they would be regularly tested 
for both Python 2 and 3 compatibility via tox.
My guess is that the RPM "doesn't know" this.

If you really want to remove the RPM from Fedora because of lack of use I 
suppose that might be OK, but the binding themselves will continue to exist 
and I hope to use them more for tests in heketi's own repo.

On Sunday, July 29, 2018 2:47:54 PM EDT Niels de Vos wrote:
> The Fedora package currently fails to build for Rawhide (Fedora 29)
> because the python bindings under client/api/python have a requirement
> on Python 2.
> 
> I do not know how much work it is to make the bindings compatible with
> both Python 2 and 3. Before spending time on this, I would like to know
> if there are any users of the Python bindings at all. In case nobody has
> a use-case for the bindings, I propose we remove them from the
> repository.
> 
> Thanks,
> Niels




___
heketi-devel mailing list
heketi-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/heketi-devel