<blockquote>
<p>I'm not sure this change is required at all.</p>
</blockquote>
<p>Yes, you are right, I've just tried and the circular dependency is no problem. Still I prefer to think of tag manager as just one specific part of geany and not a separate library so I kept the patch. But I have no problem to remove it if others disagree.</p>
<blockquote>
<p>Are their other projects that have a tagmanager or fork or has it become more or less geany-specific?</p>
</blockquote>
<p>It's basically completely Geany-specific. It was originally based on this</p>
<p><a href="https://sourceforge.net/projects/tagmanager/">https://sourceforge.net/projects/tagmanager/</a></p>
<p>which got abandoned 14 years ago, then picked and modified by Anjuta, then the Anjuta version got picked and modified by Geany (Anjuta dropped tagmanager in the meantime) and then heavily modified by me in the past years. So while the overall logic of maintaining tags is similar to the original one, the sources are very different. There's no living upstream version of what we have in Geany.</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/pull/1414#issuecomment-298174577">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ4tx3YOmHFGtI3I0NaGt1nvF-kXUks5r01J1gaJpZM4MNMkZ">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ9sUfsn0eLwUtuedMI94P5N_MY2zks5r01J1gaJpZM4MNMkZ.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/pull/1414#issuecomment-298174577"></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","title":"geany/geany","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"}},"updates":{"snippets":[{"icon":"PERSON","message":"@techee in #1414: \u003e I'm not sure this change is required at all.\r\n\r\nYes, you are right, I've just tried and the circular dependency is no problem. Still I prefer to think of tag manager as just one specific part of geany and not a separate library so I kept the patch. But I have no problem to remove it if others disagree.\r\n\r\n\u003e Are their other projects that have a tagmanager or fork or has it become more or less geany-specific?\r\n\r\nIt's basically completely Geany-specific. It was originally based on this\r\n\r\nhttps://sourceforge.net/projects/tagmanager/\r\n\r\nwhich got abandoned 14 years ago, then picked and modified by Anjuta, then the Anjuta version got picked and modified by Geany (Anjuta dropped tagmanager in the meantime) and then heavily modified by me in the past years. So while the overall logic of maintaining tags is similar to the original one, the sources are very different. There's no living upstream version of what we have in Geany."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1414#issuecomment-298174577"}}}</script>