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

luajit-rocks under windows #11

Open
rns opened this issue Jan 6, 2015 · 1 comment
Open

luajit-rocks under windows #11

rns opened this issue Jan 6, 2015 · 1 comment

Comments

@rns
Copy link

rns commented Jan 6, 2015

Hi, I played with your repo under windows xp, cmake 3.0.2 and msvc 2010 and here are some things I had to do to make it work. Otherwise, very pleasant experience, thanks a lot.

  • With this patch, luarocks (built from this repo) runs command line tools correctly
 if detected.windows then
-   local full_prefix = site_config.LUAROCKS_PREFIX.."\\"..cfg.major_version
+   local full_prefix = site_config.LUAROCKS_PREFIX
    extra_luarocks_module_dir = full_prefix.."\\lua\\?.lua"

    for _, var in ipairs(bins) do
       if defaults.variables[var] then
-         defaults.variables[var] = full_prefix.."\\tools\\"..defaults.variables[var]
+         defaults.variables[var] = full_prefix..defaults.variables[var]
       end
  • With this dir added (files from torch7-split repo) cmake doesn't complain about files missing in bin. Not sure if all of them are needed, though.
Changes to be committed:
  (use "git reset HEAD <file>..." to unstage)

        new file:   luarocks/win32/bin/7z.dll
        new file:   luarocks/win32/bin/7z.exe
        new file:   luarocks/win32/bin/chmod.exe
        new file:   luarocks/win32/bin/cp.exe
        new file:   luarocks/win32/bin/find.exe
        new file:   luarocks/win32/bin/libeay32.dll
        new file:   luarocks/win32/bin/libiconv2.dll
        new file:   luarocks/win32/bin/libintl3.dll
        new file:   luarocks/win32/bin/libssl32.dll
        new file:   luarocks/win32/bin/ls.exe
        new file:   luarocks/win32/bin/md5sum.exe
        new file:   luarocks/win32/bin/mkdir.exe
        new file:   luarocks/win32/bin/mv.exe
        new file:   luarocks/win32/bin/objdump.exe
        new file:   luarocks/win32/bin/pwd.exe
        new file:   luarocks/win32/bin/rm.exe
        new file:   luarocks/win32/bin/rmdir.exe
        new file:   luarocks/win32/bin/test.exe
        new file:   luarocks/win32/bin/uname.exe
        new file:   luarocks/win32/bin/wget.exe
@enricoschroeder
Copy link

@rns thanks for the fix
Those steps are still needed as of now to make things run (tested on Windows8). Could you please include the above changes, this would have saved me (and supposedly others trying to get this to run) quite some time...

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