**_This PR isn't meant to be merged._** I made it to show how we could allow "external" plugins, keeping their proper repositories as submodules to avoid the "all in" problem with GP currently where a plugin either has to completely move their plugin into GP repository, thus loosing the ability to have a separate Git host, repository history, and even it's own (standalone) Autotools build system.
It's still a little annoying having to maintain separate Autotools files for inside GP vs external, but it's not so bad (at least with Overview which is trivial to build). In a perfect world external plugins using Autotools themselves also could just add `AC_CONFIG_SUBDIRS` and re-use their existing (sub) build system, but I fear that would be more complicated, and I know it would be more slow. Any thoughts? You can view, comment on, or merge this pull request online at: https://github.com/geany/geany-plugins/pull/440 -- Commit Summary -- * Move GP build cruft for Overview into build/overview * Remove Overview fork from GP * Add submodule for Overview plugin * Use relative paths to allow Autotools to work -- File Changes -- A .gitmodules (3) M Makefile.am (2) D build/overview.m4 (12) R build/overview/AUTHORS (0) R build/overview/COPYING (0) R build/overview/ChangeLog (0) A build/overview/Makefile.am (33) R build/overview/NEWS (0) R build/overview/README (0) A build/overview/overview.m4 (6) M configure.ac (3) A overview (1) D overview/Makefile.am (3) D overview/data/Makefile.am (2) D overview/data/prefs.ui (356) D overview/overview/Makefile.am (29) D overview/overview/overviewcolor.c (305) D overview/overview/overviewcolor.h (79) D overview/overview/overviewplugin.c (233) D overview/overview/overviewplugin.h (36) D overview/overview/overviewprefs.c (447) D overview/overview/overviewprefs.h (78) D overview/overview/overviewprefspanel.c (273) D overview/overview/overviewprefspanel.h (47) D overview/overview/overviewscintilla.c (1302) D overview/overview/overviewscintilla.h (86) D overview/overview/overviewui.c (376) D overview/overview/overviewui.h (42) -- Patch Links -- https://github.com/geany/geany-plugins/pull/440.patch https://github.com/geany/geany-plugins/pull/440.diff --- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/440
