8000 Add automated testing by Travis CI by t-mat · Pull Request #2 · lz4/lz4 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
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 automated testing by Travis CI #2

Merged
merged 3 commits into from
Apr 16, 2014
Merged

Add automated testing by Travis CI #2

merged 3 commits into from
Apr 16, 2014

Conversation

t-mat
Copy link
Contributor
@t-mat t-mat commented Apr 16, 2014

Motivation

Recently, we've met some issues

I believe we can detect this kind of issue by automated testing.
As a first step, I'd like to introduce Travis-CI.

How to enable Travis-CI

t-mat added 3 commits April 16, 2014 22:25
.travis.yml in this commit means

 (1) Language is C.
 (2) Compiler is 'gcc'
 (3) Before testing, Travis-CI VM install 'gcc-multilib' for 32bit executables.
 (4) Travis-CI invokes 'make test'.

See also :

  Getting started
  http://docs.travis-ci.com/user/getting-started/

  Configuring your build
  http://docs.travis-ci.com/user/build-configuration/

  Building a C Project
  http://docs.travis-ci.com/user/build-configuration/
Add command-line switch '--no-prompt' to fullbench and fuzzer.
'--no-prompt' enables :

 - Throtting stdout to satisfy Travis-CI's stdout limit (10MB).
 - Prevent Travis-CI's build timeouts and ignore pause by getchar().
   http://docs.travis-ci.com/user/build-configuration/#Build-Timeouts
@Cyan4973
< 8000 div class="timeline-comment-header clearfix d-flex" data-morpheus-enabled="false">
Copy link
Member

I believe it's an excellent proposition, with very good procedure description.
I'll certainly look into it.

Cyan4973 added a commit that referenced this pull request Apr 16, 2014
Add automated testing by Travis CI
@Cyan4973 Cyan4973 merged commit 5f4a8e4 into lz4:beta Apr 16, 2014
@Cyan4973
Copy link
Member

Does it work with branches ? Or is it restricted to be used only with the 'master' branch ?

@Cyan4973
Copy link
Member

OK, it seems to work with branches :
https://travis-ci.org/Cyan4973/lz4

Fuzzer tests are really long on VM though...

@t-mat
Copy link
Contributor Author
t-mat commented Apr 17, 2014

Thanks !

Does it work with branches ?

Yes, it will build any branch, even pull requests.

Fuzzer tests are really long on VM though...

Yeah. I'm not sure yet, but using Build Matrix seems fit for this situation (testing 32bit and 64bit simultaneously). I'll check it later.

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
0