clinton-hall
commented
10 years ago Ah... https://github.com/clinton-hall/nzbToMedia/commit/7d4ccf53cc93fbae84f5dc0222c2bb3318d1cee4
This was not by design... but I believe a minor bug.
So lines 51-57 of TorrentToMedia need to be replaced with
if config.issubsection(inputCategory,["SickBeard"):
fork, fork_params = autoFork("SickBeard", inputCategory)
Torrent_NoLink = int(config()["SickBeard"][inputCategory]["Torrent_NoLink"]) # 0
if fork in config.SICKBEARD_TORRENT and Torrent_NoLink == 1:
Logger.info("MAIN: Calling autoProcessTV to post-process: %s",inputName)
result = autoProcessTV().processEpisode(inputDirectory, inputName, 0, clientAgent=clientAgent, inputCategory=inputCategory)
if result != 0:
Logger.info("MAIN: A problem was reported in the autoProcessTV script.")
Logger.info("MAIN: All done.")
sys.exit()Then, to link all archive files and not extract them, we need to change line 192-208
elif fileExtension in compressedContainer:
if config().issubsection(inputCategory,['SickBeard']) and config()["SickBeard"][inputCategory]["nzbExtractionBy"] == "Destination":
Logger.info("MAIN: Found archive file %s in %s", fileExtension, filePath)
try:
copy_link(filePath, targetDirectory, useLink, outputDestination)
copy_list.append([filePath, os.path.join(outputDestination, file)])
except:
Logger.exception("MAIN: Failed to link file: %s", file)
continue
# find part numbers in second "extension" from right, if we have more than 1 compressed file in the same directory.
if re.search(r'\d+', os.path.splitext(fileName)[1]) and os.path.dirname(filePath) in extracted_folder and not any(item in os.path.splitext(fileName)[1] for item in ['.720p','.1080p','.x264']):
part = int(re.search(r'\d+', os.path.splitext(fileName)[1]).group())
if part == 1: # we only want to extract the primary part.
Logger.debug("MAIN: Found primary part of a multi-part archive %s. Extracting", file)
else:
Logger.debug("MAIN: Found part %s of a multi-part archive %s. Ignoring", part, file)
continue
Logger.info("MAIN: Found compressed archive %s for file %s", fileExtension, filePath)
try:
extractor.extract(filePath, outputDestination)
extractionSuccess = True # we use this variable to determine if we need to pause a torrent or not in uTorrent (don't need to pause archived content)
extracted_folder.append(os.path.dirname(filePath))
except:
Logger.exception("MAIN: Extraction failed for: %s", file)
continueNow the question is, should this just be
[SickBeard]
[[inputCategory]]
ExtractionBy = Destination(note, ExtractionBy, not nzbExtractionBy). Or if we should have a separate nzbExtractionBy and TorrentExtractionBy...
This idea of allow extraction by Destination (SickBeard, CouchPotato etc), Downlaoder (SABnzbd, NZBGet, transmission etc) or nzbToMedia was a work in progress before all the recent changes, so i had forgotten about the specific scenario you use (use hardlink to send ALL files to SickBeard and then have SickBeard do the extraction)... So the question really is, do you download NZBs and if so where do you do the extraction?
@echel0n Really appreciate your feedback on this one. Not sure if you had any ideas towards this, or if you had even considered this "flow"... I certainly hadn't considered this prior to it being requested ;)
ghost
echel0n
Previously, prior to all the latest code changes; TorrentToMedia would hardlink from the seeding directory to a process directory; from which SickBeard would then process, unrar, move, etc.
With latest updates, TorrentToMedia ignores any hard linking and sends the seed directory, original, to SickBeard.
Is this by design? I have SickBeard performing unrar (I've found it more capable), so I need it to perform in a processing directory and leave seeding alone. The original working order, TorrentToMedia doing a hardlink, then SickBeard set to move, was working perfectly.
Any advice?