<html><head></head><body>I'll backport those changes onto the 0.8 branch tomorrow. I'll send another email when it's done. <br><br><div class="gmail_quote">Am 2. Januar 2017 22:48:21 MEZ, schrieb Filip Pytloun <filip@pytloun.cz>:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">Hello,<br /><br />I checked the changes and as they are done on top of huge refactoring<br />(introduce of <a href="http://vdir.py">vdir.py</a>), backporting it back to 0.8.4 seems to be too<br />risky so I will workaround this occasional FTBFS by marking the test as<br />xfail to avoid autoremoval from stretch.<br /><br />If we want to fix this issue correctly back in stretch after it's is<br />stable, it's possible to do upload of new minor version.<br /><br />Filip<br /><br />On 2016/12/19 19:06, Christian Geier wrote:<br /><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"> Hi,<br /> I could reproduce the issue and have fixed it, see the PR on github [1].<br /> Depending on how urgent this is, you can either just take the os.sync()<br /> commit and apply that to the version currently in Debian (which, while<br /> brute force, certainly fixes this problem), or you might want to wait if<br /> we
find a more elegant solution.<br /> <br /> Christian<br /> <br /> [1] <a href="https://github.com/pimutils/khal/pull/543">https://github.com/pimutils/khal/pull/543</a><br /> <br /> Quoting Filip Pytloun (2016-12-11 13:16:56)<br /><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"> Hello,<br /> <br /> unfortunately I have no longer access to environment where this was<br /> happening. But if you were able to reproduce the issue and this fixed it<br /> for you, I'll apply it and make new release.<br /> <br /> Anyway wouldn't it better to ensure data is written to disk directly<br /> during db updates and other operations? Eg. use O_SYNC for safety as<br /> these operations doesn't happen so often.<br /> <br /> Filip<br /> <br /> On 2016/12/11 01:14, Christian Geier wrote:<br /><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;"> Hi Filip,<br /> could you
perhaps try to change all those sleep()s to `os.sync()`? For<br /> me it seems to fix the issue.<br /> <br /> See [0] for a patch.<br /> <br /> If this doesn fix the issue, we obviously need to move the sync call out<br /> of the tests and into the db update.<br /> <br /> Best regards,<br /> Christian<br /> <br /> [0] <a href="https://github.com/pimutils/khal/commit/0b636f7633e86b9e136b06e9965cd3af0e3918f2">https://github.com/pimutils/khal/commit/0b636f7633e86b9e136b06e9965cd3af0e3918f2</a><br /> <br /></blockquote></blockquote></blockquote></pre></blockquote></div><br>
-- <br>
Sent from my phone. Please excuse my brevity.</body></html>