-
Notifications
You must be signed in to change notification settings - Fork 320
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
Bugfix/9816 #9827
Bugfix/9816 #9827
Conversation
Nice idea! Is this expected that the test passes on windows though ? I thought it would need the deps. I know how to install it, but I am surprised it worked already |
I'm surprised too! I read somewhere that pdftotext is installed with miktex; I wonder if tinytex also installs it? |
Oh lets me check that ! highly possible ! |
So locally I need to install it at least. Otherwise,
But when I do have, it works ok. So it seems that somehow, |
I have got the answer ! It is available with Git For Windows as part of mingw64 distribution > gcm pdftotext
CommandType Name Version Source
----------- ---- ------- ------
Application pdftotext.exe 0.0.0.0 C:\Program Files\Git\mingw64\bin\pdftotext.exe So it is found ! Though version available is 4.00 > pdftotext --version
pdftotext version 4.00
Copyright 1996-2017 Glyph & Cog, LLC But as long as it works ok ! I think we should be good. If you prefer to be safe, installing as I mentioned in other thread will be the way to go. Just tells me |
This is good enough for me! |
Closes #9816.
This will fail CI because
pdftotext
isn't installed, so that needs fixing.