Ticket #5900 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

[with patches, positive review] Add support for system python != Sage python

Reported by: tabbott Owned by: tabbott
Priority: major Milestone: sage-4.1.2
Component: debian-package Keywords:
Cc: Work issues:
Report Upstream: Reviewers: Mike Hansen
Authors: Tim Abbott Merged in: Sage 4.1.2.alpha2
Dependencies: Stopgaps:

Description

On Ubuntu jaunty, the system Python is 2.6 but Sage is built with Python 2.5. This results in problems in a few places where Sage directly invokes a python program rather than running it via python. For example, running "trial" rather than "python $(which trial)" would result in "trial" being started with Python 2.6.

I've attached the set of patches that I applied in order to deal with this issue in Jaunty. I believe they should be harmless for Sage, since it puts $SAGE_LOCAL at the start of PATH, ahead of any system copies of trial/twistd/etc. that might exist.

Attachments

sage_scripts-dsage-trial.patch Download (726 bytes) - added by tabbott 4 years ago.
sage_scripts-twistd.patch Download (600 bytes) - added by tabbott 4 years ago.

Change History

Changed 4 years ago by tabbott

Changed 4 years ago by tabbott

comment:1 Changed 4 years ago by mhansen

  • Reviewers set to Mike Hansen
  • Summary changed from [with patches, needs review] Add support for system python != Sage python to [with patches, positive review] Add support for system python != Sage python
  • Authors set to Tim Abbott

They look good to me. I'm not sure if they are still relevant since Sage switched to Python 2.6, but they shouldn't hurt.

comment:2 Changed 4 years ago by mvngu

  • Status changed from new to closed
  • Resolution set to fixed
  • Merged in set to Sage 4.1.2.alpha2

comment:3 Changed 4 years ago by mvngu

Applied to the bin repository.

Note: See TracTickets for help on using tickets.