Opened 10 years ago
Closed 18 months ago
#13448 closed task (invalid)
source sage-env in spkg/install
Reported by: | ohanar | Owned by: | jason |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | misc | Keywords: | |
Cc: | jdemeyer, burcin, fbissey, dimpase, chapoton | Merged in: | |
Authors: | Reviewers: | ||
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | #13123 | Stopgaps: |
Description (last modified by )
Currently sage environmental variables are set in both sage-env
, and a subset are set in spkg/install
. This should be refactored. Also, since sage-spkg
is now always called from within the sage environment (as it should be), so it is no longer necessary to source sage-env
.
Installation instructions:
Apply attachment:trac13448.patch to the root repo.
Attachments (1)
Change History (14)
comment:1 Changed 10 years ago by
- Cc jdemeyer burcin fbissey added
- Dependencies set to #13123
- Description modified (diff)
- Status changed from new to needs_review
comment:2 Changed 10 years ago by
This is going to break badly because sage-env
is sourced only once (in this case, at the very beginning of the installation) but it needs to be re-sourced at various times during the installation because of environment variables which are conditionally set. This can be seen during upgrading for example, where sage-env
is sourced by sage --upgrade
, requiring various ugly work-arounds. To support upgrading, spkg/install
needs to keep working with an old sage-env
.
I agree there is a little bit of duplication between spkg/install
and sage-env
but I don't see a clean way to fix this. I'm open for suggestions though...
I propose to postpone this ticket, perhaps at a later time refactoring will be easier.
comment:3 follow-up: ↓ 4 Changed 10 years ago by
Hmm, you are right. I'll see if I can't come up with a clean way around that, but for now we should postpone this (not really sure the best way to mark that on trac though...)
comment:4 in reply to: ↑ 3 Changed 10 years ago by
- Milestone changed from sage-5.4 to sage-6.0
comment:5 Changed 10 years ago by
- Status changed from needs_review to needs_work
comment:6 Changed 9 years ago by
Typo on line 95 of sage-spkg
.
comment:7 Changed 8 years ago by
- Milestone changed from sage-6.0 to sage-6.1
comment:8 Changed 8 years ago by
- Milestone changed from sage-6.1 to sage-6.2
comment:9 Changed 8 years ago by
- Milestone changed from sage-6.2 to sage-6.3
comment:10 Changed 8 years ago by
- Milestone changed from sage-6.3 to sage-6.4
comment:11 Changed 22 months ago by
- Cc dimpase added
- Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
- Status changed from needs_work to needs_review
Outdated spkg or build system ticket, should be closed
comment:12 Changed 18 months ago by
- Cc chapoton added
comment:13 Changed 18 months ago by
- Resolution set to invalid
- Status changed from needs_review to closed
The dependency is only because I don't want to rebase #13123 on this.