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 Compile #2298

Merged
merged 1 commit into from
Feb 2, 2019
Merged

Travis Compile #2298

merged 1 commit into from
Feb 2, 2019

Conversation

jmdejong
Copy link
Contributor

This is essentially the same as #2289 but with a cleaner git history.

It follows the compilation steps from http://wiki.torque3d.org/coder:compiling-in-linux#toc11

This checks both gcc and clang on linux, but only clang on mac

Because there are so many debug warnings they are suppressed in a very hacky way:
Make is run first with error output ignored.
Then an incremental build is run which has the error output, but will only build the files that errored the first time (and will error again).
There is probably a better way to ignore the warnings in cmake, but I'm a bit hesitant to change the cmake files and I couldn't find a way to do it with command line arguments.

Only after writing this I found out that there is a CI pull request already: #2211
That older PR probably has more features (including running a windows build on appveyor, and I believe it runs the tests too) but this script is simpler.

This checks both gcc and clang on linux, but only clang on mac

Becuse there are so many debug warnings they are supressed in a very hacky way:
Make is run first with error output ignored.
Then an incremental build is run which has the error output, but will only build the files that errored the first time (and will error again).
If anyone knows a better way to ignore the warnings that would be great.
@jmdejong jmdejong mentioned this pull request Jan 10, 2019
@Areloch
Copy link
Contributor

Areloch commented Feb 2, 2019

We'll go ahead and roll this in for now(as mentioned, it's simpler) and can look to bringing over aspects of #2211 next to fully flesh out our CI setup.

@Areloch Areloch merged commit e38eb34 into GarageGames:development Feb 2, 2019
Areloch added a commit to Areloch/GG-Torque3D that referenced this pull request Jun 7, 2020
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