search

dakusan / PlexPlaylistImporter

Import playlists into Plex
Other
81 stars 18 forks source link

Use `os.path` functions to support all platforms. #3

Closed mattspitz closed 8 years ago

mattspitz commented 8 years ago

Instead of using \\ assumptions, use os.path functions, which take care of the platform differences.

Tested on Linux on a playlist with relative and absolute paths. Not tested on Windows.

mattspitz commented 8 years ago

@dakusan mind reviewing this when you get a chance? Thanks!

dakusan commented 8 years ago

Finally got the sucker merged in how I wanted :-) - Wanted to give you credit for the bug fix, but kept the original code and just added os.path.isabs, as everything that was in the original was still needed.

mattspitz commented 8 years ago

Cool! Thanks for following up!

On Fri, Jun 17, 2016 at 11:54 PM, Jeffrey Riaboy notifications@github.com wrote:

Finally got the sucker merged in how I wanted :-) - Wanted to give you credit for the bug fix, but kept the original code and just added os.path.isabs, as everything that was in the original was still needed.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dakusan/PlexPlaylistImporter/pull/3#issuecomment-226920078, or mute the thread https://github.com/notifications/unsubscribe/AAKA4wkbAQQtvpeYeJygUah2YsCi1h0nks5qM2vegaJpZM4HGoyD .