-
Notifications
You must be signed in to change notification settings - Fork 3
[conan] Add header-only style conan support #9
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
base: master
Are you sure you want to change the base?
Conversation
|
I just so copied the |
- remove uneccesary GitHub settings - remove AppVeyor for now since MSVC cannot build this - add copyright notice where needed - adjust author
| env: CONAN_CLANG_VERSIONS=5.0 CONAN_DOCKER_IMAGE=conanio/clang50 | ||
|
|
||
| install: | ||
| - chmod +x .ci/install.sh |
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.
I'm still learning conan and travis, but it seems like it would be simpler (and less fraught with license complications) to do this without the shell scripts. Something like:
language: cpp
os: linux
dist: trusty
sudo: required
addons:
apt:
sources:
- ubuntu-toolchain-r-test
packages:
- g++-8
env: CC=gcc-8 CXX=g++-8
before_install: pip install conan
install: conan user
script: conan create . user/channelBut I'm probably missing something.
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.
In theory this could be done. However the xenial and trusty images comes with libstdc5 (or earlier) and it is cumbersome and error prone to upgrade to any more recent version (which is needed for C++17). Using a pre-backed docker with all the tools makes life much easier.
|
I think having Travis set up this way doesn't quite work, because it tries to fetch the latest release archive rather than using the current code, which seems to defeat the purpose of CI. See the travis build log here: https://travis-ci.org/cbbowen/trytravis-scratch/builds/481678825. |
|
Yup this seems wrong, I will investigate why it does not |
Add's
conanfile.pyvia conan template