Author Iztok.Kavkler
Recipients Christophe Simonis, Iztok.Kavkler, ajaksu2, andybuckley, brian.curtin, christian.heimes, edemaine, eric.araujo, iki, meatballhat, michael.foord, pitrou, tarek, tleeuwenburg@gmail.com
Date 2010-08-15.09:14:48
SpamBayes Score 1.82913e-08
Marked as misclassified No
Message-id <1281863691.22.0.369858975446.issue444582@psf.upfronthosting.co.za>
In-reply-to
Content
There is a subtle problem in the reference implementation: it will break if one of the paths in PATH contains quoted path separator. On windows that would be quted with ":

"c:\path;with;sep"

and on *nix something like

/path\:with\:sep

The problem is in the call
path.split(os.path.sep)
To do this properly we would need another helper function, e.g. 
shutil.split_path_list(path)
that would split paths considering quoting. I should also strip quotes  from every path in the list. 

I would write reference implementation, but I'm not sure if I know all the quoting rules of various os-es.
History
Date User Action Args
2010-08-15 09:14:51Iztok.Kavklersetrecipients: + Iztok.Kavkler, edemaine, pitrou, christian.heimes, ajaksu2, tarek, eric.araujo, Christophe Simonis, andybuckley, tleeuwenburg@gmail.com, michael.foord, brian.curtin, meatballhat, iki
2010-08-15 09:14:51Iztok.Kavklersetmessageid: <1281863691.22.0.369858975446.issue444582@psf.upfronthosting.co.za>
2010-08-15 09:14:49Iztok.Kavklerlinkissue444582 messages
2010-08-15 09:14:48Iztok.Kavklercreate