Skip to content

Reconfigure travis to perform parallel builds and linting #364

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

Merged
merged 2 commits into from
Dec 23, 2016

Conversation

tautschnig
Copy link
Collaborator

@tautschnig tautschnig commented Dec 18, 2016

The build matrix is constructed from os and compiler.
This should re-enable linting as a parallel, separate task.

@danpoe
Copy link
Contributor

danpoe commented Dec 19, 2016

Would it be possible to continue using g++ 4.9 plus the corresponding libstdc++? The versions in the suggested new build configuration unfortunately do not support some C++11 STL functions for example that can be useful.

@tautschnig
Copy link
Collaborator Author

It would be possible, but a bit more cumbersome, because Ubuntu introduced some bug in 4.9 that makes the corresponding libstdc++ incompatible with clang. The workaround is to make clang compile with -stdlib=libc++ as additional CXXFLAGS. Perfectly doable, but the travis config becomes more messy. Yet if there is a need for doing so then we can certainly go that road. @danpoe Would you say that's preferable?

@danpoe
Copy link
Contributor

danpoe commented Dec 19, 2016

Are you referring to the max_align_t bug? I encountered that one when writing the current travis file (but unfortunately don't remember anymore which change exactly made it go away). With the current travis file clang should work even with libstdc++.

@tautschnig
Copy link
Collaborator Author

Yes, the max_align_t issue. What I'd like to maintain is parallel runs of the various tasks. I'll try to cook up some best-of version to get us all these features.

@tautschnig tautschnig force-pushed the travis-reconfig branch 2 times, most recently from 517eaf0 to 61eff63 Compare December 21, 2016 21:35
@tautschnig tautschnig changed the title travis cleanup: no need for install target, build matrix Reconfigure travis to perform parallel builds and linting Dec 21, 2016
@tautschnig tautschnig force-pushed the travis-reconfig branch 4 times, most recently from 3d01951 to bfc81d0 Compare December 23, 2016 11:36
tautschnig and others added 2 commits December 23, 2016 15:25
The build matrix is constructed from os and compiler. OSX and Linux are
picked as operating systems for compilation, and linting is run as a
parallel task. Linux uses GCC/g++ 5 and Clang 3.7.
G++-5 warned: "suggest parentheses around operand of ‘!’ or change ‘&’
to ‘&&’ or ‘!’ to ‘~’"
@tautschnig
Copy link
Collaborator Author

I intend to merge the travis configuration fix and 0cf0972 to make miniBDD build using GCC 5. Thus we should be back at a stage were we can reasonably use travis.

@tautschnig tautschnig merged commit 8e473cb into diffblue:master Dec 23, 2016
@tautschnig tautschnig deleted the travis-reconfig branch December 23, 2016 14:46
smowton pushed a commit to smowton/cbmc that referenced this pull request May 9, 2018
Preparation to use of the  reachability slicer before the full-slicer.
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.

2 participants