Can't compile since 5159

[Iain McFarlane]

[ 10%] Building CXX object src/CMakeFiles/widelands_main.dir/wlapplication.cc.o
/home/iain/src/widelands/src/wlapplication.cc: In function ‘void terminate(int)’:
/home/iain/src/widelands/src/wlapplication.cc:951: error: ‘SIGKILL’ was not declared in this scope
/home/iain/src/widelands/src/wlapplication.cc:951: error: ‘raise’ was not declared in this scope
/home/iain/src/widelands/src/wlapplication.cc: In member function ‘void WLApplication::shutdown_hardware()’:
/home/iain/src/widelands/src/wlapplication.cc:975: error: ‘SIGALRM’ was not declared in this scope
/home/iain/src/widelands/src/wlapplication.cc:975: error: ‘signal’ was not declared in this scope
make[2]: *** [src/CMakeFiles/widelands_main.dir/wlapplication.cc.o] Error 1

[Iain McFarlane (iain-imcfarla)]

Sorry should have added this to the initial report

Trying to compile on Ubuntu Lucid

[Victor Pelt (victor-pelt)]

what glibc and gcc version are you using?

[Victor Pelt (victor-pelt)]

could you add "#include <signal.h>" on top of wlapplication.cc and report if that fixes it. since i'm not having this issue i also would still like to know which glibc and gcc version you are using

[Iain McFarlane (iain-imcfarla)]

I hope this helps

libglib2.0-0 2.24.0-0ubuntu1
gcc 4:4.4.3-1ubuntu1
libc6 2.11.1-0ubuntu5

yes adding it to the top of wlapplication.cc does fix it

[SirVer (sirver)]

remember to add a #ifndef WIN32 around. I will test if things still compile on mac os x.

[Victor Pelt (victor-pelt)]

not convinced windows shouldn't pick it up if proper headers are included but just in case
fixed in rev 5184

[SirVer (sirver)]

windows has no POSIX signals imho.

[Jari Hautio (jarih)]

Just saw this bug so just to clear up windows side: Windows (MSVC) has signal.h header and but it does not define SIGKILL or SIGALRM signals. I already added WIN32 ifdefs to appropriate places in r5180.

[SirVer (sirver)]

Released in build-15