[issue14443] Distutils test_bdist_rpm failure

Dave Malcolm added the comment: Fixes committed to 3.2 and default: RHEL 6.3 is now green for both branches. Closing this bug out. -- resolution: -> fixed stage: needs patch -> committed/rejected status: open -> closed ___ Python tracker

[issue14443] Distutils test_bdist_rpm failure

Roundup Robot added the comment: New changeset 76f2b99f3568 by David Malcolm in branch '3.2': Issue #14443: ensure that brp-python-bytecompile is invoked with the correct http://hg.python.org/cpython/rev/76f2b99f3568 -- ___ Python tracker

[issue14443] Distutils test_bdist_rpm failure

Roundup Robot added the comment: New changeset a9a12ad553f0 by David Malcolm in branch 'default': Issue #14443: ensure that brp-python-bytecompile is invoked with the correct http://hg.python.org/cpython/rev/a9a12ad553f0 -- ___ Python tracker

[issue14443] Distutils test_bdist_rpm failure

Dave Malcolm added the comment: (with the patch, it also continues to work on a test Fedora 15 box) -- ___ Python tracker ___ ___ Pyt

[issue14443] Distutils test_bdist_rpm failure

Dave Malcolm added the comment: The approach Nick suggests seems reasonable, but rpm.expandMacro isn't usable: there's no guarantee that the machine has Python bindings for rpm installed (especially not for the version of Python that we've just built). However, we can get at the macro by shel

[issue14443] Distutils test_bdist_rpm failure

Georg Brandl added the comment: Ping. I would like to have this test failure fixed for 3.3 rc. -- nosy: +georg.brandl priority: normal -> deferred blocker ___ Python tracker __

[issue14443] Distutils test_bdist_rpm failure

Nick Coghlan added the comment: Now I'm wondering if the problem will also manifest itself on EL rebuilds like CentOS or Scientific Linux, since that will affect how specific we can get when it comes to implementing a workaround. That is, overriding __os_install_post should definitely work, b

[issue14443] Distutils test_bdist_rpm failure

Dave Malcolm added the comment: I'm able to reproduce this on a RHEL 6 box, and I did some investigating. The stray .pyc files are indeed reported by "file" as "python 2.6 byte-compiled" so yes, it's using /usr/bin/python to byte-compile them On RHEL 6, with redhat-rpm-config-9.0.3-33.el6.no

[issue14443] Distutils test_bdist_rpm failure

Nick Coghlan added the comment: Dave, sending this one back in your direction. Is there a relevant difference between rpmbuild in RHEL6 and rpmbuild in current Fedora? -- assignee: eric.araujo -> dmalcolm ___ Python tracker

[issue14443] Distutils test_bdist_rpm failure

Ross Lagerwall added the comment: Unfortunately, it seems like it's still failing on the RHEL 6 buildbot. -- status: pending -> open ___ Python tracker ___ _

[issue14443] Distutils test_bdist_rpm failure

Nick Coghlan added the comment: Applied to 3.2 and trunk (I forgot to mention the issue name in the merge commit message). I tweaked Ross's patch slightly to: 1. Use self.python rather than sys.executable 2. Also default self.python to "python3" if neither --python nor --fix-python are specif

[issue14443] Distutils test_bdist_rpm failure

Roundup Robot added the comment: New changeset 3d61e27cc570 by Nick Coghlan in branch '3.2': Issue #14443: Tell rpmbuild to use the correct version of Python http://hg.python.org/cpython/rev/3d61e27cc570 -- nosy: +python-dev ___ Python tracker

[issue14443] Distutils test_bdist_rpm failure

Ross Lagerwall added the comment: The attached patch fixed the test for me on Fedora 16. It was necessary for the `define` to be after the -ba switch. I don't know why this wouldn't work on RHEL6 then... -- keywords: +patch Added file: http://bugs.python.org/file25650/distutils.patch

[issue14443] Distutils test_bdist_rpm failure

Nick Coghlan added the comment: I tried the simple fix a couple of different ways on the RHEL6 buildbot. First by changing line 315 of Lib/distutils/command/bdist_rpm.py" to be: rpm_cmd = ['rpmbuild', '--define', '__python %s' % sys.executable] And then a second time by adding the new "--

[issue14443] Distutils test_bdist_rpm failure

Dave Malcolm added the comment: __os_install_post is defined within /usr/lib/rpm/redhat/macros and contains this fragment: /usr/lib/rpm/brp-python-bytecompile %{__python} %{?_python_bytecompile_errors_terminate_build} \ Hence it will use %{__python} as the default when byte-compiling. %_

[issue14443] Distutils test_bdist_rpm failure

Dave Malcolm added the comment: As a post-processing step, rpmbuild will attempt to byte-compile any .py files it encounters, and the results must be listed in the %files manifest. [1] This is done by the script brp-python-bytecompile, which uses the compileall module. However, my guess is t

[issue14443] Distutils test_bdist_rpm failure

Nick Coghlan added the comment: Antoine (added to nosy list) indicated he wasn't seeing the failure on a Mageia system, so that's another point in favour of a Fedora/RHEL specific problem. Also added Dave Malcolm as the Fedora Python package maintainer. Applying #11599 to get the full failing

[issue14443] Distutils test_bdist_rpm failure

Éric Araujo added the comment: > I'm wondering if there may be a deeper problem here: how certain are we that > bdist_rpm isn't using the system Python > to handle the byte compilation step? It would explain why the files are still > being generated in the old locations. I’ve had a quick look