Duplicate build argument declaration, v8_check_microtasks_scopes_consistency, in v8 and iso build.gn file |
||
Issue description
Including //services/data_decoder/public/cpp in the deps causing the ios-simulator bot fail to generate build files.
Error is followed:
""" to /b/s/w/ir/cache/builder/src/out/Debug-iphonesimulator/args.gn.
/b/s/w/ir/cache/builder/src/buildtools/mac/gn gen //out/Debug-iphonesimulator --check
-> returned 1
ERROR at //v8/BUILD.gn:155:44: Duplicate build argument declaration.
v8_check_microtasks_scopes_consistency = ""
^-
Here you're declaring an argument that was already declared elsewhere.
You can only declare each argument once in the entire build so there is one
canonical place for documentation and the default value. Either move this
argument to the build config file (for visibility everywhere) or to a .gni file
that you "import" from the files where you need it (preferred).
See //ios/BUILD.gn:18:44: Previous declaration.
v8_check_microtasks_scopes_consistency = false
^----
See also "gn help buildargs" for more on how build arguments work.
See //services/data_decoder/BUILD.gn:68:5: which caused the file to be included.
"//v8:external_startup_data",
^---------------------------
GN gen failed: 1
step returned non-zero exit code: 1
stdout logs: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8937262449588426096/+/steps/generate_build_files__mb___with_patch_/0/stdout
The bot failed with this particular cl: https://chromium-review.googlesource.com/c/chromium/src/+/1155719/16
,
Sep 14
As a short-term workaround, we could disable this feature on iOS, to unblock landing.
,
Sep 17
I disabled the feature on IOS to unblock crbug.com/853290. I also found that some other components that is depended on //services/data_decoder, those components' unit tests set is also excluded ios. e.g. components/cast_channel, and components/payments/content/utility. |
||
►
Sign in to add a comment |
||
Comment 1 by wychen@chromium.org
, Sep 14