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

Travis #32

Merged
merged 2 commits into from
Jun 20, 2014
Merged

Travis #32

merged 2 commits into from
Jun 20, 2014

Conversation

theuni
Copy link
Contributor

@theuni theuni commented Jun 17, 2014

Fixes #29. Travis provides excellent integration with github. It will chime-in as necessary for pull-requests.

I've been working on pull-req stuff for bitcoin core, and figured this would be a good test-case to see how viable Travis is.

See an example build-matrix here. https://travis-ci.org/theuni/secp256k1/builds/27738000

To take the pull-req features for a test-run before pulling, you can submit a pull req to my travis branch here: https://github.com/theuni/secp256k1

To setup: head to https://travis-ci.org and give it the necessary api access to your repo. After that, any branch that contains a .travis.yml file will be built for each push/pull-req.

@theuni
Copy link
Contributor Author

theuni commented Jun 17, 2014

Here's an example of a pull-request with an active travis hook: travis-ci-examples/php-example#8

@sipa
Copy link
Contributor

sipa commented Jun 19, 2014

ACK.

Can you update the build instructions too, to include autogen.sh?

This adds a huge matrix of builds. The undesirable ones can be removed.
@theuni
Copy link
Contributor Author

theuni commented Jun 19, 2014

updated README

@sipa sipa merged commit b0cc21f into bitcoin-core:master Jun 20, 2014
sipa added a commit that referenced this pull request Jun 20, 2014
b0cc21f build: add travis support (Cory Fields)
d409087 build: add autogen. How was this missing? (Cory Fields)
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.

Pulltester for libsecp256k1?
3 participants