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

Problem with Lua 5.2 and lua_strlen #80

Closed
Bertram25 opened this issue Dec 17, 2012 · 5 comments
Closed

Problem with Lua 5.2 and lua_strlen #80

Bertram25 opened this issue Dec 17, 2012 · 5 comments
Assignees
Labels

Comments

@Bertram25
Copy link
Member

VT won't compile against lua 5.2 because of the lack of definition of the lua_strlen member which was renamed lua_rawlen. Defines will have to put to restore compilation on system with lua 5.2 comiled without compatibility support.

See also:
http://comments.gmane.org/gmane.comp.programming.swig/18673

And this thanks to qubodup:
http://codepad.org/CVsPwvag

/home/qubodup/src/git/ValyriaTear/src/luabind/luabind/detail/policy.hpp:
In member function ‘std::string luabind::default_converter<std::basic_string<char> >::from(lua_State*, int)’:
/home/qubodup/src/git/ValyriaTear/src/luabind/luabind/detail/policy.hpp:748:71:
error: ‘lua_strlen’ was not declared in this scope
@shirishag75
Copy link

At my end it works flawlessly with lua 5.2 . Probably the packagers did something right .

~/games/valyriatear$ lua -v
Lua 5.2.1 Copyright (C) 1994-2012 Lua.org, PUC-Rio

@Bertram25
Copy link
Member Author

Indeed, I have no evidence so far, but I'm quite sure they compiled lua 5.2 on Debian using compatibility flags, something they maybe didn't do on arch linux.

Anyway, I'll have to add compatibility ifdefs.

@Bertram25
Copy link
Member Author

Interesting commit, that i will partially and iteratively port using also the reference repo:
https://github.com/teto/luabind/commit/d03ccd182a9b35d84c1d57d77f0c5cc799576545
reference one:
https://github.com/rpavlik/luabind

Bertram25 pushed a commit that referenced this issue Jan 4, 2013
…inst lua 5.2.

Let's hope it will help with issue #80.
@ghost ghost assigned Bertram25 Jan 5, 2013
@Bertram25
Copy link
Member Author

Should be ok with:
003956f

@Bertram25
Copy link
Member Author

Closing since I've got no reaction against the latest patch. Feel free to reopen it if needed.

Bertram25 pushed a commit that referenced this issue Jan 7, 2013
…inst lua 5.2.

Let's hope it will help with issue #80.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants