dibyendumajumdar / ravi

Ravi is a dialect of Lua, featuring limited optional static typing, JIT and AOT compilers
http://ravilang.github.io/
Other
1.16k stars 60 forks source link

failed to compile in msys2+mingw64 on windows 10 64bits #237

Closed retsyo closed 1 year ago

retsyo commented 2 years ago

I run

cd ravi-master
mkdir d
cd d
cmake -G "MSYS Makefiles" ..
make

but

[  0%] Building C object CMakeFiles/libravinojit_static.dir/src/ravi_alloc.c.obj
D:/tmp/ravi-master/src/ravi_alloc.c:1668:1: error: multiple storage classes in declaration specifiers
 1668 | static FORCEINLINE void* win32mmap(size_t size) {
      | ^~~~~~
D:/tmp/ravi-master/src/ravi_alloc.c:1674:1: error: multiple storage classes in declaration specifiers
 1674 | static FORCEINLINE void* win32direct_mmap(size_t size) {
      | ^~~~~~
D:/tmp/ravi-master/src/ravi_alloc.c:1681:1: error: multiple storage classes in declaration specifiers
 1681 | static FORCEINLINE int win32munmap(void* ptr, size_t size) {
      | ^~~~~~
make[2]: *** [CMakeFiles/libravinojit_static.dir/build.make:622: CMakeFiles/libravinojit_static.dir/src/ravi_alloc.c.obj] Error 1
make[1]: *** [CMakeFiles/Makefile2:206: CMakeFiles/libravinojit_static.dir/all] Error 2
make: *** [Makefile:136: all] Error 2
dibyendumajumdar commented 2 years ago

hi thanks for the feedback. I have never tried building on MSYS.

dibyendumajumdar commented 1 year ago

hi @retsyo
I am not sure what steps to follow to reproduce the issue. Did you try to build with cmake from outside MSYS22 - or did you use the CMake that is provided with MSYS2? Please can you share details of how to reproduce? Regards

dibyendumajumdar commented 1 year ago

Hi @retsyo I pushed a fix