fail if build-ffmpeg fails

cvanwinkle · cvanwinkle · commit ba0110e8a4fa · 2021-12-07T15:17:15.000-06:00
diff --git a/descript/build-ffmpeg-descript.py b/descript/build-ffmpeg-descript.py
@@ -59,8 +59,9 @@ def buildFFmpeg(script_dir, log_file):
         '-b',                       # build
         '--full-shared',            # custom Descript shim to build shared libraries instead of static
         '--enable-gpl-and-free']    # custom Descript shim to build GPL but not non-free (libpostproc is needed by Beamcoder and requires GPL)
-    log_file.write(' '.join(args) + '\n\n')    
-    subprocess.call(args, env=env, stdout=log_file)
+    log_file.write(' '.join(args) + '\n\n')
+    log_file.flush()
+    subprocess.run(args, env=env, stdout=log_file, stderr=log_file, check=True)
 
 #
 #

Original file line number	Diff line number	Diff line change
`@@ -59,8 +59,9 @@ def buildFFmpeg(script_dir, log_file):`
`59`	`59`	`'-b', # build`
`60`	`60`	`'--full-shared', # custom Descript shim to build shared libraries instead of static`
`61`	`61`	`'--enable-gpl-and-free'] # custom Descript shim to build GPL but not non-free (libpostproc is needed by Beamcoder and requires GPL)`
`62`		`- log_file.write(' '.join(args) + '\n\n')`
`63`		`- subprocess.call(args, env=env, stdout=log_file)`
	`62`	`+ log_file.write(' '.join(args) + '\n\n')`
	`63`	`+ log_file.flush()`
	`64`	`+ subprocess.run(args, env=env, stdout=log_file, stderr=log_file, check=True)`
`64`	`65`
`65`	`66`	`#`
`66`	`67`	`#`