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

Error 43 with \setmainlanguage #24

Closed
WelliSolutions opened this issue Dec 7, 2023 · 5 comments · Fixed by #28
Closed

Error 43 with \setmainlanguage #24

WelliSolutions opened this issue Dec 7, 2023 · 5 comments · Fixed by #28

Comments

@WelliSolutions
Copy link

I get the following error and I don't know what I'm doing wrong. I'm just trying to compile the default invoice

Error producing PDF.
! Argument of \str_uppercase:n has an extra }.
<inserted text>
                \par
l.61 \setmainlanguage{}

make: *** [makefile:6: output.pdf] Error 43

I wonder a bit why it says line 61, because \setmainlanguage is on line 69 in template.tex

@sbwiecko
Copy link

sbwiecko commented Mar 7, 2024

I believe the error comes from the engine, try using xelatex, as proposed in #27 (comment)

@josephmure
Copy link
Contributor

josephmure commented Jul 30, 2024

I got the same error (using xelatex) and fixed it by switching polyglossia to babel, as suggested here: jgm/pandoc#7562. To do this,

  1. replace the following lines in template.tex:
\usepackage{polyglossia}
\setmainlanguage{$polyglossia-lang.name$}

with:

\usepackage[$lang$]{babel}
  1. replace the following line in details.yml:
lang: en-GB

with:

lang: english

josephmure added a commit to josephmure/invoice-boilerplate that referenced this issue Jul 30, 2024
@josephmure
Copy link
Contributor

I have proposed a fix at #28 but this project seems dead unfortunately...

@mrzool
Copy link
Owner

mrzool commented Jul 31, 2024

@josephmure I'm just very short on time and I apologize for that. I'll just assume that your PR works and merge it. Thanks a lot!

The sibling projects cv-boilerplate and letter-boilerplate both use polyglossia and could use the same fix. If you have time to put together similar PRs for them, I’d be happy to merge them!

@josephmure
Copy link
Contributor

@mrzool Thanks for merging! The fix works on my computer at least. I can propose the same fix for the other projects (although I haven't pulled them), but you will have to test whether it works with them too.

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 a pull request may close this issue.

4 participants