bdist_rpm should use rpmbuild, not rpm #55331

purpleidea · 2011-02-04T21:19:22Z

BPO	11122
Nosy	@ericvsmith, @tarekziade, @ned-deily, @merwok, @petere, @stratakis, @csabella, @bhyde, @miss-islington
PRs	bpo-11122: always use rpmbuild in bdist_rpm #10594 [3.8] bpo-11122: fix hardcoded path checking for rpmbuild in bdist_rpm.py (GH-10594) #13922
Files	fix-rpm-use-2.7.diff

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/merwok'
closed_at = <Date 2019-06-08.21:06:22.873>
created_at = <Date 2011-02-04.21:19:22.059>
labels = ['easy', '3.8', '3.7', 'library']
title = 'bdist_rpm should use rpmbuild, not rpm'
updated_at = <Date 2019-06-08.21:25:24.644>
user = 'https://bugs.python.org/purpleidea'

bugs.python.org fields:

activity = <Date 2019-06-08.21:25:24.644>
actor = 'miss-islington'
assignee = 'eric.araujo'
closed = True
closed_date = <Date 2019-06-08.21:06:22.873>
closer = 'cheryl.sabella'
components = ['Distutils']
creation = <Date 2011-02-04.21:19:22.059>
creator = 'purpleidea'
dependencies = []
files = ['34407']
hgrepos = []
issue_num = 11122
keywords = ['patch', 'easy']
message_count = 25.0
messages = ['127942', '127943', '127944', '127945', '127948', '127958', '127959', '128005', '128052', '128053', '153098', '155120', '175042', '177241', '193105', '213484', '213588', '315180', '315188', '315197', '315198', '315200', '330061', '345063', '345067']
nosy_count = 13.0
nosy_names = ['eric.smith', 'tarek', 'ned.deily', 'eric.araujo', 'purpleidea', 'petere', 'westley.martinez', 'James.Bennet', 'cstratak', 'cheryl.sabella', 'bhyde', 'miss-islington', 'stanp']
pr_nums = ['10594', '13922']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue11122'
versions = ['Python 3.7', 'Python 3.8']

purpleidea · 2011-02-04T21:19:22Z

Hi distutils,

When I run:

./setup.py bdist --formats=rpm

on my source directory, I get the error:

rpm -ba --define _topdir /home/james/code/scantran/build/bdist.linux-x86_64/rpm --clean build/bdist.linux-x86_64/rpm/SPECS/scantran.spec
-ba: unknown option
error: command 'rpm' failed with exit status 1

It seems the problem can be fixed by installing the rpm-build package (on fedora 14 anyways) and so perhaps something should be done so that this is more obvious...

HTH,
James

merwok · 2011-02-04T21:23:42Z

Does this mean that there is one rpm command that does not have all options, and that rpm-build overwrites that command with another one that does the right thing?

purpleidea · 2011-02-04T21:32:13Z

In the source for distutils it seems to attempt to use 'rpmbuild' if it exists, but otherwise falls back on regular 'rpm', however in my rpm:

$ rpm --version
RPM version 4.8.1

this fails as there is no -ba option.

James

merwok · 2011-02-04T21:37:33Z

What is your OS name and version? Is that rpm version considered obsolete or not?

ericvsmith · 2011-02-04T22:45:24Z

"rpm -ba" is very old. I think it's been at least since 2002 that "-ba" was supported by rpm.

I believe bdist_rpm is calling "rpmbuild -ba", and if that doesn't exist, then "rpm -ba" is a transition strategy until all systems supported "rpmbuild". This transition period should be over. Unfortunately today it's far more likely that "rpm -ba" will not be what you want to call, but "rpm" will almost always be found, giving this error.

Most systems (certain RedHat based) ship rpmbuild as a separate command. RedHat installs it via the rpm-build package.

I'd suggest just switching bdist_rpm to always use rpmbuild. Then at least the error would be that rpmbuild isn't found.

merwok · 2011-02-05T01:01:18Z

This is a gray area between bugfix and bahavior change forbidden by the feature freeze. Would it be okay for you to add a note in the docs and close this report?

merwok · 2011-02-05T01:02:31Z

Would it be okay for you to add a note in the docs and close this report?

I meant to ask whether you would agree with that outcome, not that you should do the work (you’re welcome to provide a patch if you want to, but that’s not required).

purpleidea · 2011-02-05T17:41:12Z

I'll write a docs and script patch for this next week...

I'm happy to do the work,

Thanks for the comments.

James

merwok · 2011-02-06T12:02:02Z

Thanks for volunteering. I will review and apply when the py3k branch is unfrozen.

I'll write a docs and script patch for this next week...
Note that I proposed to patch the docs only.

merwok · 2011-02-06T12:05:29Z

BTW, it may be useful to point people to http://pypi.python.org/pypi/pypi2rpm, which contains an updated command, bdist_rpm2.

merwok · 2012-02-11T05:25:58Z

Given that we’ve made changes to distutils for Debian multiarch or Mac OS X compiler breakage, I now think that switching distutils to only use rpmbuild would be reasonable.

merwok · 2012-03-07T20:54:47Z

More info on rpm vs. rpmbuild: http://bugs.python.org/issue1533164#msg82592

petere · 2012-11-07T04:50:19Z

I ran into a similar instance of this problem today and would like to add my support for just getting rid of the "rpm" calls and just call "rpmbuild" in all cases. The last release where "rpm" was used for building was more than 10 years ago.

ericvsmith · 2012-12-09T22:02:46Z

