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

Add --ignore-errors flag to run.py + W10 exe #103

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

VadeveSi
Copy link

@VadeveSi VadeveSi commented May 25, 2021

Hi,

I propose the following changes:

  • Add an --ignore-errors option to the run.py script, which avoid the sys.exit whenever a command fails due to errors.
  • Fixed wrong run.py arg in README
  • Add W10 exe (that doesn't require prior Python installation)
  • Update README with instructions on how to include the run script in TexWorks.
  • Delete glossary.sty, as I'm fairly sure this is no longer used (instead, the standard glossaries is used).

One of my Windows-loving colleagues (who shall remain unnamed) was having troubles with the glossary of the LaTeX template, so of course I was asked to fix them (I am not writing my thesis yet myself 🙂 ).
Turns out the issue was with TexWorks not supporting the makeindex out of the box.

I settled on the following solution: introduce the --ignore-errors flag (so that the script doesn't stop when it hits one of their numerous errors which will be fixed later), generate a W10 exe and add a custom tool in TexWorks which runs the exe.

This way, the entire LaTeX "process" is always run in one single sweep, from inside TexWorks itself.

The W10 exe is built in a W10 VM using PyInstaller:

pyinstaller run.py --onefile

(Minor remark: I changed all tabs to 4 spaces in run.py, because my Python was complaining about mixed indentation. If requested, I can revert the file back to tabs of course.)

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.

1 participant