[Sugar-devel] [PATCH Browse] Remove temporary downloaded (cancelled) files SL #3973

Manuel Kaufmann humitos at gmail.com
Tue Oct 2 08:24:52 EDT 2012


On Tue, Oct 2, 2012 at 6:40 AM, Martin Langhoff
<martin.langhoff at gmail.com> wrote:
> Good, but not enough. Compare mtime also to our boot time (ie: now -
> uptime). If the file is from before we booted, needs to be nuked.

You say that I should add this test as well, right? I mean, compare if
the file is older than 1 day AND check the mtime with our boot time or
remove my test and use only this one?

I think that comparing just with the boot time is not enough as well.
Think about this scenario: turn on the XO, use Browse for a while,
download and cancel many files, close Brose and close the lid. It will
suspend. So, if you repeat these steps every day and you never power
off / restart the XO, these kind of files won't be removed. Am I
right?

-- 
Kaufmann Manuel
Blog: http://humitos.wordpress.com/
Porfolio: http://fotos.mkaufmann.com.ar/
PyAr: http://www.python.com.ar/


More information about the Sugar-devel mailing list