[Github-comments] [geany/geany-plugins] External plugins (#440)

[Matthew Brush]

**_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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160609/eb17b399/attachment.html>