On 8/6/2021 3:08 PM, Andreas Rheinhardt wrote:
James Almer:
From: Matthieu Patou <mpa...@fb.com>

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

diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c
index 912e881174..fc58277df7 100644
--- a/fftools/cmdutils.c
+++ b/fftools/cmdutils.c
@@ -1160,7 +1160,8 @@ static void print_program_info(int flags, int level)
      av_log(NULL, level, "\n");
      av_log(NULL, level, "%sbuilt with %s\n", indent, CC_IDENT);
- av_log(NULL, level, "%sconfiguration: " FFMPEG_CONFIGURATION "\n", indent);
+    if (flags & SHOW_CONFIG)
+        av_log(NULL, level, "%sconfiguration: " FFMPEG_CONFIGURATION "\n", 
indent);
  }
static void print_buildconf(int flags, int level)
@@ -1203,7 +1204,7 @@ void show_banner(int argc, char **argv, const OptionDef 
*options)
  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);
+    print_program_info (SHOW_COPYRIGHT|SHOW_CONFIG, AV_LOG_INFO);
      print_all_libs_info(SHOW_VERSION, AV_LOG_INFO);
return 0;

This ensures that the configuration will be missing from most bugreports.

Yes, that was my main concern as well. But ffmpeg -version will still show the build configuration, in any case.
_______________________________________________
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