-
Notifications
You must be signed in to change notification settings - Fork 19
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
Figure out elegant packaging of the addon on OSX #42
Comments
The MacOS homebrew |
Not sure what my last comment tried to tell : I guess same goes for linux ... can we deploy to a clean machine by just using I can test for Mac when that works through PR #33 |
Regarding this and #33 (comment): On Windows, there's a specific order in which libraries are searched for - maybe something similar exists on Mac, and the one in |
We got a working and packaged Windows build, generated on Linux (relevant for #41) 🥳 I will push everything (in the form of GitHub Actions and README docs) soon, but here's the general approach for future reference: |
Windows packaging is done and #60 also publishes the packaged artifact! Same for Linux. Only Mac OS left to do. |
I found dylibbundler which seems like it does what we need on Mac. I integrated it into our build action and successfully created a Mac build: https://github.com/boku-ilen/geodot-plugin/actions/runs/3943687695 Would be great if anyone could test! btw: I also tried creating a build for the new ARM Macs, but that's currently held back by the lack of a GitHub Action runner with that chip. |
On Linux, we just copied over the
libgdal.so
intodemo/addons/geodot/x11/
(as described in the README). On Windows, we can probably do something similar, but we need to figure out which DLLs are needed. Also we should document any other dependencies.Same for OSX - as described by @clemens-tolboom, we might be able to use a copy of
/usr/local/Cellar/gdal/3.2.1/lib/libgdal.dylib
.Required for #36.
The text was updated successfully, but these errors were encountered: