-
Notifications
You must be signed in to change notification settings - Fork 81
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
Cannot get formed FFMPEG cli command as string #348
Comments
Sorry for late reply, cli command is logged, please check logging framework configuration. |
so your idea is to use integration tests and verify result via ffprobe. It makes sense, though I would like to have kind of unit tests, where I execute some logic and then can check what command is formed to make sure my logic works fine. This is one use case. Also, I didn't find how to test certain cases in the way you said. For example, I scale video to different aspect ratio and use padding with black borders - it seems not possible or difficult to test that video scaled correctly and has black borders. But I could check at least command I formed. |
Hi,
Currently the library provides ability to build command for ffmpeg in builder-like style and then execute it, but I didn't find a way how to get raw ffmpeg command which will be executed
Result of execution contains some useful data about processing, though also misses what command was executed
To Reproduce
Expected behavior
extra method or way to get raw ffmpeg command. Smth like this:
currently, I din't find a way how to solve this via subclassing, because I cannot override
ProcessHandler<FFmpegResult> runner
creation to create another runner with extended info.Additional context
Cases where raw ffmpeg command is useful
any feedback is appreciated, if you can implement these possibilities or just advise how I can bypass this with some workarounds, for instance how to override and enhance this behavior without creating separate fork. Thanks.
P.S. I really like this lib, good and convenient api.
The text was updated successfully, but these errors were encountered: