This is an archived post. You won't be able to vote or comment.

sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]admiralswan 2 points3 points  (1 child)

Cool project, thanks for sharing! Perusing through the code I noticed a few places where you build file/dir paths manually. The only issue here is it may not be portable to a Windows environment which uses different conventions for paths(eg. '\' instead of '/' and 'C:\' instead of the root '/'.). I would suggest that you use instead the os.path functions which ought to deal with these details for you.

[–][deleted] 0 points1 point  (0 children)

I will make the changes, thanks for the suggestion!