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

does not work with lnum patch #5

Closed
Nossiac opened this issue Jun 8, 2015 · 1 comment
Closed

does not work with lnum patch #5

Nossiac opened this issue Jun 8, 2015 · 1 comment

Comments

@Nossiac
Copy link

Nossiac commented Jun 8, 2015

this library does not work with lnum patch. (i'm work with lua 5.1.5 + lnum patch)

lua: /usr/lib/lua/md5.lua:376: bad argument #2 to 'format' (integer expected, got number)
stack traceback:
    [C]: in function 'format'
    /usr/lib/lua/md5.lua:376: in function 'sumhexa'
    /usr/lib/lua/md5.lua:380: in function 'sum'

similar error reported here, http://lua-users.org/lists/lua-l/2012-08/msg00170.html . i tried the workaround, but no luck....

@kikito
Copy link
Owner

kikito commented Jun 8, 2015

I don't have that patch, and have no particular need for it. I can't develop this myself. If you want, please feel free to send me a pull request, I can review and eventually accept it (provided that it does not break the unpatched version).

I must also warn you that the Lua 5.1 implementation fails for big strings (files bigger than some KBs). If you need it for that, I would recommend looking at other places (for example invoking an external tool, or updating to LuaJIT or Lua 5.2)

For now, I am closing this issue (but please send a pull request if you can)

@kikito kikito closed this as completed Jun 8, 2015
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

2 participants