This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author matheus.v.portela
Recipients a.badger, antonio, asandvig, eric.araujo, johnkeyes, matheus.v.portela, r.david.murray, tarek, xiscu
Date 2014-03-09.05:36:22
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1394343386.6.0.069625515695.issue809163@psf.upfronthosting.co.za>
In-reply-to
Content
As far as I noticed, in bdist_rpm.py, the _make_spec_file() method generates an .spec file for RPM build and attaches the install script to "setup.py install -O1 --root=$RPM_BUILD_ROOT --record=INSTALLED_FILES".

Later, the .spec refers to the record file as the one which contains all necessary files to build the RPM package with the directive "%files -f INSTALLED_FILES". If any path contains an space character, it will triggers an error.

I could fix this problem by attaching to the installation a small sed script that appends double quotes to the beginning and the end of each line of INSTALLED_FILES. I'm not sure whether this is allowed though. Any ideas how to make this more Pythonic without using sed?

Apparently, it works since the test passed. However, there is an warning which I have no idea what it means: "unknown, 0: Warning: using regular magic file `/etc/magic'".

I attached the diff file with the changes so far.
History
Date User Action Args
2014-03-09 05:36:27matheus.v.portelasetrecipients: + matheus.v.portela, tarek, antonio, eric.araujo, a.badger, r.david.murray, xiscu, asandvig, johnkeyes
2014-03-09 05:36:26matheus.v.portelasetmessageid: <1394343386.6.0.069625515695.issue809163@psf.upfronthosting.co.za>
2014-03-09 05:36:26matheus.v.portelalinkissue809163 messages
2014-03-09 05:36:25matheus.v.portelacreate