Ticket #8144 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

SageTeX is not actually installed under SAGE_LOCAL

Reported by: mvngu Owned by: tbd
Priority: blocker Milestone: sage-4.3.2
Component: packages: standard Keywords: sagetex
Cc: ddrake Work issues:
Report Upstream: N/A Reviewers: Dan Drake
Authors: John Palmieri Merged in: sage-4.3.2.rc0
Dependencies: Stopgaps:

Description

Ticket #7617 adds SageTeX as a standard spkg. However, despite modifying the spkg dependency rules in SAGE_ROOT/spkg/install and SAGE_ROOT/spkg/standard/deps to account for this new package, SageTeX isn't actually installed at all in Sage 4.3.2.alpha1. This was reported on  sage-devel.

Attachments

deps Download (12.0 KB) - added by jhpalmieri 3 years ago.
deps.patch Download (448 bytes) - added by jhpalmieri 3 years ago.

Change History

comment:1 Changed 3 years ago by jhpalmieri

  • Priority changed from major to blocker
  • Status changed from new to needs_review
  • Authors set to John Palmieri

I think you just need to add a line for sagetex to the "all" section of deps. See the attached patch and new version of deps.

Changed 3 years ago by jhpalmieri

  • attachment deps Download added

Changed 3 years ago by jhpalmieri

comment:2 Changed 3 years ago by ddrake

  • Status changed from needs_review to positive_review
  • Reviewers set to Dan Drake

That does the trick. Positive review.

comment:3 Changed 3 years ago by mvngu

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-4.3.2.rc0

Copied  deps to SAGE_ROOT/spkg/standard and replaced the previous version of deps with this one.

Note: See TracTickets for help on using tickets.