<p>I like the idea of this library and besides the mentioned additional use<br>
for "relative bundle path code used on Windows and Mac", we probably can<br>
move some of the plugin configuration config file handling in each plugin<br>
here. There is a lot of duplicated code around.</p>
<p>Since we add a new dependency for other plugins here (currently just Workbench but probably<br>
more in the future), we should emphasize this in the NEWS to help distro packagers when releasing. They probably need to add something like geany-plugins-common package and make all other plugin sub packages depend on this (as it is already on Debian).</p>
<p>Apropos, in order to be able to compile it on Windows I had to add:<br>
<code>-no-undefined</code> to <code>libgeanypluginutils_la_LDFLAGS</code>.<br>
Without, linking fails with:<br>
<code>can't build shared library unless -no-undefined is specified</code>.<br>
I'm not sure whether we should add it there or better in<br>
<code>build/vars-build.mk</code> where it is already specified for <code>AM_LDFLAGS</code>.</p>

<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/622#issuecomment-336713644">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ2GVMa9oC-BRKqLh0DBeGZtWyhXdks5sshB-gaJpZM4P114Z">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJwGeLatxz4HG2AkY8RtGUp0OVDuqks5sshB-gaJpZM4P114Z.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/622#issuecomment-336713644"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany-plugins","title":"geany/geany-plugins","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany-plugins"}},"updates":{"snippets":[{"icon":"PERSON","message":"@eht16 in #622: I like the idea of this library and besides the mentioned additional use\r\nfor \"relative bundle path code used on Windows and Mac\", we probably can\r\nmove some of the plugin configuration config file handling in each plugin\r\nhere. There is a lot of duplicated code around.\r\n\r\nSince we add a new dependency for other plugins here (currently just Workbench but probably\r\nmore in the future), we should emphasize this in the NEWS to help distro packagers when releasing. They probably need to add something like geany-plugins-common package and make all other plugin sub packages depend on this (as it is already on Debian).\r\n\r\nApropos, in order to be able to compile it on Windows I had to add:\r\n`-no-undefined` to `libgeanypluginutils_la_LDFLAGS`.\r\nWithout, linking fails with:\r\n`can't build shared library unless -no-undefined is specified`.\r\nI'm not sure whether we should add it there or better in\r\n`build/vars-build.mk` where it is already specified for `AM_LDFLAGS`."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany-plugins/pull/622#issuecomment-336713644"}}}</script>