cleishm / libcypher-parser

Cypher Parser Library
Apache License 2.0
147 stars 39 forks source link

Add Win32 support with CMake build #13

Closed Loupi closed 5 years ago

Loupi commented 5 years ago

I was willing to add Windows support to node-cypher-parser. So here is what this PR does:

image

The CMake build depends on these libraries, which can also be built with CMake and the CMake GUI:

Here are the environment variables that need to be set. D:/Sources is the directory where all the dependencies were git cloned and built with cmake and Visual Studio devenv. They must all target the same architecture.

CHECK_INCLUDE_DIR D:/Sources/check/build/src CHECK_LIBRARY D:/Sources/check/build/src/Release/check.lib COMPAT_LIBRARY D:/Sources/check/build/lib/Release/compat.lib FMEM_INCLUDE_DIR D:/Sources/fmem/build/gen FMEM_LIBRARY D:/Sources/fmem/build/Release/fmem.lib LEG_PROGRAM D:/Sources/peg/build/Release/leg.exe

The CMake GUI can be used to point the cypher build to it's dependencies: cmake-libcypher-parser

cleishm commented 5 years ago

This is so awesome. Thanks!

I have a few questions which I'll put into a review, although I don't know much about cmake - so they might be a little naive!

Loupi commented 5 years ago

The review items have been addressed, and quick_parser support was also added.

cleishm commented 5 years ago

Rebased and merged.