New issue
Advanced search Search tips

Issue 640333 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Aug 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

GN desc configs doesn't show computed configs

Project Member Reported by vmi...@chromium.org, Aug 23 2016

Issue description

The ffmpeg target (/usr/local/google/code/chrome/src/third_party/ffmpeg/BUILD.gn) removes certain default configs and adds others.

Examples:

  configs -= [ "//build/config/compiler:chromium_code" ]
  configs -= [ "//build/config/compiler:default_optimization" ]
  configs += [ "//build/config/compiler:optimize_max" ]

These configs changes don't show when running the "gn desc" command.


> gn desc out/debug_android/ //third_party/ffmpeg configs

//build/config:feature_flags
//build/config/compiler:compiler
//build/config/compiler:clang_stackrealign
//build/config/compiler:compiler_arm_fpu
//build/config/compiler:chromium_code
//build/config/compiler:default_include_dirs
//build/config/compiler:default_optimization
//build/config/compiler:default_omit_frame_pointer
//build/config/compiler:default_symbols
//build/config/compiler:no_rtti
//build/config/compiler:runtime_library
//build/config/sanitizers:default_sanitizer_flags
//build/config/gcc:no_exceptions
//build/config/gcc:symbol_visibility_hidden
//build/config/android:default_cygprofile_instrumentation
//build/config/clang:find_bad_constructs
//build/config/clang:extra_warnings
//build/config:debug
//build/config:default_libs
//build/config:shared_library_config
//build/config/android:hide_native_jni_exports
//third_party/ffmpeg:ffmpeg_dependent_config
 
Components: Build
Owner: vmi...@chromium.org
Status: Assigned (was: Untriaged)
//third_party/ffmpeg is equivalent to //third_party/ffmpeg:ffmpeg . So, you're printing the configs for the shared library, but the config manipulation you're referring to is happening on the :ffmpeg_internal target.

So, you should see the right thing if you do `gn desc out/debug_android //third_party/ffmpeg:ffmpeg_internal configs`.

Let me know if you don't.

Comment 3 by vmi...@chromium.org, Aug 30 2016

Status: WontFix (was: Assigned)
Thanks, confirmed!

Sign in to add a comment