search

rembo10 / headphones

Automatic music downloader for SABnzbd
GNU General Public License v3.0
3.39k stars 600 forks source link

If download client and Headphones host are different, directory check fails #3191

Closed Xalaxis closed 9 months ago

Xalaxis commented 5 years ago

At https://github.com/rembo10/headphones/blob/master/headphones/deluge.py#L608 the Deluge code checks to see whether the 'Move after complete' directory exists and if not creates it. If the Headphones host and download client host are on different devices this will fail because the paths are different.

Temporary workaround for those affected, comment out line 608.

Is this check actually required? I'll investigate whether Deluge automatically creates the path.

Xalaxis commented 5 years ago

It seems this wrong path might also be checked for completed downloads rather than the local path. Generalising title.

Xalaxis commented 5 years ago

Temporary workaround, "Force Post-Process Albums"

Xalaxis commented 5 years ago

Reminder to self, check https://github.com/rembo10/headphones/blob/master/headphones/postprocessor.py#L53 later

Xalaxis commented 5 years ago

So the best way I can see of resolving this, is implementing a new config value for "Host and download client are in different locations" which when set forces the use of headphones.CONFIG.DOWNLOAD_TORRENT_DIR for the checks in deluge.py and postprocessor.py.

rembo10 commented 10 months ago

Hi, so sorry for missing this. Are you still using this workaround? I'm still not 100% sure I understand the problem. So you have deluge on one machine and headphones on another machine.... sorry I'm a little bit thick at the moment :sweat_smile: , would you mind just explaining it like I'm 5 :sweat_smile: ?

Xalaxis commented 9 months ago

It's no problem - Honestly It's been ages since I've used this so I'm afraid I can't remember what I was trying to fix. Happy to close this, and thanks for getting back to it :)