-
Notifications
You must be signed in to change notification settings - Fork 49
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 release binaries #32
Comments
The C code in ERFA is extremely standards-compliant and should be trivial to build on Windows. It should not need a Unix environment. Someone who knows about windows and has a Microsoft compiler should find it very easy to do. It's more a case of finding a volunteer. |
You're right. I was able to cross-compile using
Posted the binaries here for the time being: https://bintray.com/kbarbary/generic/erfa/1.2.0/view/files#files/ It might be nice to make this part of the release process. My autotools skills are not currently up to the task. |
Am going through old issues to see if they are still relevant. Is this one still an issue given that we have |
Hi @timj @kbarbary @mhvk @mdboom @phn I was able to build successfully ERFA and tests using only Visual Studio 2022. Did not use any cygwin, mingw or similar environments. Of course I had to slightly modify two header (erfa.h, erfaextra.h) and one source (erfaversion.c) file to build successfully everything. I have two questions:
BR, Ladislav |
@mhvk |
Hmm, hadn't realized that! You can still pull the PR with
(where (Not sure it won't need a rebase, though...) |
OK, thanks for hint. Using this PR I was able to create a new Visual Studio solution and build library + tests. Everything works. One note, that it only creates a static library (.lib file) but skips shared library (.dll file) creation. When I manually set the created project properties to produce shared library, it however creates but the result .dll file has no export symbols. This is due to missing __declspec(dllexport) keyword in header files at the function signatures. Otherwise no issues. |
Is there any hope of building and distributing Windows shared library binaries along with the source tarballs for each release? This would make it easier to support Windows in ERFA.jl. Ref: JuliaAstro/ERFA.jl#17
The text was updated successfully, but these errors were encountered: