Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

createVideo does not wait for ffmpeg to finish #9

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

andre-luiz-dos-santos
Copy link

A script that loads a page and calls process.exit at the end, creates all the image files, but does not create the video file.

The problem is that stop() returns a Promise that is resolved before ffmpeg has had enough time to complete.

A script that loads a page and calls process.exit at the end, creates all the image files, but does not create the video file.

The problem is that stop() returns a Promise that is resolved before ffmpeg has had enough time to complete.
@andre-luiz-dos-santos
Copy link
Author

Actually, instead of printing stdout, I'd rather just return it:

return await promisify(exec)(_ffmpegCommand);

@acuesta
Copy link

acuesta commented Jun 10, 2022

Hi everyone!
Is this PR going to be reviewed and merged ?
We are looking for the same feature and is a nive job!
:)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants