Code review comment for lp:~phill-ridout/openlp/1012110_2.0

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

I can't test this (as I don't have Windows), but there are a few things you need to change:

On line 17 you should use os.path.normpath(), don't do a manual find and replace. See http://docs.python.org/2/library/os.path.html#os.path.normpath

On line 22 you should use os.path.join().

review: Needs Fixing

« Back to merge proposal