Ticket #13197 (needs_review defect)

Opened 11 months ago

Last modified 9 months ago

make setuptools actually respect the --egg-path argument

Reported by: rohana Owned by: GeorgSWeber
Priority: major Milestone: sage-5.10
Component: build Keywords:
Cc: kini Work issues:
Report Upstream: Reported upstream. No feedback yet. Reviewers:
Authors: R. Andrew Ohana Merged in:
Dependencies: Stopgaps:

Description (last modified by rohana) (diff)

Currently if you specify the egg-path when using setuptools' setup.py develop, it still places the absolute path in easy_install.pth. This causes issues with relocation. Currently this only affects the sage notebook (which works around it by using a sed command), but with #13190 it will also affect the sage library. Since the easy_install.pth file is regenerated whenever setuptools detects that it needs to be, trying to use sed commands to work around this issue is unrealistic.

New SPKG:  http://wstein.org/home/ohanar/spkgs/setuptools-0.6.16.p1.spkg

Change History

comment:1 follow-up: ↓ 2 Changed 11 months ago by rohana

  • Status changed from new to needs_review
  • Description modified (diff)
  • Authors set to R. Andrew Ohana

I've attached an spkg which includes a work in progress patch that I intend to submit upstream once finished. It is currently functional for our purposes, hence the needs review.

comment:2 in reply to: ↑ 1 Changed 11 months ago by rohana

Ok, I updated it with the actual patch I'm going to submit upstream, so there should be no hesitation on reviewing it now. Same URL.

comment:3 Changed 11 months ago by rohana

  • Report Upstream changed from Not yet reported upstream; Will do shortly. to Reported upstream. No feedback yet.

comment:4 Changed 11 months ago by rohana

  • Report Upstream changed from Reported upstream. No feedback yet. to Not yet reported upstream; Will do shortly.

comment:5 Changed 11 months ago by rohana

  • Report Upstream changed from Not yet reported upstream; Will do shortly. to Reported upstream. No feedback yet.

comment:6 Changed 9 months ago by kini

  • Cc kini added

CCing myself since we should remove the sed stuff from the sagenb installer once this is in.

Note: See TracTickets for help on using tickets.