Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent infinite loop with process dir and keep original files #111

Open
terbolous opened this issue Oct 7, 2013 · 13 comments
Open

Prevent infinite loop with process dir and keep original files #111

terbolous opened this issue Oct 7, 2013 · 13 comments
Labels

Comments

@terbolous
Copy link

Any chance that you could import this pull into this fork?
midgetspy#629

@gitrickm
Copy link

gitrickm commented Oct 7, 2013

Yes please! I just hit this problem this morning.

I have vuze downloading the torrents, then when completed vuze moves the file to the directory that sickbeard monitors. If SB moves the file, vuze gets errors because the file disappears. I thought I'd try telling SB to leave the file there such that I could manually tell vuze to remove it, but then I found the SB was post-processing the files over and over every 10 minutes.

Am I doing this all wrong?

@junalmeida
Copy link
Owner

Yes, but I prefer to wait this to be merged at least with the development branch of midgetspy, to avoid problems if he decide to use another pull.

@terbolous
Copy link
Author

Roger. Any chance of this being merged as well?
midgetspy#686

@lansinghd
Copy link

As noted in #149, midgetspy/Sick-Beard@9f309a4 merges this functionality into the development branch and seems to be functional when I manually merged to this torrent fork for testing:

2014-03-25 11:47:04 DEBUG    POSTPROCESSER :: Ignoring file: /path/filename.ext looks like it's been processed already

This creates a native solution to allow the "Scan & Process" and "Keep Original Files" features work together without triggering unnecessary disk i/o and notifications every 10 minutes due to seeding.

I'd love to see this functionality in the torrent fork as I depend on the Transmission RPC client to set a separate path for Sickbeard downloads - which works perfectly, thanks for maintaining this fork!

@junalmeida
Copy link
Owner

Nice. Going to test this as soon as possible @lansinghd .

@wjbeckett
Copy link

Any update on this?
I'm really looking to have this functionality!

@dmarcelino
Copy link

+1 This functionality would probably lower the CPU usage on my Synology, where I run SickBeard.

@Mikk36
Copy link

Mikk36 commented Nov 4, 2014

+1

It's really annoying to see SickBeard process the downloaded files over and over, because without keeping the original files, you can't share the downloaded files. Isn't this supposed to be the torrent edition of SickBeard?

@rui-castro
Copy link

I'm also waiting for this fixed to be merged.
Until that day, I'm using my fork of Sick-Beard (https://github.com/rui-castro/Sick-Beard), that is exactly like this, but has a fix to the "infinite processing loop" applied.

@dmarcelino
Copy link

@rui-castro, would you know if your current fork also prevents SickBeard from repeatedly searching for subtitles in cases where it can't find them?
I'm seeing this particularly with anime episodes with odd name patten, where SickBeard is constantly trying to find subtitles driving my Synology's CPU usage.

@rui-castro
Copy link

@dmarcelino, besides the patch to prevent the infinite processing loop, my fork is exactly like this one. So, no.

@dmarcelino
Copy link

I see, thanks!

@benyjr
Copy link

benyjr commented Sep 14, 2015

Any up date on this? If I could leave the original files where they are, I wouldn't have so many seeding errors with files being moved all over the place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

9 participants