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

running on travis #8

Closed
physikerwelt opened this issue May 8, 2019 · 11 comments
Closed

running on travis #8

physikerwelt opened this issue May 8, 2019 · 11 comments

Comments

@physikerwelt
Copy link

The program runs great on my local computer. However, when I tried to run it on the CI service Travis, only the main tex file and the bbl file were included. The image file included files and others included tex files were not included. Do you by chance have any experience to run the arxiv-collector from Travis?

@djsutherland
Copy link
Owner

I haven't tried it, but sounds like some issue with latexmk there. Try running with --debug and/or posting the output of latexmk -pdf -deps WHATEVER.tex, which is what this script parses to look at what should be included.

@physikerwelt physikerwelt mentioned this issue May 17, 2019
@physikerwelt
Copy link
Author

I have created a pull request #9 that is not supposed to be merged but to demonstrate the problem. It would great if we could integrate your arxiv-collector to our open-science template https://github.com/ag-gipp/OpenScienceTemplate

@djsutherland
Copy link
Owner

It seems it was caused by a synctex message that doesn't end in a newline for some reason, so (depending on the local compilation state) the Dependents line that arxiv-collector looks for wasn't on its own line. This should be fixed in the new 0.2.3 release, but I'll of course reopen if not.

And you're of course welcome to use it in open-science template; let me know if there's anything I should change to make it easier for you, or you run into any other problems. :)

@physikerwelt
Copy link
Author

the same problem remains with the current master branch
see
https://travis-ci.org/physikerwelt/arxiv-collector/builds/535311964#L2077-L2087

@djsutherland djsutherland reopened this May 21, 2019
@djsutherland
Copy link
Owner

Ohhhh, huh. This is actually a weird different issue. Working on a fix now.

@djsutherland
Copy link
Owner

It's not what I thought at first – it actually seems to be that the most recent versions of latexmk have broken -deps. From http://personal.psu.edu/jcc8//software/latexmk-jcc/versions.html, it looks like it broke in between version 4.61 (last October) and 4.63b (last March). It's not something about your paper in particular, it happens even on a trivial test paper.

I'll report the bug to the author. In the meantime, maybe I'll bundle a version of latexmk with arxiv-collector or something....

@djsutherland
Copy link
Owner

The new 0.3.0 throws an error if the latexmk version is known broken, and has a utility to download a working version of latexmk. You can use it in your travis builds like this:

script:
  - cd test
  - arxiv-collector --get-latexmk latexmk
  - arxiv-collector --latexmk ./latexmk test.tex

@physikerwelt
Copy link
Author

Thank you for your fix. However, Travis still refused to compile.

https://travis-ci.org/physikerwelt/arxiv-collector/builds/537643156#L2064

Did I miss something in physikerwelt@db89c67 ? Or are there additional dependencies?

https://github.com/physikerwelt/arxiv-collector/blob/db89c6726a5c57af924ba3751039bb45ac67370a/.travis.yml#L15-L19

@djsutherland
Copy link
Owner

djsutherland commented May 27, 2019

It was a python 2 error, fixed in 0.3.2 (and then another python 2 error, fixed in 0.3.3); try again?

You can also remove the plain latexmk line; it's not needed, though I don't think it'll break anything.

@physikerwelt
Copy link
Author

physikerwelt commented Jun 3, 2019

I have the feeling that there are more python 2 errors. At least it still fails on Travis with python 2.7.

works on Python 3, though- That's fine with me.
See
https://travis-ci.org/physikerwelt/arxiv-collector/builds/540865849
for the details.
Maybe it would be a good idea to drop python 2 support officially?

@djsutherland
Copy link
Owner

Turned out to be because I didn't read properly or, for some reason, test, the "fix" in 0.3.3. This version actually works. Thanks again!

I should definitely set up CI here to avoid that happening in the future, but I think enough people use python 2 by default still that I should support it while it's relatively easy.

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

No branches or pull requests

2 participants