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

Compiling a Debug-Version of Rev7939 #21

Closed
timfel opened this issue Nov 19, 2015 · 5 comments
Closed

Compiling a Debug-Version of Rev7939 #21

timfel opened this issue Nov 19, 2015 · 5 comments

Comments

@timfel
Copy link
Member

timfel commented Nov 19, 2015

On Ubuntu (maverick 64bit) I have to modify the source of the latest trunk-Revision (7939). Otherwise the debug-version (./configure --enable-debug) stops compiling..


bzr diff
=== modified file 'src/tolua/tolua.cpp'
--- src/tolua/tolua.cpp 2011-02-26 20:35:26 +0000
+++ src/tolua/tolua.cpp 2011-03-06 20:12:38 +0000
@@ -20440,6 +20440,6 @@
#if defined(LUA_VERSION_NUM) && LUA_VERSION_NUM >= 501
TOLUA_API int luaopen_stratagus (lua_State* tolua_S) {
return tolua_stratagus_open(tolua_S);
-};
+}
#endif

=== modified file 'src/video/font.cpp'
--- src/video/font.cpp 2011-02-27 10:49:24 +0000
+++ src/video/font.cpp 2011-03-06 20:13:47 +0000
@@ -475,7 +475,7 @@
return w + 1;
}

-CGraphic *CFont::GetFontColorGraphic(const CFontColor *fontColor) const
+CGraphic *CFont::GetFontColorGraphic(const CFontColor *) const
{
if (!UseOpenGL) {

return this->G;


Imported from Launchpad using lp2gh.

@timfel
Copy link
Member Author

timfel commented Nov 19, 2015

(by pali)
I see that problem appered in this commit:
http://bazaar.launchpad.net/~stratagus/stratagus/trunk/revision/7937

Can you chack if before this commit is all ok?

@timfel
Copy link
Member Author

timfel commented Nov 19, 2015

(by rheld)
Thats right... Revision 7936 compiles fine.

@timfel
Copy link
Member Author

timfel commented Nov 19, 2015

(by pali)
Joris can you look at your commit?

@timfel
Copy link
Member Author

timfel commented Nov 19, 2015

(by joris-dauphin)

  • I make a typo in GetFontColorGraphic (use global FontColor instead of arg fontColor)
    For tolua, the file is generated (from *.pkg) and the modification in pkg file seem good (for this commit)
    I will try to fix it the correct way.

@timfel
Copy link
Member Author

timfel commented Nov 19, 2015

(by pali)
I see that in bzr problem with FontColor was fixed.

For tolua.cpp: Stratagus now using CMake build system and tolua.cpp is generated automatically by cmake script. Now generated file depends on your version of tolua++ (it is not included in Stratagaus tree).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant