Commit 65bf714
committed
Fix NameError: subprocess not defined in _spawn_transcribe (issue #6)
After v0.12.6/v0.12.7's HLS download fix, the next code path users hit was
_spawn_transcribe → subprocess.Popen, which crashed with NameError because
the function uses the bare name `subprocess` while the rest of the module
imports it locally as `import subprocess as _sp`.
Reproduces immediately when the user has --transcribe enabled during
download. Fix is a one-line module-level `import subprocess`.
User report (issue #6):
Saved → /Volumes/.../EE4802IE4213Week1.mp4
[transcribe] Starting background transcription: EE4802IE4213Week1.mp4
[error] Unexpected error: name 'subprocess' is not defined1 parent fae1e78 commit 65bf714
2 files changed
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
| 27 | + | |
27 | 28 | | |
28 | 29 | | |
29 | 30 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| |||
0 commit comments