search

leocrawford / picasawebsync

Sync local directories with picasaweb
MIT License
59 stars 16 forks source link

remove temporary files as soon as possible #9

Closed DrVanScott closed 11 years ago

DrVanScott commented 11 years ago

several times my root file system was completely filled by 'shrink pictures'. As a result, following pictures were uploaded fullsize, or incomplete.

leocrawford commented 11 years ago

I've just checked in a fix. Let me know if it works for you.

DrVanScott commented 11 years ago

This works for me. Note that now a small change is neccessary for my current pull reqeuets:

call(["jhead", "-te", path, imagePath])

must be changed to

call(["jhead", "-te", path, imagePath.name])
leocrawford commented 11 years ago

Just to check, you're going to submit a patch which checks for the presence of jhead before calling it?

On 23 April 2013 09:12, DrVanScott notifications@github.com wrote:

This works for me. Note that now a small change is neccessary for my current pull reqeuets:

call(["jhead", "-te", path, imagePath])

must be changed to

call(["jhead", "-te", path, imagePath.name])

— Reply to this email directly or view it on GitHubhttps://github.com/leocrawford/picasawebsync/issues/9#issuecomment-16844959 .

DrVanScott commented 11 years ago

Just to check, you're going to submit a patch which checks for the presence of jhead before calling it?

Hm, maybe you didn't noticed that my current pull request includes this check? :-)