-
Notifications
You must be signed in to change notification settings - Fork 25
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
Pr cmake #8
base: master
Are you sure you want to change the base?
Pr cmake #8
Conversation
dirkcgrunwald
commented
Dec 8, 2022
- Add build multi-platform build support using cmake
- perform builds using github actions
- Fix minor Windows / C++ incompatibilities (intermixing wchar/char output)
I cloned this PR and did my normal CMake based build steps and it built just fine. I didn't have to read any build instructions and there weren't any surprises. Built on macOS+M1. |
I've created rust wrapper for this branch and we're using it in production at decent scale. I'd love to see this work merged, otherwise there's a decent chance of having long-lived forks people depend on. |
We're going through some internal reorganization with the larger PropLib effort over this winter, which will include work being done in this branch. Sorry its taken longer than it probably should. Just how things are being prioritized. |
I'm happy to help however I can. Good to know the patch has been useful. @JayKickliter - I'd be interested in learning about the project you're doing. Looking at the HexTree and geoprop repos, it looks like you're developing something that I had been hoping to work on within NSF SpectrumX similar to what e.g. https://cloudrf.com/ does but in a way that can support open datasets and new prop models. Email is [email protected] |
@dirkcgrunwald you nailed what we're doing. I'll follow up with an email |