Skip to content

Test \gresetgregpath #336

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 9, 2019
Merged

Conversation

rpspringuel
Copy link
Contributor

Instead of having an alias which translates into a copied file, the test is modified to borrow the gabc file from another test using the new \gresetgregpath mechanism.

Corresponds to gregorio-project/gregorio#1440

Instead of having an alias which translates into a copied file, the test is modified to borrow the gabc file from another test using the new `\gresetgregpath` mechanism.
@rpspringuel rpspringuel requested a review from henryso June 7, 2019 17:36
@rpspringuel
Copy link
Contributor Author

This does not test the kpse search implementation because testing that would require setting TEXINPUTS for a test. As there is no way of doing that from within a test, it would require some modification to the testing script itself. @henryso, if you have an idea as to how to do that, I'm all ears.

@henryso
Copy link
Contributor

henryso commented Jun 27, 2019

This seems to be a really specialized thing. Maybe create this as a scripted test?

@rpspringuel
Copy link
Contributor Author

Possibly. Unlike the other scripted tests, however, this would have to run lualatex, not just gregorio. I'm not sure that it would be necessary to compare the output, however. The point of the test would just be to see if lualatex could find the gabc file when TEXINPUTS is suitably defined.

@henryso henryso merged commit 7f9373e into gregorio-project:ctan Nov 9, 2019
@rpspringuel rpspringuel deleted the kpsewhich_search branch November 12, 2019 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants