Skip to content
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

Tmp gre #1543

Merged
merged 6 commits into from Mar 19, 2024
Merged

Tmp gre #1543

merged 6 commits into from Mar 19, 2024

Conversation

rpspringuel
Copy link
Contributor

Trying to solve some errors when it come to compilations of files in unusual locations.

Addresses several issues: #1542 #1541 #1393

@jperon and @Nokel81 I would appreciate it if you could give this a try and see if your particular issue gets solved in an acceptable manner by this change.

I'll add documentation after receiving feedback on the fix.

Some errors for gregorio print to stderr but do not write to glog.  This leads to the error being lost when autocompiling, so we add a redirect which will capture the stderr output and put it in glog.
When compiling gabc files, tmp-gre will be created in the cwd and gtex and glog files will be written there.  This allows for compilation of gabc files which are outside the current tree and should allow for compilation of tex files from outside outside of the project folder.
@rpspringuel
Copy link
Contributor Author

This fixes the issues I've been having with directory writing problems. I haven't been able to test all the variations though, so I would appreciate it if @jperon, @Nokel81, and @MaestroGlanz would please check their particular circumstances to see if this works for them.

@Nokel81
Copy link

Nokel81 commented Oct 16, 2023

I'll see if I have some time to set up a windows development environment

@rpspringuel
Copy link
Contributor Author

I've been using this branch on a project of mine since creating this pull request without problems so I'm going to merge on Monday if no one objects.

Copy link
Contributor

@henryso henryso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't fully understand what the lua code is doing (on the TeX side), but since this is battle-tested, I'm ok with the merge.

@eschwab
Copy link
Contributor

eschwab commented Mar 8, 2024

Looks fine to me. So much boilerplate dealing with / vs \

@rpspringuel rpspringuel merged commit 2a22c58 into gregorio-project:ctan Mar 19, 2024
@rpspringuel rpspringuel deleted the tmp-gre branch March 19, 2024 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants