<div dir="ltr">JB,<div><br></div><div>As a fellow developer, I can sympathize with your situation.  <span style="line-height:1.5">I commend you for your quick response, your honest explanation, and your efforts to prevent such issues in the future.</span></div><div><br></div><div>Thanks again for all of your work on Kdenlive.</div><div><br></div><div>~Gus</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Sun, May 29, 2016 at 2:51 PM Jean-Baptiste Mardelle <<a href="mailto:jb@kdenlive.org">jb@kdenlive.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
As you might know, a recent bug in Kdenlive's git master caused major data<br>
loss to some users. I am deeply sorry about it and would like to provide<br>
some details about what happened, why and what we did and plan about it.<br>
<br>
1- what happened:<br>
<br>
In Kdenlive, we sometimes manage a lot of temporary data. As I was working<br>
on a new feature (timeline preview), it generated lots of temporary<br>
rendered videos. So at some point, this data needs to be deleted. To do<br>
this, I used a standard Qt class, QDir. However, if for some reason QDir is<br>
not correctly initialized or an operation fails, it defaults to your home<br>
directory. You can guess the end. As with many bugs, the error did not<br>
trigger everytime, so when I tried it before committing, I did not realize<br>
the possible problem.<br>
<br>
2 - how could such stupid code be committed:<br>
<br>
One of the reason it was committed is that I have been working too much in<br>
the last weeks. Working on Kdenlive is great, but at some point I went too<br>
far. Combined with my day job, I did not sleep enough and committed this<br>
stupid mistake.<br>
<br>
3 - what about the future, is it safe to use Kdenlive ?<br>
<br>
As soon as I realized the extend of the problem, I commented out the<br>
problematic code and deleted the PPA packages. Git master is safe now, as<br>
is current kdenlive-master PPA built from the 26th of May :<br>
kdenlive - 4:16.07.70+git201605260633~ubuntu16.04.1<br>
<br>
Causing data loss on our users deeply affected me, and I think it should<br>
also be a big reminder for everyone that using a daily built development<br>
version from git is not safe, and that you must backup your data!<br>
<br>
I will put a big warning now on the development version, and will discuss<br>
with Vincent about possible changes in our PPA/development policy to help<br>
prevent such problems.<br>
<br>
I will also of course revise our temporary data handling to make sure such<br>
a mistake does not happen twice.<br>
<br>
So again, I send my sincere apologies and hope we can continue to move<br>
forward in the next months.<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
_______________________________________________<br>
kdenlive mailing list<br>
<a href="mailto:kdenlive@kde.org" target="_blank">kdenlive@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kdenlive" rel="noreferrer" target="_blank">https://mail.kde.org/mailman/listinfo/kdenlive</a><br>
</blockquote></div>