aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRicardo Garcia <sarbalap+freshmeat@gmail.com>2011-02-25 21:53:26 +0100
committerRicardo Garcia <sarbalap+freshmeat@gmail.com>2011-02-25 21:53:26 +0100
commitda273188f33ace3b48290a8cf35d36033a6fa960 (patch)
tree659f88593c2d6d8f3bfe85ce51710ce38fbec981
parent1bd9258272dd4884b5c159ae77e20ae75176e8d1 (diff)
Catch possible exceptions when running ffprobe
-rwxr-xr-xyoutube-dl13
1 files changed, 8 insertions, 5 deletions
diff --git a/youtube-dl b/youtube-dl
index 9e9be6778..617ac1339 100755
--- a/youtube-dl
+++ b/youtube-dl
@@ -2619,10 +2619,13 @@ class FFmpegExtractAudioPP(PostProcessor):
@staticmethod
def get_audio_codec(path):
- handle = subprocess.Popen(['ffprobe', '-show_streams', path],
- stderr=file(os.path.devnull, 'w'), stdout=subprocess.PIPE)
- output = handle.communicate()[0]
- if handle.wait() != 0:
+ try:
+ handle = subprocess.Popen(['ffprobe', '-show_streams', path],
+ stderr=file(os.path.devnull, 'w'), stdout=subprocess.PIPE)
+ output = handle.communicate()[0]
+ if handle.wait() != 0:
+ return None
+ except (IOError, OSError):
return None
audio_codec = None
for line in output.split('\n'):
@@ -2646,7 +2649,7 @@ class FFmpegExtractAudioPP(PostProcessor):
filecodec = self.get_audio_codec(path)
if filecodec is None:
- self._downloader.to_stderr(u'WARNING: no audio codec found in file')
+ self._downloader.to_stderr(u'WARNING: unable to obtain file audio codec with ffprobe')
return None
more_opts = []