Rubidium wrote:

Oh lovelyl... as expected it fails horribly in the 'real' world where it barely worked with the compile farm.

For some reason it isn't defining that it is a 64 bits compiler, yet it did compile the 64 bits test application as a 64 bits compiler.

What's extra 'nice' in this is that the headers are littered with those little LP64 checks to stop you from building against things like quicktime.

This comment was imported from FlySpray: https://bugs.openttd.org/task/3190#comment6632

Rubidium wrote:

For those who think that I am going to fix this: no I am not going. All efforts to get something somewhat working with the compile farm ended in something, which broke a compilation at lots of other people, so stuff was done to make their previously working systems compile OpenTTD again. This is just another case where OSX does not behave the same for everyone.

I do not have the equipment real, virtual or otherwise to give any proper support so the best I can do is not make it worse; I won't be touching it anymore including patches written by others as those have backfired more often than not.

This comment was imported from FlySpray: https://bugs.openttd.org/task/3190#comment6633

planetmaker wrote:

a query for whether STRGEN is compiled or not also helps and leaves the check otherwise unharmed

Attachments

• make_compile.diff (0.46 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/3190#comment6741

planetmaker wrote:

accidential repost.

Attachments

• make_compile.diff (0.46 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/3190#comment6742

planetmaker wrote:

One of the revisions up to r17717 changed src/video/cocoa/fullscreen.mm and requires some fixes related to the goto jumps in order to compile with gcc 4.2. The solution is to declare all variables prior to the first possible jump. The attached patch fixes that as well as the compile failure in strgen reported above.

[SRC] Compiling video/cocoa/fullscreen.mm
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm: In member function ‘bool FullscreenSubdriver::SetVideoMode(int, int)’:
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:346: error: jump to label ‘ERR_NOT_INDEXED’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:318: error: from here
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:337: error: crosses initialization of ‘NSPoint pt’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:333: error: crosses initialization of ‘NSRect screen_rect’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:349: error: jump to label ‘ERR_DOUBLEBUF’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:313: error: from here
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:337: error: crosses initialization of ‘NSPoint pt’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:333: error: crosses initialization of ‘NSRect screen_rect’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:351: error: jump to label ‘ERR_NO_SWITCH’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:300: error: from here
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:337: error: crosses initialization of ‘NSPoint pt’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:333: error: crosses initialization of ‘NSRect screen_rect’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:353: error: jump to label ‘ERR_NO_CAPTURE’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:294: error: from here
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:337: error: crosses initialization of ‘NSPoint pt’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:333: error: crosses initialization of ‘NSRect screen_rect’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:355: error: jump to label ‘ERR_NO_MATCH’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:277: error: from here
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:337: error: crosses initialization of ‘NSPoint pt’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:333: error: crosses initialization of ‘NSRect screen_rect’
/Users/ingo/ottd/trunk/src/video/cocoa/fullscreen.mm:289: error: crosses initialization of ‘int gamma_error’

Attachments

• compile_macosx_r17717.diff (2.27 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/3190#comment6789

michi_cc closed the ticket.

Reason for closing: Fixed

In r17767 and r17768.

This comment was imported from FlySpray: https://bugs.openttd.org/task/3190