Compilation error (SVN r12537)

Compilation error (SVN r12537)

Postby WillemS » 02 Mar 2013, 14:33

I just updated from the SVN repository to revision 12537, and now compilation fails for me (Linux Kubuntu, 32-bit) with the following output:
{l Code}: {l Select All Code}
[ 61%] Building CXX object CMakeFiles/supertuxkart.dir/src/karts/controller/player_controller.cpp.o
/home/willem/SuperTuxKart/stk/src/karts/controller/player_controller.cpp: In memberfunctie ‘virtual void PlayerController::update(float)’:
/home/willem/SuperTuxKart/stk/src/karts/controller/player_controller.cpp:313:9: fout: ‘Log’ has not been declared
make[2]: *** [CMakeFiles/supertuxkart.dir/src/karts/controller/player_controller.cpp.o] Fout 1
make[1]: *** [CMakeFiles/supertuxkart.dir/all] Fout 2
make: *** [all] Fout 2

(the complete compilation output is attached).

It seems that the problematic line (player_controller.cpp:313) is added in revision 12531:
{l Code}: {l Select All Code}
        Log::debug("irr_driver", "-------------------------------------\n");


Is this just my problem, or do others experience the same?
Attachments
stk-compilatie-fout.txt
The make output
(11.44 KiB) Downloaded 257 times
WillemS
 
Posts: 81
Joined: 03 Nov 2012, 17:42

Re: Compilation error (SVN r12537)

Postby Funto » 02 Mar 2013, 18:42

Seems you have to add #include "utils/log.hpp" to player_controller.cpp.
I can't do it now as I can't test it.

BTW, hiker, on that logging functionality: couldn't you make a macro that used __FILE__ / __LINE__ instead of having to always write where the message comes from?
And I still totally dislike the error-prone printf-style formatting ^^.
Funto
 
Posts: 459
Joined: 09 Dec 2009, 13:47
Location: Bordeaux, France

Re: Compilation error (SVN r12537)

Postby WillemS » 02 Mar 2013, 20:31

Funto {l Wrote}:Seems you have to add #include "utils/log.hpp" to player_controller.cpp.


That fixes it indeed :) thank you!
WillemS
 
Posts: 81
Joined: 03 Nov 2012, 17:42

Re: Compilation error (SVN r12537)

Postby hiker » 03 Mar 2013, 13:18

WillemS {l Wrote}:That fixes it indeed :) thank you!

I fixed that in r12538 - thanks for the report. Strangely enough r12537 compiled fine for me on windows and linux.

Cheers,
Joerg
hiker
 
Posts: 1435
Joined: 07 Dec 2009, 12:15
Location: Melbourne, Australia

Who is online

Users browsing this forum: No registered users and 1 guest