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

Windows 64 bit build #4

Closed
starius opened this issue Apr 26, 2015 · 8 comments
Closed

Windows 64 bit build #4

starius opened this issue Apr 26, 2015 · 8 comments

Comments

@starius
Copy link
Contributor

starius commented Apr 26, 2015

lua-rapidjson can not be built under Windows 64 bit in Appveyor

I installed it for testing purposes in my project in Appveyor.

x86 version can be built, but 64 bit version fails.

One of error messages:

rapidjson.obj : error LNK2019: unresolved external symbol _lua_pushfstring referenced in
 function "int __cdecl decode<struct rapidjson::GenericStringStream<struct rapidjson::UTF8
<char> > >(struct lua_State *,struct rapidjson::GenericStringStream<struct rapidjson::UTF8
<char> > *)" 
@xpol
Copy link
Owner

xpol commented Jun 7, 2015

Sorry too late back to here.

lua-rapidjson use luarocks and cmake to build.

but luarocks does not have 64 bit cmake build support yet. And have reported here luarocks/luarocks/issues/382

you may need to use cmake directly:

  1. checkout lua-rapidjson
  2. in the lua-rapidjson porject root run cmake -H. -Bbuild.luarocks -G Visual Studio 2013 Win64
  3. install the result manually

@starius
Copy link
Contributor Author

starius commented Jun 7, 2015

Thank you!

@starius starius closed this as completed Jun 7, 2015
@ignacio
Copy link
Contributor

ignacio commented Jun 17, 2015

I'm working on a PR for LuaRocks to address this issue.

@ignacio
Copy link
Contributor

ignacio commented Jun 17, 2015

Here is a build using my proposed PR:

https://ci.appveyor.com/project/ignacio/lua-rapidjson/build/0.1.0.12

It passes for x86, it fails for x64 but it is something related to busted. If you see, lua-rapidjson is being compiled as 64 bits.

@ignacio
Copy link
Contributor

ignacio commented Jun 17, 2015

BTW, the PR is here.

@xpol
Copy link
Owner

xpol commented Jun 18, 2015

@ignacio Thank you very much!

@ignacio
Copy link
Contributor

ignacio commented Jun 25, 2015

@xpol A fix has landed on LuaRocks allowing to build with cmake on x64. It is on the scm version but I presume there is gonna be a new release sometime soon.

@xpol
Copy link
Owner

xpol commented Jun 27, 2015

@ignacio Great! Thank you very much.

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

No branches or pull requests

3 participants