Opened 10 years ago

Closed 16 months ago

#13197 closed defect (invalid)

make setuptools actually respect the --egg-path argument

Reported by: R. Andrew Ohana Owned by: Georg S. Weber
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: build Keywords:
Cc: Keshav Kini, Samuel Lelièvre Merged in:
Authors: Reviewers: Samuel Lelièvre
Report Upstream: Reported upstream. No feedback yet. Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by R. Andrew Ohana)

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 (14)

comment:1 Changed 10 years ago by R. Andrew Ohana

Authors: R. Andrew Ohana
Description: modified (diff)
Status: newneeds_review

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 10 years ago by R. Andrew Ohana

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 10 years ago by R. Andrew Ohana

Report Upstream: Not yet reported upstream; Will do shortly.Reported upstream. No feedback yet.

comment:4 Changed 10 years ago by R. Andrew Ohana

Report Upstream: Reported upstream. No feedback yet.Not yet reported upstream; Will do shortly.

comment:5 Changed 10 years ago by R. Andrew Ohana

Report Upstream: Not yet reported upstream; Will do shortly.Reported upstream. No feedback yet.

comment:6 Changed 10 years ago by Keshav Kini

Cc: Keshav Kini added

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

comment:7 Changed 9 years ago by Jeroen Demeyer

Milestone: sage-5.11sage-5.12

comment:8 Changed 9 years ago by For batch modifications

Milestone: sage-6.1sage-6.2

comment:9 Changed 9 years ago by For batch modifications

Milestone: sage-6.2sage-6.3

comment:10 Changed 8 years ago by For batch modifications

Milestone: sage-6.3sage-6.4

comment:11 Changed 8 years ago by Karl-Dieter Crisman

Priority: majorminor
Status: needs_reviewneeds_work

What's the status here? No branch, for instance, though setuptools in Sage is now at 12.4.

comment:12 Changed 2 years ago by Matthias Köppe

Authors: R. Andrew Ohana
Milestone: sage-6.4sage-duplicate/invalid/wontfix
Status: needs_workneeds_review

Outdated, should be closed

comment:13 Changed 16 months ago by Samuel Lelièvre

Cc: Samuel Lelièvre added
Reviewers: Samuel Lelièvre
Status: needs_reviewpositive_review

Let us close this.

comment:14 Changed 16 months ago by Matthias Köppe

Resolution: invalid
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.