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

Introduce meta.yml file, in place of description file. #66

Merged
merged 3 commits into from
Feb 3, 2019

Conversation

Zimmi48
Copy link
Member

@Zimmi48 Zimmi48 commented Feb 2, 2019

Generate various files automatically.
Use standard solution for CI.
We use Nix for stable versions and opam for the development version
(it is harder to use Nix to test compatibility with moving dependencies).

Generate various files automatically.
Use standard solution for CI.
We use Nix for stable versions and opam for the development version
(it is harder to use Nix to test compatibility with moving dependencies).
@Zimmi48 Zimmi48 force-pushed the meta-file branch 2 times, most recently from 20009b0 to 1a34231 Compare February 3, 2019 12:09
@Zimmi48
Copy link
Member Author

Zimmi48 commented Feb 3, 2019

I don't understand the failure in the opam build: https://travis-ci.com/coq-community/corn/jobs/174931943#L2217-L2234
@erikmd Sorry to bother you with this: do you have an idea what's going on?

@erikmd
Copy link
Member

erikmd commented Feb 3, 2019

Hi @Zimmi48, thanks for your message!
At first sight it seems that the opam install command lasts more than 10' (20' with travis_retry) without outputting anything, so Travis CI stops the job.
Could you try with opam install -y -v -j ${NJOBS} . instead of opam install -y -j ${NJOBS} .? (without travis_retry because it would be unnecessary if each individual file compiles in less than 10')

@erikmd
Copy link
Member

erikmd commented Feb 3, 2019

And if ever this wouldn't work because a given file would take 15' or 25' to compile with no warning nor any other message, one could always add a Check true. or so in the middle of that file…:)

@Zimmi48
Copy link
Member Author

Zimmi48 commented Feb 3, 2019

OK I've replaced the travis_wait with a verbose build. Let's hope this works better.

@Zimmi48
Copy link
Member Author

Zimmi48 commented Feb 3, 2019

Thanks @erikmd! Your suggestion worked.

@spitters spitters merged commit 6bcdf24 into coq-community:master Feb 3, 2019
@Zimmi48 Zimmi48 deleted the meta-file branch July 13, 2019 12:21
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 this pull request may close these issues.

None yet

3 participants