Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)
Closed #130 as completed via #2231. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/130#event-7629947590 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)
There you go: https://github.com/rpm-software-management/rpm/pull/2231 -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/130#issuecomment-1282293306 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)
It's a deal :grin: -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/130#issuecomment-1282252678 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)
We can probably do this with the CMake build and rip out everything else. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/130#issuecomment-1282247535 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)
This resurfaced via #2230. So okay, losing egg-info would be a loss, but if one actually looks at this precious egg: ``` etadata-Version: 1.0 Name: rpm Version: 4.18.0 Summary: Python bindings for rpm Home-page: http://www.rpm.org/ Author: UNKNOWN Author-email: rpm-maint@lists.rpm.org License: GNU General Public License v2 Description: UNKNOWN Platform: UNKNOWN ``` I think we're able to create such a file without the aid of a Python library :sweat_smile: -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/130#issuecomment-1282156960 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)
Right, that's a fair point. Ultimately there should indeed be just one way to build the bindings, maintaining multiple ways was never a good idea anyway. OTOH calling setup.py from inside automake sounds like a nightmare too. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/130#issuecomment-605952790___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)
Without the setup.py, we don't have the egg-info data which other projects use to depend on the Python bindings. The Autofoo should probably just be reworked to call setup.py instead. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/130#issuecomment-605949977___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)
The issue here is pretty convoluted: the setup.py method of building was originally added primarily to support shipping python-bindings as a separate tarball, to support building multiple versions of the bindings. The former never happened, and to support Py 2 vs 3 people have been building the bindings in-tree with the setup.py method, which never really really worked. Now that that Python 2 is no longer supported at all, I think the solution to this is either axe the setup.py method entirely, or make it error-out if trying to build in-tree. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/130#issuecomment-605944390___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)
I've encountered a similar issue when I was building pythonX-rpm by hand on PCLinuxOS when I was attempting to upgrade them from 4.8.1 to 4.12.0.1. There didn't appear to be an obvious way to fix this, as the setuptools is wont to refer to system libraries for linking rather than the ones provided in the build. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/130#issuecomment-275408206___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)
I will make a patch for it -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/130#issuecomment-273552868___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint