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

Add minimal travis support. #76

Merged
merged 3 commits into from
May 18, 2016
Merged

Add minimal travis support. #76

merged 3 commits into from
May 18, 2016

Conversation

mstemm
Copy link
Contributor

@mstemm mstemm commented May 17, 2016

Add minimal travis.yml file that builds and packages falco. No actual
tests yet.

@mstemm
Copy link
Contributor Author

mstemm commented May 17, 2016

@henridf

Add minimal travis.yml file that builds and packages falco. No actual
tests yet.
mstemm added 2 commits May 17, 2016 16:19
Showing both dev/master branches for now.
Add a basic test that loads the kernel module from the source directory
and runs falco. No testing of behavior yet.
- set -e
- export CC="gcc-4.8"
- export CXX="g++-4.8"
- wget https://s3.amazonaws.com/download.draios.com/dependencies/cmake-3.3.2.tar.gz
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just curious why take cmake here and not from standard package repos?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was what sysdig was doing. Part of the reason might be that the travis distro has an older version of CMake available. I remembered that on centos 7 I had to install a newer version.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

arright

@henridf
Copy link
Contributor

henridf commented May 18, 2016

👍

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