Opened 9 years ago
Last modified 9 years ago
#14556 closed enhancement
The maxima spkg has cruft dating back to days of clisp use — at Version 8
| Reported by: | Snark | Owned by: | jdemeyer |
|---|---|---|---|
| Priority: | minor | Milestone: | sage-5.10 |
| Component: | packages: standard | Keywords: | noreadline |
| Cc: | fbissey | Merged in: | |
| Authors: | Julien Puydt | Reviewers: | Jeroen Demeyer |
| Report Upstream: | N/A | Work issues: | |
| Branch: | Commit: | ||
| Dependencies: | Stopgaps: |
Status badges
Description (last modified by )
The maxima spkg creates a maxima-noreadline script, which is an old hack. This should go away.
spkg: http://boxen.math.washington.edu/home/jdemeyer/spkg/maxima-5.29.1.p3.spkg (spkg diff p2 + spkg diff p3)
Change History (11)
comment:1 in reply to: ↑ description Changed 9 years ago by
- Description modified (diff)
comment:2 Changed 9 years ago by
- Cc fbissey added
comment:3 Changed 9 years ago by
- Status changed from new to needs_review
The two attached patches should fix this ticket.
comment:4 Changed 9 years ago by
- Description modified (diff)
- Keywords noreadline added
You should also provide a "pre-patched" new spkg, with SPKG.txt updated as well.
comment:5 Changed 9 years ago by
I updated the patch, and the patched spkg can be found here
comment:6 Changed 9 years ago by
- Description modified (diff)
comment:7 Changed 9 years ago by
- Description modified (diff)
Changed 9 years ago by
comment:8 Changed 9 years ago by
- Description modified (diff)
- Reviewers set to Jeroen Demeyer
Positive review to your changes.
However, I added some more small changes to the maxima spkg, please review.
Note: See
TracTickets for help on using
tickets.
Replying to Snark:
http://boxen.math.washington.edu/home/release/sage-5.10.beta2/sage-5.10.beta2/spkg/standard/maxima-5.29.1.p1.spkg