<blockquote>
<p>Without the fix Geany does not copy the filetype files (albeit accidentally), so the user config doesn't hide the system config, so currently Geany does the right, but user unfriendly, thing.</p>
<p>"Fixing" the paths will make Geany do the wrong thing again so it shouldn't happen alone.</p>
</blockquote>
<p>The <code>filetypes.README</code> file inside the filedefs directory says:</p>
<pre><code>Copy files from /usr/local/Cellar/geany/1.24.1/share/geany to this directory to overwrite them. To use the defaults, just delete the file in this directory.
For more information read the documentation (in /usr/local/Cellar/geany/1.24.1/share/doc/geany/html/index.html or visit http://www.geany.org/).
</code></pre>
<p>and the code does just that for the user. Apart from overriding default settings another use case is just to look at the default configuration and without this patch nothing is displayed. Finally, this works the same way as editing <code>filetype_extensions.conf</code>, <code>filetypes.common</code> etc. always worked and the same overriding problem existed there too.</p>
<p>I'd say if the user wants to override just something, he'll have to delete the parts of the file he doesn't want to override.</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/1482#issuecomment-297547512">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ4r_S0ToG5Ct0F-It_XhYlHPquzDks5rz7f7gaJpZM4NIGlb">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ4fYFWyMab1vhcDOyCMoOEpETJKKks5rz7f7gaJpZM4NIGlb.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/1482#issuecomment-297547512"></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 #1482: \u003e Without the fix Geany does not copy the filetype files (albeit accidentally), so the user config doesn't hide the system config, so currently Geany does the right, but user unfriendly, thing.\r\n\u003e\r\n\u003e \"Fixing\" the paths will make Geany do the wrong thing again so it shouldn't happen alone.\r\n\r\nThe `filetypes.README` file inside the filedefs directory says:\r\n\r\n```\r\nCopy files from /usr/local/Cellar/geany/1.24.1/share/geany to this directory to overwrite them. To use the defaults, just delete the file in this directory.\r\nFor more information read the documentation (in /usr/local/Cellar/geany/1.24.1/share/doc/geany/html/index.html or visit http://www.geany.org/).\r\n```\r\n\r\nand the code does just that for the user. Apart from overriding default settings another use case is just to look at the default configuration and without this patch nothing is displayed. Finally, this works the same way as editing `filetype_extensions.conf`, `filetypes.common` etc. always worked and the same overriding problem existed there too.\r\n\r\nI'd say if the user wants to override just something, he'll have to delete the parts of the file he doesn't want to override."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1482#issuecomment-297547512"}}}</script>