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 grapite project to CI service #20

Closed
tysmith opened this issue Mar 1, 2018 · 2 comments
Closed

Add grapite project to CI service #20

tysmith opened this issue Mar 1, 2018 · 2 comments

Comments

@tysmith
Copy link

tysmith commented Mar 1, 2018

Adding graphite to a CI can help prevent regression and is fairly easy to setup. It can automatically perform different builds and runs unit tests on each commit.

There are many good free options available:

@mhosken
Copy link
Contributor

mhosken commented Mar 2, 2018

We are already doing CI Building (as we have since 2011) and we do therefore catch regressions. In the case of mingw, we don't officially support that and I doubt any of these services would provide a mingw based build out of the box.

@tysmith
Copy link
Author

tysmith commented Mar 2, 2018

That's great! I see @tim-eves just added the badges.

In the case of mingw setting it up on TravisCI (the system I know best) should be trivial. You just apt-get what is needed in setup and list the compilers and commands to run.

Another reason I brought this up is because my build broke when the mingw fix went in. My build script looks basically like this (run from a subdir):

cmake -G "Unix Makefiles" -DCMAKE_SKIP_RPATH:BOOL=YES -DGRAPHITE2_NTRACING:BOOL=YES -DCMAKE_CXX_COMPILER:FILEPATH=clang++ -DCMAKE_C_COMPILER:FILEPATH=clang -DCMAKE_BUILD_TYPE:STRING=Debug -DBUILD_SHARED_LIBS:BOOL=OFF  -DCMAKE_CXX_FLAGS="-frtti" -DCMAKE_EXE_LINKER_FLAGS:STRING="-lstdc++" ..
make -j8

However it was fixed by commit 0218a47.

I'd happy to have this closed unless there are other plans that I am not aware of.

@tim-eves tim-eves closed this as completed Mar 5, 2018
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

No branches or pull requests

3 participants