Hi
Dne Fri, 15 May 2009 14:31:38 +0200
Pavol Rusnak
My first version of the change just removed dir as optional parameter (module should figure it out by itself what the tempdir is, regardless of platform). But later in the code (function delete_tmpdir) there is an explicit check if the path starts with '/tmp' and this couldn't be guaranteed anymore (module could use '/var/tmp').
What do you think about rewriting the code so it doesn't explicitly set tempdir in these calls and removing the check in delete_tmpdir ?
I think it would be best. But I totally miss the point of delete_tmpdir logic. What is wrong on deleting directories which osc created in any temporary directory user has defined? -- Michal Čihař | http://cihar.com | http://blog.cihar.com