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

More travis-ci testing #642

Merged
merged 6 commits into from May 30, 2018
Merged

More travis-ci testing #642

merged 6 commits into from May 30, 2018

Conversation

isuruf
Copy link
Member

@isuruf isuruf commented May 20, 2018

No description provided.

@isuruf isuruf mentioned this pull request May 20, 2018
@franz
Copy link
Contributor

franz commented May 22, 2018

Thanks, i'll try it with my Travis account.

@isuruf
Copy link
Member Author

isuruf commented May 23, 2018

Here's a job in my account, https://travis-ci.org/isuruf/pocl/builds/381242588

@franz
Copy link
Contributor

franz commented May 23, 2018

OK, found a few issues (sort of).

  1. the Docker builds do not run the tests (they only call docker build not docker run) and they don't install pocl before running tests (which is my fault, missing ninja install in some Dockerfiles)

  2. Just FYI, the Conda builds are using Ubuntu 14.04 which has known issues with pocl - see system fubar or a real issue? test_issue_445 and other cl2.hpp fails with a system error on Kaveri/Ubuntu 14.04 #590. This problem pops up somewhat randomly, and seems to be a bug in GCC - https://bugs.launchpad.net/ubuntu/+source/gcc-defaults/+bug/1228201 ...so you may or may not run into this.

@franz
Copy link
Contributor

franz commented May 23, 2018

OK, looks good. Running in Travis now.

@franz
Copy link
Contributor

franz commented May 23, 2018

https://travis-ci.org/franz/pocl/builds/382514677 one of the Docker jobs seems to be stuck in apt-get for some reason, but i'm guessing it's a build environment problem, not a pocl problem. But the other Docker job runs the tests now, so LGTM.

@isuruf
Copy link
Member Author

isuruf commented May 25, 2018

Looks like that issue was resolved, but there are test failures in ppc64le. Shall I mark the job as "allowed failure" ?

@franz
Copy link
Contributor

franz commented May 25, 2018

@isuruf well, how much do you care about ppc64 ? the two tests failing are not really suprising, since the whole passing-struct-as-kernel-args is not well defined in OpenCL. We can if-not-ppc them in CMake.

This reverts commit 16edb22.
@isuruf
Copy link
Member Author

isuruf commented May 28, 2018

how much do you care about ppc64 ?

I don't. I reverted that commit.

@pjaaskel pjaaskel merged commit 5272c7d into pocl:master May 30, 2018
@isuruf isuruf deleted the travis2 branch May 30, 2018 05:11
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