-
Notifications
You must be signed in to change notification settings - Fork 21
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #5 from robertodr/master
Ameliorate .travis.yml for a larger selection of compilers. Inspired by @daniel-j-h
- Loading branch information
Showing
21 changed files
with
315 additions
and
283 deletions.
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
7fa10e7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great to see this getting used! You removed your sanitizer code, though. What is your experience with it? I tried getting sanitizers to work on Travis CI but ran into a few issues, mainly the packages being broken.
7fa10e7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! Thanks again for the great work! The scripts for the sanitizers were actually dead wood. They were copy-pasted from another project and never used into PCMSolver.