Closed tschoonj closed 6 years ago
Excellent job, Tom!
2017-11-21 13:32 GMT+01:00 Tom Schoonjans notifications@github.com:
Hi @golosio https://github.com/golosio,
I have added the necessary configuration files for building XRMC on Travis-CI (Linux and macOS) and AppVeyor (Windows) This will allow for testing code that is being added to pull requests.
On Linux it will test using gcc, both with and without using XMI-MSIM support. On macOS it will build once with clang and once with gcc. Building with clang actually allowed me to find a bug! On Windows it will build with gcc, both in 64-bit mode and 32-bit mode.
If you would look to see the latest build results:
https://travis-ci.org/tschoonj/xrmc/builds/305220875 https://ci.appveyor.com/project/tschoonj/xrmc
In order for this work for new pull requests, you will need to do some things on the Travis-CI and AppVeyor websites though (I cannot do this for you unfortunately). For Travis-CI: do steps 1. and 2. as explained at https://docs.travis-ci.com/user/getting-started/ For AppVeyor: do steps 1. and 2. as explained at https://www.appveyor.com/docs/
Make sure you always use Github authentication, there is no need to create new accounts.
Let me know if you need any help!
Best,
Tom
You can view, comment on, or merge this pull request online at:
https://github.com/golosio/xrmc/pull/34 Commit Summary
- Travis-CI: add Linux support
- Travis-CI: add macOS support
- Fix illegal instruction segfault with clang
- Examples: set HeaderFlag always to 1
- AppVeyor: add support
- Travis-CI: add Linux build with XMI-MSIM support
- Travis-CI: add macOS build with GCC7
- CI: build master only
File Changes
- A .travis.yml https://github.com/golosio/xrmc/pull/34/files#diff-0 (69)
- A appveyor.yml https://github.com/golosio/xrmc/pull/34/files#diff-1 (30)
- M examples/anisotropicsource/detector.dat https://github.com/golosio/xrmc/pull/34/files#diff-2 (2)
- M examples/beamsource/detector.dat https://github.com/golosio/xrmc/pull/34/files#diff-3 (2)
- M examples/cylind_shell/detector.dat https://github.com/golosio/xrmc/pull/34/files#diff-4 (2)
- M examples/fluor_layers/detector_0deg.dat https://github.com/golosio/xrmc/pull/34/files#diff-5 (2)
- M examples/materials/detector.dat https://github.com/golosio/xrmc/pull/34/files#diff-6 (2)
- M examples/phasecontrast/phcdetector.dat https://github.com/golosio/xrmc/pull/34/files#diff-7 (2)
- M examples/quadrics/detector.dat https://github.com/golosio/xrmc/pull/34/files#diff-8 (2)
- M examples/star1/detector.dat https://github.com/golosio/xrmc/pull/34/files#diff-9 (2)
- M examples/star2/detector.dat https://github.com/golosio/xrmc/pull/34/files#diff-10 (2)
- M examples/wheel/detector.dat https://github.com/golosio/xrmc/pull/34/files#diff-11 (2)
- M src/composition/composition.cpp https://github.com/golosio/xrmc/pull/34/files#diff-12 (6)
- M src/geom3d/geom3d.cpp https://github.com/golosio/xrmc/pull/34/files#diff-13 (6)
- M src/geom3d/xrmc_geom3d.h https://github.com/golosio/xrmc/pull/34/files#diff-14 (2)
Patch Links:
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/golosio/xrmc/pull/34, or mute the thread https://github.com/notifications/unsubscribe-auth/ADrHOZ6gxk7XHSNyUb4vCk4Z7LNptCIVks5s4sLGgaJpZM4Qlw-b .
-- Bruno Golosio, PhD Associate Professor of Applied Physics Department of Physics University of Cagliari (Italy)
Thanks!
Please let me know when you have activated the repository on the Travis-CI and AppVeyor websites: I will then merge this branch which should result in the first builds...
Best,
Tom
Bruno, could you please look into activating Travis-CI and AppVeyor? Shouldn't take more than five minutes of your time...
Thanks!
Hi Tom, I tried to activate them, let me know if it's ok or I still have to do something.
2017-11-23 15:47 GMT+01:00 Tom Schoonjans notifications@github.com:
Bruno, could you please look into activating Travis-CI and AppVeyor? Shouldn't take more than five minutes of your time...
Thanks!
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/golosio/xrmc/pull/34#issuecomment-346636792, or mute the thread https://github.com/notifications/unsubscribe-auth/ADrHOWdMTleE7KGPnHloRiYRm8dS1ZP8ks5s5YWTgaJpZM4Qlw-b .
-- Bruno Golosio, PhD Associate Professor of Applied Physics Department of Physics University of Cagliari (Italy)
Thanks Bruno!
Travis-CI looks good. AppVeyor I cannot tell as I don't know what the path to your account is there. Now that I merged it in, a build should have started there too though, which you should be able to confirm.
Hi @golosio,
I have added the necessary configuration files for building XRMC on Travis-CI (Linux and macOS) and AppVeyor (Windows) This will allow for testing code that is being added to pull requests.
On Linux it will test using gcc, both with and without using XMI-MSIM support. On macOS it will build once with clang and once with gcc. Building with clang actually allowed me to find a bug! On Windows it will build with gcc, both in 64-bit mode and 32-bit mode.
If you would look to see the latest build results:
https://travis-ci.org/tschoonj/xrmc/builds/305220875 https://ci.appveyor.com/project/tschoonj/xrmc
In order for this work for new pull requests, you will need to do some things on the Travis-CI and AppVeyor websites though (I cannot do this for you unfortunately). For Travis-CI: do steps 1. and 2. as explained at https://docs.travis-ci.com/user/getting-started/ For AppVeyor: do steps 1. and 2. as explained at https://www.appveyor.com/docs/
Make sure you always use Github authentication, there is no need to create new accounts.
Let me know if you need any help!
Best,
Tom