FS#55930 - [lvm2] Request: use --enable-python{2,3}_bindings

Attached to Project: Arch Linux
Opened by brent saner (sanerb) - Monday, 09 October 2017, 11:36 GMT
Last edited by Christian Hesse (eworm) - Tuesday, 10 October 2017, 09:29 GMT
Task Type Feature Request
Category Packages: Extra
Status Closed
Assigned To Christian Hesse (eworm)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
LVM2 offers python bindings. It'd be great if you could build them with the package, because otherwise it'd need an entirely new package in the AUR just to get the python modules.

Additional info:
Relevant lines in configure script (line is link-anchored, so should be first line in display of browser):
* Python:
** https://sourceware.org/git/?p=lvm2.git;a=blob;f=configure;h=9f2f74414e3f2bdc2989216a90de72a21299a0a7;hb=HEAD#l1719
* Python 2:
* https://sourceware.org/git/?p=lvm2.git;a=blob;f=configure;h=9f2f74414e3f2bdc2989216a90de72a21299a0a7;hb=HEAD#l1721
* Python 3:
** https://sourceware.org/git/?p=lvm2.git;a=blob;f=configure;h=9f2f74414e3f2bdc2989216a90de72a21299a0a7;hb=HEAD#l1723

Steps to reproduce:
N/A
This task depends upon

Closed by  Christian Hesse (eworm)
Tuesday, 10 October 2017, 09:29 GMT
Reason for closing:  Won't implement
Additional comments about closing:  deprecated feature
Comment by brent saner (sanerb) - Monday, 09 October 2017, 11:51 GMT
confirmed- manually built from your package. patch attached for PKGBUILD, but it should literally be no big deal, it's just two lines added to the configure opts. :P

(it may be more prudent to instead add them to the './configure $CONFIGUREOPTS --enable-udev-systemd-background-jobs' line, instead of the CONFIGUREOPTS var as i do in the attached patch, as i doubt python bindings would be needed in an initramfs but this is up to your discretion of course.)

[root@archdev ~]# python
Python 3.6.2 (default, Jul 20 2017, 03:52:27)
[GCC 7.1.1 20170630] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import lvm
>>> lvm.
lvm.LibLVMError( lvm.configFindBool( lvm.gc( lvm.listVgUuids( lvm.scan( lvm.vgNameValidate(
lvm.THIN_DISCARDS_IGNORE lvm.configOverride( lvm.getVersion( lvm.percentToFloat( lvm.vgCreate( lvm.vgOpen(
lvm.THIN_DISCARDS_NO_PASSDOWN lvm.configReload( lvm.listPvs( lvm.pvCreate( lvm.vgNameFromDevice(
lvm.THIN_DISCARDS_PASSDOWN lvm.error( lvm.listVgNames( lvm.pvRemove( lvm.vgNameFromPvid(

[root@archdev ~]# python2
Python 2.7.14 (default, Sep 20 2017, 01:25:59)
[GCC 7.2.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import lvm
>>> lvm.scan()
>>>

(this is expected; i have no LVM devices on this box)

Comment by Christian Hesse (eworm) - Tuesday, 10 October 2017, 09:29 GMT
configure gives:

configure: WARNING: Python bindings are deprecated. Use D-Bus API

So no, we will not have deprecated feature.

Loading...