Opened 3 years ago

Closed 17 months ago

#26343 closed defect (fixed)

Sagetex should unset TEXINPUTS when testing

Reported by: jhpalmieri Owned by:
Priority: minor Milestone: sage-9.1
Component: packages: standard Keywords:
Cc: kcrisman, jdemeyer, ddrake, vbraun, aapitzsch Merged in:
Authors: John Palmieri Reviewers: Matthias Koeppe
Report Upstream: N/A Work issues:
Branch: 594cce6 (Commits, GitHub, GitLab) Commit: 594cce604502a26e37e3acdbd92fa5195f14838b
Dependencies: Stopgaps:

Status badges

Description (last modified by jhpalmieri)

The test suite for sagetex should unset TEXINPUTS when testing. As it is, it runs latex on the wrong file example.tex: it tests one that happens to lie in a path in TEXINPUTS, and then it uses the version of sagetex.sty from elsewhere, also. (My TEXINPUTS has . at the end of the list of paths, not the beginning, so it looks there last. It has PATH_TO_SAGE/local/share/texmf//, where PATH_TO_SAGE is the location of my standard stable Sage installation, earlier in the list of paths.)

Change History (10)

comment:1 Changed 3 years ago by jhpalmieri

  • Branch set to u/jhpalmieri/sagetex-TEXINPUTS

comment:2 Changed 3 years ago by jhpalmieri

  • Commit set to a1a1cb978d48a45af7e5d856372a1fe54ef6d77d
  • Status changed from new to needs_review

New commits:

​a1a1cb9trac 26343: unset TEXINPUTS when testing sagetex.

comment:3 Changed 3 years ago by jhpalmieri

  • Description modified (diff)

comment:4 Changed 3 years ago by jhpalmieri

I don't know why latex file.tex looks for file.tex in places other than the current working directory, but it seems to. I knew it looked for style files elsewhere, but not the actual LaTeX files. I found that out by examining the sagetex-3.0 log file after running ./sage -f -c sagetex.

comment:5 Changed 3 years ago by jhpalmieri

  • Description modified (diff)

comment:6 Changed 3 years ago by jhpalmieri

  • Cc kcrisman jdemeyer ddrake vbraun aapitzsch added
  • Reviewers kcrisman, jdemeyer, ddrake, vbraun, aapitzsch deleted

comment:7 Changed 17 months ago by mkoeppe

  • Branch changed from u/jhpalmieri/sagetex-TEXINPUTS to u/mkoeppe/sagetex-TEXINPUTS

comment:8 Changed 17 months ago by mkoeppe

  • Commit changed from a1a1cb978d48a45af7e5d856372a1fe54ef6d77d to 594cce604502a26e37e3acdbd92fa5195f14838b
  • Milestone changed from sage-8.4 to sage-9.1
  • Reviewers set to Matthias Koeppe
  • Status changed from needs_review to positive_review

New commits:

​594cce6trac 26343: unset TEXINPUTS when testing sagetex.

comment:9 Changed 17 months ago by jhpalmieri

Thank you!

comment:10 Changed 17 months ago by vbraun

  • Branch changed from u/mkoeppe/sagetex-TEXINPUTS to 594cce604502a26e37e3acdbd92fa5195f14838b
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.