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

Windows compatibility #6

Open
davegelwan opened this issue Apr 5, 2023 · 0 comments · May be fixed by #7
Open

Windows compatibility #6

davegelwan opened this issue Apr 5, 2023 · 0 comments · May be fixed by #7

Comments

@davegelwan
Copy link
Collaborator

davegelwan commented Apr 5, 2023

I was unable to install this package on Windows 10. I worked through some issues compiling/linking the C++ dependency, but was unable to load it in Julia.

I tried using MinGW 64-bit to run the build.sh script (which calls ./configure and make). At first, there was a configure error regarding OpenMP:
configure: error: don't know how to enable OpenMP
and this error for finding an MPI library:
configure: error: could not find mpi library for --enable-mpi

I followed these instructions for combining MSMPI with GCC. I made the following changes to the build.sh script to link against libmsmpi.a:
image

Next, there was an error due to a space in the MinGW installation path:
image (162)
This was resolved by replacing all $(SHELL) with "$(SHELL)" in Makefile.in

Next, the Makefile generated from configure has the JLCXX_H include path commented out
image (164)

Finally, srand48 and drand48 functions do not exist on Windows:
image
This can be addressed via:
https://social.msdn.microsoft.com/Forums/vstudio/en-US/9356f151-51d2-412d-8889-d53230cc6ec9/porting-srand48-and-drand48-of-unix-like-platform-based-source?forum=vclanguage

After all those changes, I was able to successfully compile, link, and generate fmm.so. Unfortunately, it does not load in Julia:
image (166)
I had to add the file extension to @wrapmodule(joinpath(module_path, "fmm")) --> @wrapmodule(joinpath(module_path, "fmm.so.exe"))

@davegelwan davegelwan linked a pull request Apr 5, 2023 that will close this issue
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

Successfully merging a pull request may close this issue.

1 participant