Closed cbarrick closed 2 years ago
Can this be merged please? This fixes the problem I'm having as well
Thank you btw!
I dare bumping this, it's a simple fix and without it, the examples wouldn't even compile.
I'm surprised this hasn't been looked at since. @posquit0
This should be merged! @posquit0
This saved my day. Thanks
This is a great fix! It saved my day too.Thanks.
Thanks - this worked for me Using XeLatex on Windows 10
Thank you - I think many of us have made this or similar fixes over the years as posquit0's had other priorities. I have push access now to help maintain Awesome-CV; so I went with my old PR for this (#235/#344) for no reason other than my own familiarity, that I know it's been working for me.
Please do ping me/open another PR if something's still not right or I missed something though! Cheers.
The
\FA
font family is declared both infontawesome.sty
(without a path) and inawesome-cv.cls
(with a path). This confuses xelatex and produces a font-not-found error if Font Awesome is not installed globally. It is fixed by only using the declaration with a path.This closes #181 and #221.
This is an alternative to #246. That PR un-defines the first
\FA
before redefining it. This PR simply removes the first definition.EDIT: It looks like I tried to push this same change about a year ago in #191. I'll close that in favor of this one since the commit message is better and it's based on the current master.
Here is the full output, including the error, when calling
make examples
on current master: