[Sugar-devel] [PATCH sugar] don't delete bundles on storage media (SL #2456)

Simon Schampijer simon at schampijer.de
Mon Nov 29 07:53:17 EST 2010


On 10/17/2010 12:51 PM, Sascha Silbe wrote:
> model.get_file() already wraps data store checkouts in TempFilePath. Because
> misc.get_bundle() unconditionally wrapped the path returned by
> model.get_file() in TempFilePath as well, bundles located on storage media
> (which are referenced by their actual location, not copied or linked) were
> deleted after misc.get_bundle() finished.
>
> Signed-off-by: Sascha Silbe<sascha-pgp at silbe.org>

Acked-by: Simon Schampijer <simon at schampijer.de>

This one looks good to me. I made sure I had the same results as well.

I think is a candidate for a backport (use case: handing an activity 
bundle to another learner via an external device).

Regards,
    Simon


More information about the Sugar-devel mailing list