Iftex is not installed, then setting the program name to it will fail to find the 'program' in the current directory, and then be unable to find any TeX files.

Setting it tolatex should be okay, since that's what we're running to create the.dvi in the first place.

PR Checklist

Tests and Styling

[n/a] Has pytest style unit tests (andpytest passes).
IsFlake 8 compliant (installflake8-docstrings and runflake8 --docstring-convention=all).

Documentation

[n/a] New features are documented, with examples if plot related.
[n/a] New features have an entry indoc/users/next_whats_new/ (follow instructions in README.rst there).
[n/a] API changes documented indoc/api/next_api_changes/ (follow instructions in README.rst there).
[n/a] Documentation is sphinx and numpydoc compliant (the docs shouldbuild without error).

QuLogic added the topic: text/usetex label

Jan 29, 2022

QuLogic added this to thev3.5.2 milestone

Jan 29, 2022

QuLogic requested a review fromanntzer

January 29, 2022 05:30

Use latex as the program name for kpsewhich

3a11df4

If `tex` is not installed, then setting the program name to it will failto find the 'program' in the current directory, and then be unable tofind any TeX files.Setting it to `latex` should be okay, since that's what we're runningto create the `.dvi` in the first place.

QuLogic force-pushed theluatex-kpsewhich branch from30cecaf to3a11df4Compare

January 29, 2022 05:31

QuLogic changed the title~~Use luatex as the program name for kpsewhich~~Use latex as the program name for kpsewhich

Jan 29, 2022

Copy link

MemberAuthor

QuLogic commentedJan 29, 2022

I also just noticed thatcheckdep_usetex checks fortex, but I don't see us runningtex, onlylatex; should that be switched too?

anntzer approved these changes

Jan 29, 2022

View reviewed changes

Copy link

Contributor

anntzer commentedJan 29, 2022

I don't think I have seen a case where latex is installed and not tex, but sure, this seems reasonable.

Also agreed that checkdep_usetex should probably check latex indeed.

timhoffm approved these changes

Jan 30, 2022

View reviewed changes

timhoffm merged commit6eba0af intomatplotlib:main

Jan 30, 2022

meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request

Jan 30, 2022

Backport PRmatplotlib#22349: Use latex as the program name for kpsew…

67334ab

…hich

meeseeksmachine mentioned this pull request

Jan 30, 2022

Backport PR #22349 on branch v3.5.x (Use latex as the program name for kpsewhich)#22358

Merged

timhoffm added a commit that referenced this pull request

Jan 30, 2022

Merge pull request#22358from meeseeksmachine/auto-backport-of-pr-22…

39a3802

…349-on-v3.5.xBackport PR#22349 on branch v3.5.x (Use latex as the program name for kpsewhich)

QuLogic deleted the luatex-kpsewhich branch

January 31, 2022 22:54

Copy link

MemberAuthor

QuLogic commentedJan 31, 2022

I don't think I have seen a case where latex is installed and not tex, but sure, this seems reasonable.

It is the case on my (new-ish) laptop, as I did not install all of texllive, or even a specific scheme, just the few.sty, etc. files I knew were needed for Matplotlib tests.