Skip to content
This repository has been archived by the owner on Nov 20, 2020. It is now read-only.

require("luars232") results in Invalid access to memory location. #154

Closed
jayachar88 opened this issue Oct 19, 2013 · 7 comments
Closed
Assignees
Labels

Comments

@jayachar88
Copy link

Have LuaDist installed installed on Windows 7 64-bit (professional edition), on a Intel Core i5 laptop.

Here's the simple test that shows the error -

C:\Users\jay>lua
Lua 5.1.5 Copyright (C) 1994-2012 Lua.org, PUC-Rio

local luars232 = require("luars232")
error loading module 'luars232' from file 'C:\Binaries-LuaDist-batteries-0.9.8-W
indows-x86\lib\lua\luars232.dll':
Invalid access to memory location.

stack traceback:
[C]: ?
[C]: in function 'require'
stdin:1: in main chunk
[C]: ?

@drahosp
Copy link
Member

drahosp commented Oct 21, 2013

Thank you for reporting the issue. I have updated the binary of librs232-1.0.3. Please use luadist from the commandline to update your distribution as follows

cd [path to luadist]\bin
luadist remove librs232
luadist install librs232

@jayachar88
Copy link
Author

While attempting the "luadist remove librs232", I get an error popup. Here's the URL to the error popup screenshot -- http://imgur.com/lCEI4Iq

@drahosp
Copy link
Member

drahosp commented Oct 21, 2013

Can you please provide more information on the nature of the error. LuaDist itself does not display any popup windows so my suspicion is that this may be your antivirus/firewall.

@jayachar88
Copy link
Author

Problem solved, and also verified that the updated luars232 library no
longer gives the memory violation error while the DLL is being loaded.

To allow Zerobrane Studio IDE to work with LuaDist on Windows, I had to
define the LUA_DEV, LUA_PATH and LUA_CPATH. However this was probably
rendering "dist" library itself unloadable and thus the error.

@drahosp
Copy link
Member

drahosp commented Oct 21, 2013

Excellent, also note there is (an older) version of ZBS preset in the package already that needs no LUA_PATH and LUA_CPATH settings.

@drahosp drahosp closed this as completed Oct 21, 2013
@jayachar88
Copy link
Author

Thanks. I noticed that. Any plans of updating the repos with latest ZBS?
Significant progress have been made on it by Paul.

On Oct 22, 2013 4:14 AM, "Peter Drahoš" notifications@github.com wrote:

Excellent, also note there is (an older) version of ZBS preset in the
package already that needs no LUA_PATH and LUA_CPATH settings.


Reply to this email directly or view it on GitHub.

@drahosp
Copy link
Member

drahosp commented Oct 22, 2013

Actually @pkulchenko is working on ZBS and LuaDist integration. He seems to have solved most of the issues, hopefully we may have a new LuaDist release and ZBS integration ready before the Lua workshop.

@drahosp drahosp added the bug label Feb 18, 2014
@drahosp drahosp self-assigned this Feb 18, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants