<p>Oh shoot, this doesn't produce the same result as the current implementation.</p>

<p>I completely forgot about metadata, with this implementation all files given new names are new files, so they have new file metadata.  Owner and permissions may change, but specifically the execute bit won't be carried over from the original file.  So shell scripts or Python scripts or similar will no longer run until the user manually re-adds the execute.</p>

<p>With the current implementation the metadata is preserved depending on the type of saving in use, the "safe" file save mode will not preserve the metadata but the others will.</p>

<p>Sorry <a href="https://github.com/konsolebox" class="user-mention">@konsolebox</a> we probably should stick to the existing implementation.</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/1190#issuecomment-241949135">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ5nsRm_8WOK39nJQQwcqyj_SfhUdks5qi70ogaJpZM4JrfCw">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ8YFRz3-rPjhGgUiCiaMLg1x5Ogpks5qi70ogaJpZM4JrfCw.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/1190#issuecomment-241949135"></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":"@elextr in #1190: Oh shoot, this doesn't produce the same result as the current implementation.\r\n\r\nI completely forgot about metadata, with this implementation all files given new names are new files, so they have new file metadata.  Owner and permissions may change, but specifically the execute bit won't be carried over from the original file.  So shell scripts or Python scripts or similar will no longer run until the user manually re-adds the execute.\r\n\r\nWith the current implementation the metadata is preserved depending on the type of saving in use, the \"safe\" file save mode will not preserve the metadata but the others will.\r\n\r\nSorry @konsolebox we probably should stick to the existing implementation."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1190#issuecomment-241949135"}}}</script>