Opened 7 years ago
Closed 7 years ago
#16760 closed defect (fixed)
stopgap for #16759
Reported by: | jhpalmieri | Owned by: | |
---|---|---|---|
Priority: | critical | Milestone: | sage-6.3 |
Component: | packages: standard | Keywords: | |
Cc: | Merged in: | ||
Authors: | John Palmieri | Reviewers: | Jeroen Demeyer |
Report Upstream: | N/A | Work issues: | |
Branch: | b6711a4 (Commits, GitHub, GitLab) | Commit: | b6711a45ba18bc54cbf776d82f541ad08425476c |
Dependencies: | Stopgaps: |
Description
Here is a stopgap for #16759: turn off install_package
and print a message after installing a package via sage -i
.
Change History (15)
comment:1 Changed 7 years ago by
- Branch set to u/jhpalmieri/install
comment:2 Changed 7 years ago by
- Commit set to 836629cbea5e772bae77d869f1d7349b72647162
comment:3 Changed 7 years ago by
- Commit changed from 836629cbea5e772bae77d869f1d7349b72647162 to d0b1b82747c2a1c6a8fbf08f0c62c79d2b91f2fc
Branch pushed to git repo; I updated commit sha1. New commits:
d0b1b82 | Move message about running 'make' to install() in src/bin/sage
|
comment:4 Changed 7 years ago by
- Commit changed from d0b1b82747c2a1c6a8fbf08f0c62c79d2b91f2fc to e711f7753574b9739bf185931cb2314fa7254a47
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
e711f77 | Move message about running 'make' to install() in src/bin/sage
|
comment:5 Changed 7 years ago by
- Commit changed from e711f7753574b9739bf185931cb2314fa7254a47 to ad9994bee83cb50b3a681c4f5928a39e4afc8e18
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
ad9994b | Move message about running 'make' to install() in src/bin/sage
|
comment:6 Changed 7 years ago by
- Status changed from new to needs_review
Good idea. I think I'll mark this as "needs review" now, although I haven't tested it very thoroughly. I'm not even sure it's the right thing to do...
comment:7 Changed 7 years ago by
- Reviewers set to Jeroen Demeyer
comment:8 Changed 7 years ago by
This is sub-optimal, but I don't think it's a major problem:
jdemeyer@tamiyo:/usr/local/src/sage-config$ ./sage -i pexpect Found local metadata for pexpect-2.0.p6 Found local sources at /usr/local/src/sage-config/upstream/pexpect-2.0.tar.bz2 Package pexpect-2.0.p6 is already installed. Use 'sage -f /usr/local/src/sage-config/upstream/pexpect-2.0.tar.bz2' to force a reinstallation. It is a good idea to update the Sage library now: please run 'make' from $SAGE_ROOT.
comment:9 Changed 7 years ago by
- Branch changed from u/jhpalmieri/install to u/jdemeyer/ticket/16760
- Created changed from 08/05/14 01:23:06 to 08/05/14 01:23:06
- Modified changed from 08/05/14 14:52:40 to 08/05/14 14:52:40
comment:10 Changed 7 years ago by
- Commit changed from ad9994bee83cb50b3a681c4f5928a39e4afc8e18 to 4e12496227b7f704aa3b8dfbce4e4feb7b15cf46
A bit of a hack, but I think this does the job.
John, if you agree, then it's positive review.
New commits:
4e12496 | Show the "please run make" message only if we actually installed something
|
comment:11 Changed 7 years ago by
- Branch changed from u/jdemeyer/ticket/16760 to u/jhpalmieri/ticket/16760
comment:12 follow-up: ↓ 13 Changed 7 years ago by
- Commit changed from 4e12496227b7f704aa3b8dfbce4e4feb7b15cf46 to b6711a45ba18bc54cbf776d82f541ad08425476c
- Status changed from needs_review to positive_review
Okay, good enough. (I made one small change on top of your commit: "bij" --> "by" in the comment in src/bin/sage
.)
New commits:
b6711a4 | Typo
|
comment:13 in reply to: ↑ 12 Changed 7 years ago by
Replying to jhpalmieri:
"bij" --> "by" in the comment in
src/bin/sage
.
Classic mistake: "bij" is a Dutch word...
Upgrading priority, a simple stopgap warrants this.
comment:14 Changed 7 years ago by
- Priority changed from major to critical
comment:15 Changed 7 years ago by
- Branch changed from u/jhpalmieri/ticket/16760 to b6711a45ba18bc54cbf776d82f541ad08425476c
- Resolution set to fixed
- Status changed from positive_review to closed
I know this is not yet needs_review, but could you move the message
to
src/bin/sage
at the end of theinstall()
function? That way, it will only be shown after an explicit package install (sage -i
orsage -f
), but not during the normal build.New commits:
Stopgap for #16759: disable install_package