-
Notifications
You must be signed in to change notification settings - Fork 73
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
Fix Windows build process #11
base: master
Are you sure you want to change the base?
Conversation
This happens because you need to pass an absolute path, now you broke that. |
Can you specify to which paths, relative or absolute, |
I think I saw it wrong when looking at the changes to README initially. I would suggest to make sure both variables are expanded to relative paths before passing them to |
You probably mean extended to absolute paths, don't you? Perhaps you could describe how you would build the project on windows, giving |
Took me a while, but finally had some time to try building master: git submodule update --init
cmake -B build -DIDA_SDK_DIR:FILEPATH=c:\IDA90SP1\idasdk90 -DHEXRAYS_SDK_DIR:FILEPATH=c:\IDA90SP1\idasdk90
cmake --build build I think things could be improved in a few ways:
|
When building NtRays for Windows, it tries to link against "idasdk90\lib\x64_win_vc_64\ida.lib.lib" which does not exist. Removing the .lib suffix in CMakeLists fixes the problem.
I have also updated the build instructions to make it easier to build the plugin.