Geany currently loads only plugins with the ".so" extension on macOS. This extension is still needed since we use autotools for geany-plugins which also generate ".so".
I tested it and it does nothing on linux while on macOS it changes the extension correctly to ".so".
Unless it's a big a problem, I'd like to have this merged for 2.0 so I can use meson for the build.
Fixes #3606.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/3608
-- Commit Summary --
* mscos: use ".so" extension for plugins instead of ".dylib"
-- File Changes --
M plugins/meson.build (6)
-- Patch Links --
https://github.com/geany/geany/pull/3608.patch https://github.com/geany/geany/pull/3608.diff
I don't know Meson nor macOS very well but the changes look good.
Same as @eht16, it looks good if I believe the docs that a value of `[]` make it use the default behavior.
it looks good if I believe the docs that a value of [] make it use the default behavior.
Yeah, kind of weird because otherwise it expects a string but it works. I tested it on linux, using some crazy suffix in the "darwin" branch and it gets ignored and plugins with the "so" extension are created with the [] suffix. So I think it works as expected.
Merging.
Merged #3608 into master.
github-comments@lists.geany.org