Skip to content

[fix] : ensure ffmpeg test script always specifies output file#5

Merged
maulik-mk merged 1 commit intodevfrom
fix/ffmpeg-output-file
Feb 17, 2026
Merged

[fix] : ensure ffmpeg test script always specifies output file#5
maulik-mk merged 1 commit intodevfrom
fix/ffmpeg-output-file

Conversation

@maulik-mk
Copy link
Copy Markdown
Owner

@maulik-mk maulik-mk commented Feb 17, 2026

CI

This commit fixes a bug in the ffmpeg test script where a comment inside a multi-line command caused the output file argument to be omitted. The script now correctly passes the output file to ffmpeg, ensuring the transcoding test runs as intended and prevents the "At least one output file must be specified" error.


This pull request makes a minor update to the run_transcode_test() function in test/ffmpeg.test.sh, clarifying that only the first 10 seconds of the video are processed during transcoding. The main change is moving the comment about processing the first 10 seconds to directly above the relevant Docker command for better code clarity.

  • Comment about processing only the first 10 seconds of the video moved above the Docker command in run_transcode_test() for improved clarity. [1] [2]

Comment : https://github.com/maulik-mk/mp.ii-worker/issues/6#issue-3953279502

@maulik-mk
Copy link
Copy Markdown
Owner Author

continue

On #6

@maulik-mk maulik-mk deleted the fix/ffmpeg-output-file branch April 5, 2026 13:19
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.

1 participant