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.

Sign up for GitHub

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

Jump to bottom

Added a travis.yml to run test builds #23

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Added a travis.yml to run test builds #23

wants to merge 21 commits into from

Conversation

petr-tik
Copy link
Member

Closes #7

@petr-tik petr-tik self-assigned this Apr 24, 2018
jbcoe
jbcoe requested changes Apr 25, 2018
View reviewed changes
Copy link
Collaborator

@jbcoe jbcoe left a comment

Choose a reason for hiding this comment

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

Looks like you are missing a trailing new line.

Otherwise LGTM

petr-tik added 19 commits April 25, 2018 14:10
@petr-tik
Update .travis.yml
861d97a
@petr-tik
bash script prepares a travis box for testing
a710388
@petr-tik
Remove docker and run unit tests on travis box
9d32a59 
Moves ffig upstream dependency from dockerfile to build.sh
@petr-tik
Update .travis.yml
ff401ae
@petr-tik
Remove pip2
12c60a5 
fix dir paths for ffig main dependency
@petr-tik
Request only py3.6
cf17ef8
@petr-tik
Changing the build script and giving it right to execute
c930567
@petr-tik
Move travis install step into a separate shell script
2a94bf7
@petr-tik
Don't need to change to prev directory - we never leave rest-api dir
f94b225
@petr-tik
Add both ffig folders to pythonpath
00cf693
@petr-tik
Using the wrong folder for ffig
0c883c8
@petr-tik
Set python path and grep env for it
425073d
@petr-tik
Remove dead ls and add another init
6620e29
@petr-tik
Reduce the import search space
c81355b
@petr-tik
Investigating sys.path on travis
308a562
@petr-tik
Trying to insert into sys path, since pythonpath doesn't work
883c2b3
@petr-tik
find and set ld_lib_path with libclang
1770b7a
@petr-tik
Explicitly install llvm-5.0 to fix libclang error
5238ce4
@petr-tik
try version-less llvm-config
2b54dce
@petr-tik
Copy link
Member Author

sorry about the stream of notificaitons - struggling to install and locate all of ffig's dependencies. Importing it was bad enough (current try-except + sys.path solution - v hacky), now trying to find libclang on travis machine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jbcoe jbcoe jbcoe requested changes

Requested changes must be addressed to merge this pull request.

Assignees

@petr-tik petr-tik

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@petr-tik @jbcoe