<div class="gmail_quote">On Fri, Apr 23, 2010 at 11:07 PM, Albert Astals Cid <span dir="ltr"><<a href="mailto:aacid@kde.org">aacid@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">
</div>Please add context to your i18n calls, at least to the "General" string since<br>
it's quite difficult to know how to translate "General" if you don't know what<br>
it is about.<br></blockquote><div><br>Thanks for the feedback.<br>
I've turned every i18n into a i18nc, and added a comment to the .desktop
file, I hope that's all of it.<br> <br></div><div class="gmail_quote">On Sat, Apr 24, 2010 at 3:00 AM,
Michael Pyne <span dir="ltr"><<a href="mailto:mpyne@kde.org">mpyne@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Now with the understanding that I haven't so much as looked at
this, something<br>
that does this type of thing (monitoring free space on a partition)
would be<br>
very useful for /var/tmp as well, since filling up the /var/tmp
partition<br>
apparently plays very poorly with KPixmapCache (the obvious code paths I
look<br>
at in KPixmapCache do check for proper file open and resize results, so I
have<br>
no quick fix for that).<br>
<br>
So, it would be nice if this could be easily extended to checking other<br>
partitions (even if it's just a command line or kconfig option with no
GUI).<font color="#888888"><br>
</font></blockquote></div><br>Yes, adding support for watching multiple partitions is the one remaining item on my todo list. <br>Although if you don't have separate partitions, watching your /home will work for /var/tmp too :)<br>
<br>I didn't have time to do it yet, but decided to submit the current code that is tested and working nicely because of the upcoming soft feature freeze, and probably delay adding that feature for 4.6.<br>Also, this way openSUSE, Kubuntu and other distros that already carry this code or the old patch can start using the upstream version.<br>
</div><br>Ivo Anjo<br>