<blockquote>
<p>As I see it, the <code>Makefile.am</code> changes are backward-compatible. Thus I am not sure where the problem lies?</p>
</blockquote>

<p>It currently doesn't, I was just referring to possible future occurrences of a similar issue.</p>

<blockquote>
<p>A few comments:</p>

<ul>
<li>You should not bundle (other projects) m4 files in Git; these files are considered as development files and should be included in the relevant <code>-dev(el)</code> packages in your favorite distro.</li>
</ul>
</blockquote>

<p>Well.  There's 2 problems here</p>

<ol>
<li>can we rely on every distros having autoconf-archive packaged, and can we trust it will contain ax_python_devel.m4?</li>
<li>this would be a build dependency (from Git) no matter whether you want geanypy or not.

<ul>
<li>well, OK, we could use <code>m4_ifdef()</code> to conditionally use that.  But it would be one more possible reason why the plugin is not available, and a lot less straightforward than a normal build dep.</li>
</ul>
</li>
</ol>

<blockquote>
<ul>
<li>If you really <em>must</em> trick <code>aclocal</code>, you can increase the <code>#serial</code> line so that it always picks the one in the repo. But you should not.</li>
</ul>
</blockquote>

<p>Oh, so it looks at the serial?  I learned something :)</p>

<blockquote>
<ul>
<li>Do you really want Python 2 support? If not, I think Python 3 is distributed with a <code>pkg-config</code> file which you should definitely use. Maybe add a <code>./configure --with-python={2|3}</code> switch?</li>
</ul>
</blockquote>

<p>Hum, maybe. In Debian there's also a pkg-config file for python2.7.  Though, I don't think either one have the same information as the ones fetched by <code>ax_python_devel</code> -- but maybe they are just fine.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/geany/geany-plugins/pull/339#issuecomment-172526587">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJw2Ugyj8qfV13LeiQB_egI_1l46xks5pbN82gaJpZM4HGnhk.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/339#issuecomment-172526587"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>