-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/binaries publishing #2
base: feature/macosdistribution
Are you sure you want to change the base?
Conversation
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.
Does this PR going to be pushed upstream at some point? If not, why do we have so many configurations that are going to be built?
.travis.yml
Outdated
env: PATH="/usr/bin:$PATH" TEST=analyze | ||
# - os: linux | ||
# compiler: gcc | ||
# env: CUDA=8.0.61-1 |
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.
commented out stuff can be removed
.travis.yml
Outdated
dist: trusty | ||
env: FEATURES="--enable-memcached" | ||
- os: linux | ||
compiler: clang |
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.
what does that mean? Does it compile with clang, or for clang? Does ccache have to be compiled for certain compiler?
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.
Documentation of travis says that are tests suits agains clang/gcc: https://docs.travis-ci.com/user/languages/c/
I removed other linux entries - Left only those added by Martin.
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.
It's still not entirely clear to me, but I can figure it out later not to block this PR.
These are default settings from upstream so we should keep them to ensure nothing is broken due to our changes
Von meinem iPhone gesendet
… Am 02.08.2018 um 14:56 schrieb Anna Nowak ***@***.***>:
@wiorka approved this pull request.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or mute the thread.
|
No description provided.