search

wnielson / Plex-Remote-Transcoder

A distributed transcoding backend for Plex
MIT License
646 stars 58 forks source link

Check_config, library paths with spaces fix #37

Closed Awarua- closed 8 years ago

Awarua- commented 8 years ago

Fix check_config command for share paths which have spaces.

Expected behaviour: Host 192.168.2.203 Connect: OK Path: '/media/nfs/Media/Anime' User: nobody Mode: 777 WARN: Not owned by plex user Path: '/media/nfs/Media/Documentaries' User: nobody Mode: 777 WARN: Not owned by plex user Path: '/media/nfs/Media/HD Movies' User: UNKNOWN Mode: 777 WARN: Not owned by plex user Path: '/media/nfs/Media/Tv' User: UNKNOWN Mode: 777 WARN: Not owned by plex user Path: '/usr/lib/plexmediaserver/' User: root Mode: 755 WARN: Not owned by plex user Path: '/opt/plex/tmp' User: plex Mode: 755

Actual behaviour: Host 192.168.2.203 Connect: OK Path: '/media/nfs/Media/Anime' User: nobody Mode: 777 WARN: Not owned by plex user Path: '/media/nfs/Media/Documentaries' User: nobody Mode: 777 WARN: Not owned by plex user Path: '/media/nfs/Media/HD Movies' Traceback (most recent call last): File "/usr/local/bin/prt", line 9, in load_entry_point('prt==0.4.1', 'console_scripts', 'prt')() File "build/bdist.linux-x86_64/egg/prt.py", line 684, in main File "build/bdist.linux-x86_64/egg/prt.py", line 542, in check_config

wnielson commented 8 years ago

Thanks for the PR! If you change it to use pipes.quote(path) instead of "'" + path "'", then I'll accept it. Cheers!

Awarua- commented 8 years ago

@wnielson Fixed, sorry python is not my strong suit

wnielson commented 8 years ago

Thanks!