<div dir="ltr">On Wed, Aug 19, 2015 at 11:52 AM, Thomas Martitz <span dir="ltr"><<a href="mailto:kugel@rockbox.org" target="_blank">kugel@rockbox.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<span><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I would suggest removing the file saving option mentioned in (3) above and have just a single settings option like<br>
<br>
use_gio_file_operations<br>
<br>
Opinions?<br>
<br>
</blockquote>
<br>
<br></span>
IIRC saving to temp file, then renaming isn't the default, because of concerns since both the rename and setting permissions can fail depending on the filesystem and permissions on the parent directly.<br></blockquote><div><br></div><div>The use_gio_unsafe_file_saving=true is the default (and it should be kept this way) and it does the renaming.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
IIRC#2 it's called unsafe because the above issues are even more apparent on remote file systems.<br></blockquote><div><br></div><div>In my opinion use_atomic_file_saving=true is much worse on remote file systems because it destroys file metadata.</div><div> </div><div>Cheers,</div><div><br></div><div>Jiri</div></div></div></div>