<p><strong><em>This PR isn't meant to be merged.</em></strong></p>
<p>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.</p>
<p>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 <code>AC_CONFIG_SUBDIRS</code> and re-use their existing (sub) build system, but I fear that would be more complicated, and I know it would be more slow.</p>
<p>Any thoughts?</p>
<hr>
<h4>You can view, comment on, or merge this pull request online at:</h4>
<p> <a href='https://github.com/geany/geany-plugins/pull/440'>https://github.com/geany/geany-plugins/pull/440</a></p>
<h4>Commit Summary</h4>
<ul>
<li>Move GP build cruft for Overview into build/overview</li>
<li>Remove Overview fork from GP</li>
<li>Add submodule for Overview plugin</li>
<li>Use relative paths to allow Autotools to work</li>
</ul>
<h4>File Changes</h4>
<ul>
<li>
<strong>A</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-0">.gitmodules</a>
(3)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-1">Makefile.am</a>
(2)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-2">build/overview.m4</a>
(12)
</li>
<li>
<strong>R</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-3">build/overview/AUTHORS</a>
(0)
</li>
<li>
<strong>R</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-4">build/overview/COPYING</a>
(0)
</li>
<li>
<strong>R</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-5">build/overview/ChangeLog</a>
(0)
</li>
<li>
<strong>A</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-6">build/overview/Makefile.am</a>
(33)
</li>
<li>
<strong>R</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-7">build/overview/NEWS</a>
(0)
</li>
<li>
<strong>R</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-8">build/overview/README</a>
(0)
</li>
<li>
<strong>A</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-9">build/overview/overview.m4</a>
(6)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-10">configure.ac</a>
(3)
</li>
<li>
<strong>A</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-11">overview</a>
(1)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-12">overview/Makefile.am</a>
(3)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-13">overview/data/Makefile.am</a>
(2)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-14">overview/data/prefs.ui</a>
(356)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-15">overview/overview/Makefile.am</a>
(29)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-16">overview/overview/overviewcolor.c</a>
(305)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-17">overview/overview/overviewcolor.h</a>
(79)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-18">overview/overview/overviewplugin.c</a>
(233)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-19">overview/overview/overviewplugin.h</a>
(36)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-20">overview/overview/overviewprefs.c</a>
(447)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-21">overview/overview/overviewprefs.h</a>
(78)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-22">overview/overview/overviewprefspanel.c</a>
(273)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-23">overview/overview/overviewprefspanel.h</a>
(47)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-24">overview/overview/overviewscintilla.c</a>
(1302)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-25">overview/overview/overviewscintilla.h</a>
(86)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-26">overview/overview/overviewui.c</a>
(376)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany-plugins/pull/440/files#diff-27">overview/overview/overviewui.h</a>
(42)
</li>
</ul>
<h4>Patch Links:</h4>
<ul>
<li><a href='https://github.com/geany/geany-plugins/pull/440.patch'>https://github.com/geany/geany-plugins/pull/440.patch</a></li>
<li><a href='https://github.com/geany/geany-plugins/pull/440.diff'>https://github.com/geany/geany-plugins/pull/440.diff</a></li>
</ul>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany-plugins/pull/440">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/ABDrJzPbWh6imbyJ3lE5Bsf6Q3kdjMusks5qKM6AgaJpZM4Iyk9X">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ5CU2bAapQ9lloSeCHT_XcGJsixAks5qKM6AgaJpZM4Iyk9X.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
<link itemprop="url" href="https://github.com/geany/geany-plugins/pull/440"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>