I agree on just switching to rpmbuild, at least for 3.4.

JamesBennet · 2013-07-15T14:28:44Z

I have observed this bug under CentOS 5.9 using the version of python-setuptools from the official CentOS repository. -ba is not a valid option for that version of RPM. I am able to get further by installing the rpm-build package.

merwok · 2014-03-13T22:22:17Z

Can someone test the attached patch for 2.7? I don’t know if the second use (rpm -q --qf etc) should be converted too.

petere · 2014-03-14T20:15:51Z

No, the second use should not be converted.

stanp · 2018-04-10T21:03:12Z

can this patch be sped up?
i am running into this issue with the latest python version
the patch was submitted 7 years ago

i don't see any workaround.

stratakis · 2018-04-11T09:55:08Z

I could make a PR out of the patch but I wonder if anyone is actually using this functionality. Wearing my Fedora's python maintainer hat, we have a lot of tooling around rpm that handles those things, so I don't think there is a valid usecase for Fedora at least. Is it useful somehow for other rpm based distros or is the resulting rpm's functional enough for some testing at least?

stanp · 2018-04-11T17:30:10Z

i am using this on redhat enterprise and mac. both running python 3.5
both running latest rpm. if you can make a pr it would great.

stratakis · 2018-04-11T17:34:57Z

The bug exists indeed but I would like to know what is the actual use case here.

stanp · 2018-04-11T20:20:01Z

the use case is being able to publish python package as rpm using setup.py on mac.

ned-deily · 2018-11-18T19:29:03Z

Note that bpo-32281 also describes the problem of trying to use build_rpm on non-rpm systems, again specifically macOS. Before stumbling on this issue,I had suggested over there to use change bdist_rpm to not use hardwired paths for rpmbuild but, if the better solution is to just unconditionally use rpmbuild, let's do that instead.

csabella · 2019-06-08T21:05:10Z

New changeset 45a1494 by Cheryl Sabella (Marcin Niemira) in branch 'master':
bpo-11122: fix hardcoded path checking for rpmbuild in bdist_rpm.py (GH-10594)
45a1494

miss-islington · 2019-06-08T21:25:25Z

New changeset 3f7629d by Miss Islington (bot) in branch '3.8':
bpo-11122: fix hardcoded path checking for rpmbuild in bdist_rpm.py (GH-10594)
3f7629d

purpleidea mannequin assigned tarekziade Feb 4, 2011

purpleidea mannequin added the stdlib Python modules in the Lib dir label Feb 4, 2011

merwok assigned merwok and unassigned tarekziade Feb 6, 2011

merwok added the easy label Mar 7, 2012

merwok changed the title ~~bdist_rpm fails~~ bdist_rpm should use rpmbuild, not rpm Mar 7, 2012

stanp mannequin added 3.7 (EOL) end of life 3.8 only security fixes labels Apr 10, 2018

csabella closed this as completed Jun 8, 2019

ezio-melotti transferred this issue from another repository Apr 10, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

bdist_rpm should use rpmbuild, not rpm #55331

bdist_rpm should use rpmbuild, not rpm #55331

purpleidea mannequin commented Feb 4, 2011

purpleidea mannequin commented Feb 4, 2011

merwok commented Feb 4, 2011

purpleidea mannequin commented Feb 4, 2011

merwok commented Feb 4, 2011

ericvsmith commented Feb 4, 2011

merwok commented Feb 5, 2011

merwok commented Feb 5, 2011

purpleidea mannequin commented Feb 5, 2011

merwok commented Feb 6, 2011

merwok commented Feb 6, 2011

merwok commented Feb 11, 2012

merwok commented Mar 7, 2012

petere mannequin commented Nov 7, 2012

ericvsmith commented Dec 9, 2012

JamesBennet mannequin commented Jul 15, 2013

merwok commented Mar 13, 2014

petere mannequin commented Mar 14, 2014

stanp mannequin commented Apr 10, 2018

stratakis mannequin commented Apr 11, 2018

stanp mannequin commented Apr 11, 2018

stratakis mannequin commented Apr 11, 2018

stanp mannequin commented Apr 11, 2018

ned-deily commented Nov 18, 2018

csabella commented Jun 8, 2019

miss-islington commented Jun 8, 2019

bdist_rpm should use rpmbuild, not rpm #55331

bdist_rpm should use rpmbuild, not rpm #55331

Comments

purpleidea mannequin commented Feb 4, 2011

purpleidea mannequin commented Feb 4, 2011

merwok commented Feb 4, 2011

purpleidea mannequin commented Feb 4, 2011

merwok commented Feb 4, 2011

ericvsmith commented Feb 4, 2011

merwok commented Feb 5, 2011

merwok commented Feb 5, 2011

purpleidea mannequin commented Feb 5, 2011

merwok commented Feb 6, 2011

merwok commented Feb 6, 2011

merwok commented Feb 11, 2012

merwok commented Mar 7, 2012

petere mannequin commented Nov 7, 2012

ericvsmith commented Dec 9, 2012

JamesBennet mannequin commented Jul 15, 2013

merwok commented Mar 13, 2014

petere mannequin commented Mar 14, 2014

stanp mannequin commented Apr 10, 2018

stratakis mannequin commented Apr 11, 2018

stanp mannequin commented Apr 11, 2018

stratakis mannequin commented Apr 11, 2018

stanp mannequin commented Apr 11, 2018

ned-deily commented Nov 18, 2018

csabella commented Jun 8, 2019

miss-islington commented Jun 8, 2019