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

Linux Build Failure #23013

Closed
Treah opened this issue Feb 22, 2018 · 5 comments

Comments

Projects
None yet
5 participants
@Treah
Copy link
Contributor

commented Feb 22, 2018

Game version:
Current Git pull commit: 825d0fe
Operating system: Linux, Jenkins reports only windows builds are succeeding.

Tiles or curses:
Both

Build log from Jenkins.

10:07:33 src/overmap.cpp: In member function ‘bool overmap::generate_sub(int)’:
10:07:33 src/overmap.cpp:1869:5: error: no return statement in function returning non-void [-Werror=return-type]
10:07:33 }
10:07:33 ^
10:07:33 src/overmap.cpp: At global scope:
10:07:33 src/overmap.cpp:1870:5: error: expected unqualified-id before ‘for’
10:07:33 for (auto &i : ant_points) {
10:07:33 ^
10:07:33 src/overmap.cpp:1874:5: error: expected unqualified-id before ‘for’
10:07:33 for (auto &i : cities) {
10:07:33 ^
10:07:33 cc1plus: all warnings being treated as errors
10:07:33 Makefile:732: recipe for target 'obj/overmap.o' failed
10:07:33 make: *** [obj/overmap.o] Error 1
10:07:33 make: *** Waiting for unfinished jobs....
10:07:40 Build step 'Conditional step (single)' marked build as failure
10:07:40 SSH: Current build result is [FAILURE], not going to run.
10:07:40 Recording fingerprints
10:07:40 IRC notifier plugin: Sending notification to: #CDDA-dev
10:07:40 Finished: FAILURE

@Treah

This comment has been minimized.

Copy link
Contributor Author

commented Feb 22, 2018

Reverting to commit 901c5dd fixes the compile issue. There was quite a few commits after this in regard to overmap code changes so I suspect the problem is in there.

@Leland

This comment has been minimized.

Copy link
Contributor

commented Feb 22, 2018

Those overmap changes would be #22864. @kevingranade is doing some refactoring of it in #23012, try checking out f6c39ec and see if it builds

@Treah

This comment has been minimized.

Copy link
Contributor Author

commented Feb 22, 2018

@Leland Ahh ok i was a bit confused by your link to a commit there since its not merged with master so Ill just wait and see when it is if it resolved the compile problem.

@boydkr

This comment has been minimized.

Copy link
Contributor

commented Feb 22, 2018

Should be fixed by #23015

@kevingranade

This comment has been minimized.

Copy link
Member

commented Feb 22, 2018

just waiting for auto-builds to complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.