-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Fix #234: redefinition of \FA, upgrade to fontawesome5 #344
Conversation
This commit fixes an issue when using recent fontspec versions. Described in: https://tex.stackexchange.com/a/453198/73121 Closes posquit0#234.
I wish I had checked PRs before spending hours trying to figure out why I couldn't compile the PDFs. Anyway, I approached the same issue you're talking about in a different way so it's definitely a problem that exists and has not been addressed. It may be useful for you to merge my approach with yours so that Github Actions auto generates PDFs on each new push, saving you the trouble of having to have the TeX environment set up locally. |
@vpatel9202 I'm not sure that relates to this issue, what is there to render from awesome-cv itself? Funnily enough though my own CV is also GitHub Action, and formerly Circle CI, rendered/released. I have an auto-updating texlive image at OJFord/docker-texlive for the purpose too, in case that's of any help. I don't think this PR needs anything to do with how it's rendered though, this is just fixing an error that occurs however (so far as I can tell) the rendering is done. |
The way I solved the font issue was to install the font package into the docker container that does the rendering (which, admittedly seems like a roundabout way to do it comparing to how you remedied the situation). So we're having the same issue with TeX throwing errors when it can't finding FontAwesome, but we solved it in different ways. I think your approach is more direct and solves problem with Awesome-CV itself while I was utilizing a docker to get around the problem. |
Should the symlink to |
Quite possibly @dcsim, I'll check later (writing from phone). Thanks for all your help in issues recently by the way! |
Oh yes, for sure. Thanks! |
Leftover in posquit0#344.
This was missed in posquit0#344 & 3280a87. Closes posquit0#399.
My previous PR #235 was closed as 'invalid duplicate bug', which I'm not quite sure what means, and this seems to remain unfixed to a lot of people including me.
I tried master again, but still had the issue; rebased my changes, and it still fixes it. (Tested Linux & macOS.)
If this fix is 'invalid' for some reason, or it should work as-is I'm just doing something wrong, please, let me know - I'd love to be using
awesome-cv
properly so that I don't have to maintain my fork, or if a fix is needed just not this one, I can try to improve it.Thanks.