Skip to content
This repository has been archived by the owner on Dec 3, 2023. It is now read-only.

Commit

Permalink
fix for missing env variable in win
Browse files Browse the repository at this point in the history
If the PATH is not provided to `python` executable in env variables then youtube-dl fails. This will enable parent to pass process env variables. Will add proper python search future within youtube-dl later on.
  • Loading branch information
przemyslawpluta committed Sep 7, 2014
1 parent 3cb82a9 commit 4115b5e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/youtube-dl.js
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ function call(video, args1, args2, options, callback) {

var opt = [file, args];

if (isWin) { opt = ['python', [file].concat(args)]; }
if (isWin) { opt = [process.env.PYTHON || 'python', [file].concat(args)]; }

// Call youtube-dl.
execFile(opt[0], opt[1], options, function(err, stdout, stderr) {
Expand Down

0 comments on commit 4115b5e

Please sign in to comment.