Opened 7 years ago

Closed 7 weeks ago

Last modified 6 weeks ago

#4808 closed enhancement (fixed)

Game does not compile with Clang 6

Reported by: gentz Owned by:
Priority: Should Have Milestone: Alpha 26
Component: Build & Packages Keywords:
Cc: leper, Itms, gentz Patch:

Description (last modified by gentz)

Spidermonkey currently fails to link with Clang 6, however everything else with minimum tweaking does compile.

Comment #2 has my attempt to upgrade to the spidermonkey from the gecko-dev repository which I have given up on.

Comment #4 has a small patch which compiles everything but spidermonkey with Clang 6

Change History (9)

comment:1 by gentz, 7 years ago

Changes so far: https://code.wildfiregames.com/P86

I do plan to separate it into multiple patches... trust me that will happen one day.

Note: I'm pretty sure it brakes building spidermonkey on non-linux OSes, I'll also fix that one day.

comment:3 by Stan, 7 years ago

Cc: leper Itms added

comment:4 by gentz, 7 years ago

I can't be bothered to figure out how proxies work and that sorta puts a stop to upgrading 0ad to use the latest spidermonkey.

Instead I've made it compile spidermonkey 38 with GCC and the rest with clang. See my finished diff: https://code.wildfiregames.com/D953

comment:5 by gentz, 7 years ago

Cc: gentz added
Description: modified (diff)

comment:6 by gentz, 7 years ago

Milestone: Work In ProgressBacklog

comment:7 by phosit, 7 weeks ago

Do we still suport Clang 6?

comment:8 by Ralph Sennhauser, 7 weeks ago

Resolution: fixed
Status: newclosed

From the age of this ticket we can infer this is about adding support for the latest clang version or at least a recent one.

The CI uses either version 7 or 8, so we can say 6 is no longer supported even if it worked (possibly using gcc' stl)

comment:9 by Stan, 6 weeks ago

Milestone: BacklogAlpha 26

Limit was raised for A26 as it triggered some bad assertion in debug mode, preventing the CI to build.

Refs: r27489, r27100

Note: See TracTickets for help on using tickets.