Opened 15 years ago

Closed 13 years ago

#11184 closed Bug (fixed)

scripts/rpm-install.sh does not respect options passed to setup.py

Reported by: pgriess@… Owned by: nobody
Component: Uncategorized Version: 1.0
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Setuptools supports changing the target installation environment when building an RPM (setup.py bdist_rpm) with the --python or --fix-python options. These have the effect of changing the path to the python executable to invoke in the generated .spec file.

However, Django's setup.py specifies a custom installation script in scripts/rpm-install.sh, which always invokes python directly:

...
# This is what dist.py normally does.
python setup.py install --root=${RPM_BUILD_ROOT} --record="INSTALLED_FILES"
...

As a result, it's impossible to generate a Django install to target non-default environments.

Attachments (1)

t11184-r11366.patch (527 bytes ) - added by rjc 15 years ago.
use ${python} instead of python in rpm-install.sh

Download all attachments as: .zip

Change History (7)

comment:1 by Karen Tracey, 15 years ago

milestone: 1.1

Not 1.1 material at this point.

comment:2 by anonymous, 15 years ago

This can be worked around by manually specifying the correct path in scripts/rpm-install.sh before running setup.py.

comment:3 by Alex Gaynor, 15 years ago

Triage Stage: UnreviewedAccepted

by rjc, 15 years ago

Attachment: t11184-r11366.patch added

use ${python} instead of python in rpm-install.sh

comment:4 by Julien Phalip, 13 years ago

Severity: Normal
Type: Bug

comment:5 by Aymeric Augustin, 13 years ago

Easy pickings: unset
Triage Stage: AcceptedReady for checkin
UI/UX: unset

comment:6 by Aymeric Augustin, 13 years ago

Resolution: fixed
Status: newclosed

In [17331]:

Fixed #11184 -- Made it possible to build RPMs with a non-default python executable.

Note: See TracTickets for help on using tickets.
Back to Top