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

check if travis cache picked up LLVM patches from master #24953

Merged
merged 1 commit into from
Dec 7, 2017

Conversation

vchuravy
Copy link
Member

@vchuravy vchuravy commented Dec 6, 2017

This prevents cache poisoning from master which lead to #24855

Fixes #24855

@ararslan ararslan added building Build system, or building Julia or its dependencies ci Continuous integration labels Dec 6, 2017
@ararslan ararslan added this to the 0.6.x milestone Dec 6, 2017
.travis.yml Outdated
@@ -109,6 +109,9 @@ script:
- contrib/download_cmake.sh
- make -C moreutils mispipe
- make $BUILDOPTS -C base version_git.jl.phony
# Check that the llvm cache is not poisned
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"poisned" typo

@ararslan
Copy link
Member

ararslan commented Dec 6, 2017

Nice to see Travis Linux green on this branch again. Thanks Valentin!

@ararslan
Copy link
Member

ararslan commented Dec 6, 2017

Oh, hmm, well the tests seem to be passing on x86-64, but there are some accuracy issues on i686 causing the tests to fail there. Unrelated to this PR, but it'll be the next thing to address once this is merged.

@ararslan
Copy link
Member

ararslan commented Dec 7, 2017

32-bit Linux failures reported in #24959.

@vchuravy
Copy link
Member Author

vchuravy commented Dec 7, 2017

I will fix the typo and the merge this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
building Build system, or building Julia or its dependencies ci Continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants