On 8/6/2021 7:08 PM, Soft Works wrote:


-----Original Message-----
From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of
James Almer
Sent: Friday, 6 August 2021 23:49
To: ffmpeg-devel@ffmpeg.org
Subject: Re: [FFmpeg-devel] [PATCH v2] fftools/cmdutils: don't print build
configuration by default

On 8/6/2021 6:38 PM, Soft Works wrote:


-----Original Message-----
From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of
James Almer
Sent: Friday, 6 August 2021 23:17
To: ffmpeg-devel@ffmpeg.org
Subject: [FFmpeg-devel] [PATCH v2] fftools/cmdutils: don't print
build configuration by default

Suggested-by: ffm...@fb.com
Signed-off-by: James Almer <jamr...@gmail.com>
---
   fftools/cmdutils.c | 4 +++-
   1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c index
912e881174..e6ae749167 100644
--- a/fftools/cmdutils.c
+++ b/fftools/cmdutils.c
@@ -1152,6 +1152,8 @@ static void print_all_libs_info(int flags, int
level) static void print_program_info(int flags, int level)  {
       const char *indent = flags & INDENT? "  " : "";
+    // Show config only if level is verbose or higher
+    int config_level = level <= AV_LOG_INFO ? AV_LOG_VERBOSE :
+ level;

I don't think it will work this way because level is hardcoded to
AV_LOG_INFO in both upstream calls.

I can hardcode it to VERBOSE if that's preferred. I made it this way since there
could be new callers in the future that could choose a different level.
Otherwise we could just remove the level parameter altogether.

Yes, that would make it more clear. But another parameter is needed,
because when it's called from show_version, it should always be printed,
regardless of loglevel.

Does "ffmpeg -version" not show it for you with this patch? It does for me.


I have it like this:

void show_banner(int argc, char **argv, const OptionDef *options)
{
     int idx = locate_option(argc, argv, options, "version");
     if (hide_banner || idx)
         return;

     print_program_info (INDENT|SHOW_COPYRIGHT, AV_LOG_INFO, 0);  <==
     print_all_libs_info(INDENT|SHOW_CONFIG,  AV_LOG_INFO);
     print_all_libs_info(INDENT|SHOW_VERSION, AV_LOG_DEBUG);
}

int show_version(void *optctx, const char *opt, const char *arg)
{
     av_log_set_callback(log_callback_help);
     print_program_info (SHOW_COPYRIGHT, AV_LOG_INFO, 1) );  <==
     print_all_libs_info(SHOW_VERSION, AV_LOG_INFO);

     return 0;
}

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".


_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to