Opened 9 years ago

Closed 6 years ago

#15900 closed defect (fixed)

Move local/share/texmf/tex/generic/sagetex to local/share/texmf/tex/latex/sagetex

Reported by: Clemens Heuberger Owned by:
Priority: minor Milestone: sage-7.6
Component: user interface Keywords: sagetex, directory structure
Cc: Daniel Krenn, Dan Drake Merged in:
Authors: Paul Masson Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: c965348 (Commits, GitHub, GitLab) Commit: c965348bfd957f3a7e4062006d10b85698d81dcb
Dependencies: Stopgaps:

Status badges

Description (last modified by Frédéric Chapoton)

The TeX directory structure (TDS) reserves "generic" for macro files that do "not rely on any particular format" (cf. http://tug.org/tds/tds.html#Macros). This does certainly not apply to sagetex, which is clearly a latex package. Therefore, local/share/texmf/tex/generic/sagetex should be moved to local/share/texmf/tex/latex/sagetex and the documentation should be adapted to reflect this fact.

Apart from violating the TDS, this is also annoying: on my system, a prepackaged version of sagetex.sty comes with the package texlive-latex-extra (version 2009-10ubuntu1). While outdated, it is installed in a TDS compliant location (/usr/share/texmf-texlive/tex/latex/sagetex/sagetex.sty).

The standard search path for _latex_ is (removed non-pertinent items and inserted line breaks as separators)

$kpsepath -n latex tex |tr ':' '\n'
  .
  /home/[...]/texmf/tex/latex//
  !!/usr/local/share/texmf/tex/latex//
  !!/usr/share/texmf/tex/latex//
  /home/[...]/texmf/tex/generic//
  !!/usr/share/texmf/tex/generic//

which means that the systems version of sagetex.sty is given precedence over that in my homedirectory resulting from cp -R SAGE_ROOT/local/share/texmf/tex TEXMFHOME as recommended by http://www.sagemath.org/doc/installation/sagetex.html

I'd like to push a git branch correcting the problem, but sagetex seems to be developed outside the sage git respository?

Change History (15)

comment:1 Changed 9 years ago by Daniel Krenn

Cc: Daniel Krenn added

comment:2 Changed 9 years ago by John Palmieri

Cc: Dan Drake added

Here is the web page for sagetex. You could submit a pull request there, and meanwhile fix whatever needs fixing in the Sage library. (Maybe the Sage tutorial is the only place that needs modification? Edit: and the installation guide.)

cc'ing ddrake, who wrote and maintains sagetex.

Last edited 9 years ago by John Palmieri (previous) (diff)

comment:3 Changed 9 years ago by Dan Drake

This'll be easy to fix.

Two other notes...first, please try to update your TeX distribution! The sagetex.sty in the 2009 distribution has caused no end of trouble. It's necessary to keep the LaTeX and Python sides of SageTeX synchronized. Second, I guess I should move SageTeX over to github, since that's what all the cool kids are using.

comment:4 Changed 9 years ago by Dan Drake

Okay, I've made the fix and pushed it to bitbucket. If someone else wants to make a spkg, that's fine with me -- I'm busy this week and won't be able to work on it until next week.

comment:5 in reply to:  3 Changed 9 years ago by Daniel Krenn

Replying to ddrake:

It's necessary to keep the LaTeX and Python sides of SageTeX synchronized. Second, I guess I should move SageTeX over to github, since that's what all the cool kids are using.

It would be really nice to see sagetex fully inside the sage repository (so also on trac, which is IMHO mirrored on github).

comment:6 Changed 8 years ago by For batch modifications

Milestone: sage-6.2sage-6.3

comment:7 Changed 8 years ago by For batch modifications

Milestone: sage-6.3sage-6.4

comment:8 Changed 7 years ago by Frédéric Chapoton

Description: modified (diff)

comment:9 Changed 6 years ago by Karl-Dieter Crisman

See #22499 which might make this easier, as per comment:5.

comment:10 Changed 6 years ago by Paul Masson

This was fixed some time ago upstream and integrated into Sage. As of at least 7.6.beta5 the directory correctly reads "latex" instead of "generic".

Pushing a branch to update the documentation where needed.

comment:11 Changed 6 years ago by Paul Masson

Branch: u/paulmasson/15900

comment:12 Changed 6 years ago by Paul Masson

Commit: c965348bfd957f3a7e4062006d10b85698d81dcb
Milestone: sage-6.4sage-7.6
Status: newneeds_review

New commits:

c965348Update sagetex location

comment:13 Changed 6 years ago by Paul Masson

Authors: Paul Masson

comment:14 Changed 6 years ago by Frédéric Chapoton

Reviewers: Frédéric Chapoton
Status: needs_reviewpositive_review

lgtm

comment:15 Changed 6 years ago by Volker Braun

Branch: u/paulmasson/15900c965348bfd957f3a7e4062006d10b85698d81dcb
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.