Sick-Beard icon indicating copy to clipboard operation
Sick-Beard copied to clipboard

process files by size (smallest first)

Open jayme-github opened this issue 11 years ago • 5 comments

I just hat two downloads with similar problems like the one described in sabnzbd/sabnzbd#32. Of cause I have configured Sabnzbd to not download sample files.

The downloaded archive contained two .mkv files, the episode (1.7GB) and a sample (24MB) which was not identifiable as a sample by name. sabToSickBeard.py first processed the episode.mkv which then got replaced by the (smaller) samle.mkv because the complete archive was marked as priority download due to it was snatched by sick-beard.

With this little change sabToSickBeard.py will process all video files sorted by their filesize (smallest first). In the described case, the samle.mkv would get overriden by episode.mkv. As far as I can say multi-ep archives are processed normally as the names of the video files do match against episode numbers.

Sorry for the pull-request (based off development branch) but it seems as if it is not possible to change existing ones (?) ...

jayme-github avatar Oct 30 '12 09:10 jayme-github

i think you could have rebase and then force pushed.. then update github commit.. but its honestly a lot of work when its just easier to create new and resubmit. i've still yet to figure out an easy way short of cherrypicking into my own branch and submitting. ty, will test and get this pulled

thezoggy avatar Oct 30 '12 13:10 thezoggy

so... does that mean midget's suggestion works fine?

thezoggy avatar Feb 20 '13 06:02 thezoggy

Yep!

Cheers

Please excuse the brevity of this message as it was sent from my iPhone

On 20/02/2013, at 5:04 PM, thezoggy [email protected] wrote:

so... does that mean midget's suggestion works fine?

— Reply to this email directly or view it on GitHubhttps://github.com/midgetspy/Sick-Beard/pull/520#issuecomment-13817357.

aukieranw avatar Feb 20 '13 06:02 aukieranw

wanna submit a pull request (use development branch) so you get the credit then ?

thezoggy avatar Feb 20 '13 06:02 thezoggy

Finally managed to update my commit in this request ;) Suggested code from midget works for me too

jayme-github avatar Feb 20 '13 08:02 jayme-github