Skip to content
This repository has been archived by the owner on Oct 7, 2021. It is now read-only.

-report option not working #302

Closed
JayParikh20 opened this issue Dec 22, 2019 · 4 comments
Closed

-report option not working #302

JayParikh20 opened this issue Dec 22, 2019 · 4 comments
Assignees
Labels
bug Something isn't working fixed

Comments

@JayParikh20
Copy link

JayParikh20 commented Dec 22, 2019

Built library using ./android.sh --enable-gpl --enable-x264 --lts

Sample command
"-y -i " + srcMp4Path + " -vf select=eq(pict_type\\,PICT_TYPE_I) -report -loglevel debug -vsync 2 -f image2 "+ outMp4Path
output should be in thumbnails-%02d.jpeg format

Environment variable set using "setNativeEnvironmentVariable" function in Config file
FFREPORT=file=/storage/emulated/0/test/fflog.txt:level=48

Expected Behavior
Report option should print complete log in detail, for eg PICT_TYPE of each frame.

Current Behavior
It is saving the file in the given FFREPORT location, but log has only the command line printed.

Command line: -y -i .... etc thats it!

Environment
Platform: Kali Linux for Android
Architecture: [arm-v7a, arm-v7a-neon, arm64-v8a, x86, x86_64]
FFmpeg Source brach: 4.3-dev
Android NDK version: R20b

@tanersener tanersener self-assigned this Dec 23, 2019
@tanersener tanersener added the bug Something isn't working label Dec 23, 2019
@tanersener
Copy link
Owner

tanersener commented Dec 23, 2019

Thank you for reporting this issue.

mobile-ffmpeg replaces log_callback_report function with mobileffmpeg_log_callback_function to implement LogCallback functionality of the library. Unfortunately this change breaks the behaviour of -report option. Because -report option uses log_callback_report to write output to a file.

I'll work on the fix and provide a solution in the next release.

@tanersener tanersener added this to To do in Next MobileFFmpeg release due to 01.2020 via automation Dec 23, 2019
@tanersener tanersener moved this from To do to In progress in Next MobileFFmpeg release due to 01.2020 Dec 23, 2019
tanersener added a commit that referenced this issue Jan 3, 2020
@tanersener
Copy link
Owner

fixed in development branch

@tanersener tanersener moved this from In progress to Done in Next MobileFFmpeg release due to 01.2020 Jan 3, 2020
@tanersener
Copy link
Owner

Fixed in v4.3.1 release.

@JayParikh20
Copy link
Author

Thank you so much!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working fixed
Projects
No open projects
Development

No branches or pull requests

2 participants