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

Build fixes and Windows support guide #58

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

Conversation

Adrilihan
Copy link

A small pull request with a collection of fixes to the template. The most critical change is with the first commit, c08f629, where the quotes of the wildcard operator on the rm command were modified, but I tested and didn't 💥 my whole WSL system, so I think it works 🤞.

I also created a small setup guide on using this template on Windows with the use of WSL. You can find it here: gist link. Since this latter change would be done onto the README, I left for your consideration. Here is an example of how I would incorporate it:

...

A thesis LaTeX template that complies with the University of Aveiro's
guidelines and provides a simple CLI workflow around `make` that was developed
and tested for cross-compatibility on Linux (Slackware, ArchLinux, Debian, Ubuntu) and macOS.

The use of this template in a Windows is also possible but through the use of WSL, for a detailed 
walkthrough check this [gist](https://gist.github.com/Adrilihan/b1a73f1d126b3c87969f966eabd0f5d0)

This template was developed by professors and students. 
We will try to keep up to date with thesis requirements but some discrepancies may exist.
Feel free to open issues and pull requests with new options, packages and fixes.

...

./pedro

Fix misplaced quotes on the target definition of the rm command
The correct citation shown in the example was not defined in the
bibliography file. It has been updated accordingly.
Quotes around the build path variables on the Latexmk flags were
missing, resulting in not being able to build the project when the path
contained spaces.
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