Skip to content

Travis builds #32

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
Merged
merged 23 commits into from
Jan 29, 2017
Merged

Travis builds #32

merged 23 commits into from
Jan 29, 2017

Conversation

henryiii
Copy link
Member

This PR addresses three of the outstanding issues for GooFit 2.0.0, and includes a few bugfixes for the new CMake build. The main features are:

Several of the issues still need a few more features to be complete, but this PR should be ready.

henryiii and others added 23 commits January 23, 2017 15:49
@henryiii
Relax min requirement for ROOT
75b5b56
@henryiii
Adding a small dependency fix
9bc0201
@henryiii
Adding Travis build
5c9339e
@henryiii
Updates to docs
7e8be8c
@henryiii
Preparing to auto-download datafiles
7cc9f3c
Auto download of datafiles
1ef2574
@maddocbf
Fix for in source build in subfile
8918a49
@henryiii
removing unpacked dir from git status
ac76d4e
@henryiii
Adding better download of thrust
95fb7db
@henryiii
Adding a basic test
5de3529
@henryiii
Moving to GOO varaibles to allow cmake shortcut to work
8b0572e
@henryiii
Fixes for tests on Travis
d24b996
@henryiii
Fix permissions
de357e4
@henryiii
Fix for travis
e2e6141
@henryiii
Fix typo in travis
1d9d010
@henryiii
Build status for main branches
2b7efb7
@henryiii
Remove some kludges no longer needed due to download order
6e1f4ce
@henryiii
Removing extra endif
2a325e8
@henryiii
Adding changelog up to 1.0.0
3fe5ce6
@henryiii
Adding 2.0 details to changelog
a8b97bf
@henryiii
Adding PR to changelog
3b7896a
@hassec
Improve Travis badges
2583ee4 
Badges now link to the Travis webpage.
@hassec
fixed warnings in the pipipi0 example
b30fde8
hassec
hassec approved these changes Jan 29, 2017
View reviewed changes
Copy link
Member

@hassec hassec left a comment

Choose a reason for hiding this comment

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

Just checked out this build and ran some tests.

Noticed two warnings in one of the examples. Not sure why they hadn't surfaced in earlier versions.. But they are fixed now.

Think we can go ahead and merge this.

@henryiii henryiii merged commit 97e131d into master Jan 29, 2017
@henryiii henryiii modified the milestone: V 2.0 Jun 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hassec hassec hassec approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
V 2.0
Development

Successfully merging this pull request may close these issues.

Set up tests, add Travis CI Add a changelog to GooFit
3 participants
@henryiii @hassec @maddocbf