Opened 10 years ago

Closed 5 months ago

#13197 closed defect (invalid)

make setuptools actually respect the --egg-path argument

Reported by: ohanar Owned by: GeorgSWeber
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: build Keywords:
Cc: kini, slelievre 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 ohanar)

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 follow-up: Changed 10 years ago by ohanar

  • Authors set to R. Andrew Ohana
  • Description modified (diff)
  • Status changed from new to needs_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 ohanar

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 ohanar

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

comment:4 Changed 10 years ago by ohanar

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

comment:5 Changed 10 years ago by ohanar

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

comment:6 Changed 9 years ago by kini

  • Cc kini added

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

comment:7 Changed 8 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:8 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:9 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:10 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:11 Changed 7 years ago by kcrisman

  • Priority changed from major to minor
  • Status changed from needs_review to needs_work

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

comment:12 Changed 18 months ago by mkoeppe

  • Authors R. Andrew Ohana deleted
  • Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
  • Status changed from needs_work to needs_review

Outdated, should be closed

comment:13 Changed 5 months ago by slelievre

  • Cc slelievre added
  • Reviewers set to Samuel Lelièvre
  • Status changed from needs_review to positive_review

Let us close this.

comment:14 Changed 5 months ago by mkoeppe

  • Resolution set to invalid
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.