Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(162)

Issue 2160533004: Revert of [GN] Add JSON project writer (Closed)

Created:
4 years, 5 months ago by grt (UTC plus 2)
Modified:
4 years, 5 months ago
Reviewers:
brettw, matt.k
CC:
chromium-reviews, Dirk Pranke, tfarina
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of [GN] Add JSON project writer (patchset #11 id:200001 of https://codereview.chromium.org/2064533002/ ) Reason for revert: Broke Win8 GYP (dbg) builder: [822/12249] LINK_EMBED gn.exe FAILED: gn.exe gn.exe.pdb E:\b\depot_tools\python276_bin\python.exe gyp-win-tool link-with-manifests environment.x86 True gn.exe "E:\b\depot_tools\python276_bin\python.exe gyp-win-tool link-wrapper environment.x86 False link.exe /nologo /OUT:gn.exe @gn.exe.rsp" 1 mt.exe rc.exe "obj\tools\gn\gn.gn.exe.intermediate.manifest" obj\tools\gn\gn.gn.exe.generated.manifest ..\..\build\win\compatibility.manifest gn_lib.lib(gn_lib.command_desc.obj) : error LNK2019: unresolved external symbol "public: static class std::unique_ptr<class base::DictionaryValue,struct std::default_delete<class base::DictionaryValue> > __cdecl DescBuilder::DescriptionForTarget(class Target const *,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,bool,bool,bool)" (?DescriptionForTarget@DescBuilder@@SA?AV?$unique_ptr@VDictionaryValue@base@@U?$default_delete@VDictionaryValue@base@@@std@@@std@@PBVTarget@@ABV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@3@_N22@Z) referenced in function "bool __cdecl commands::`anonymous namespace'::PrintTarget(class Target const *,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,bool,bool,bool,bool)" (?PrintTarget@?A0x51180473@commands@@YA_NPBVTarget@@ABV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@_N222@Z) gn_lib.lib(gn_lib.command_desc.obj) : error LNK2019: unresolved external symbol "public: static class std::unique_ptr<class base::DictionaryValue,struct std::default_delete<class base::DictionaryValue> > __cdecl DescBuilder::DescriptionForConfig(class Config const *,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &)" (?DescriptionForConfig@DescBuilder@@SA?AV?$unique_ptr@VDictionaryValue@base@@U?$default_delete@VDictionaryValue@base@@@std@@@std@@PBVConfig@@ABV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@3@@Z) referenced in function "bool __cdecl commands::`anonymous namespace'::PrintConfig(class Config const *,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,bool)" (?PrintConfig@?A0x51180473@commands@@YA_NPBVConfig@@ABV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@_N@Z) gn_lib.lib(gn_lib.command_gen.obj) : error LNK2019: unresolved external symbol "public: static bool __cdecl JSONProjectWriter::RunAndWriteFiles(class BuildSettings const *,class Builder const *,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,bool,class Err *)" (?RunAndWriteFiles@JSONProjectWriter@@SA_NPBVBuildSettings@@PBVBuilder@@ABV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@222_NPAVErr@@@Z) referenced in function "bool __cdecl commands::`anonymous namespace'::RunIdeWriter(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,class BuildSettings const *,class Builder *,class Err *)" (?RunIdeWriter@?A0x1eff038c@commands@@YA_NABV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@PBVBuildSettings@@PAVBuilder@@PAVErr@@@Z) gn.exe : fatal error LNK1120: 3 unresolved externals Traceback (most recent call last): File "gyp-win-tool", line 323, in <module> sys.exit(main(sys.argv[1:])) File "gyp-win-tool", line 29, in main exit_code = executor.Dispatch(args) File "gyp-win-tool", line 71, in Dispatch return getattr(self, method)(*args[1:]) File "gyp-win-tool", line 179, in ExecLinkWithManifests subprocess.check_call(ldcmd + add_to_ld) File "E:\b\depot_tools\python276_bin\lib\subprocess.py", line 540, in check_call raise CalledProcessError(retcode, cmd) subprocess.CalledProcessError: Command 'E:\b\depot_tools\python276_bin\python.exe gyp-win-tool link-wrapper environment.x86 False link.exe /nologo /OUT:gn.exe @gn.exe.rsp gn.exe.manifest.res' returned non-zero exit status 1120 Original issue's description: > [GN] Add JSON project writer > > Output is a JSON file containing information about targets. The generator > can also optionally invoke a python script on generated file. > > Example: > gn gen --ide=json ./out-json --json-ide-script=//scripts/custom-ide-generator.py \ > --ison-ide-script-args="additional script arguments" > > Also implements --format=json for gn desc as described in https://bugs.chromium.org/p/chromium/issues/detail?id=620132 > > BUG= > > Committed: https://crrev.com/d2edeb9a743ad5a2046e655997277d3bb630db03 > Cr-Commit-Position: refs/heads/master@{#406064} TBR=brettw@chromium.org,matej.knopp@gmail.com # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Committed: https://crrev.com/5a1b4b1725ec0685c99741620752e9f65db17d90 Cr-Commit-Position: refs/heads/master@{#406074}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+783 lines, -1393 lines) Patch
M AUTHORS View 1 chunk +0 lines, -1 line 0 comments Download
M tools/gn/BUILD.gn View 2 chunks +0 lines, -4 lines 0 comments Download
M tools/gn/command_desc.cc View 5 chunks +733 lines, -290 lines 0 comments Download
M tools/gn/command_gen.cc View 10 chunks +9 lines, -55 lines 0 comments Download
M tools/gn/commands.h View 2 chunks +0 lines, -6 lines 0 comments Download
M tools/gn/commands.cc View 7 chunks +28 lines, -41 lines 0 comments Download
D tools/gn/desc_builder.h View 1 chunk +0 lines, -27 lines 0 comments Download
D tools/gn/desc_builder.cc View 1 chunk +0 lines, -694 lines 0 comments Download
D tools/gn/json_project_writer.h View 1 chunk +0 lines, -26 lines 0 comments Download
D tools/gn/json_project_writer.cc View 1 chunk +0 lines, -217 lines 0 comments Download
M tools/gn/target.cc View 2 chunks +12 lines, -13 lines 0 comments Download
M tools/gn/visibility.h View 2 chunks +0 lines, -8 lines 0 comments Download
M tools/gn/visibility.cc View 3 chunks +1 line, -11 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
grt (UTC plus 2)
Created Revert of [GN] Add JSON project writer
4 years, 5 months ago (2016-07-18 20:00:36 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2160533004/1
4 years, 5 months ago (2016-07-18 20:02:25 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 5 months ago (2016-07-18 20:03:21 UTC) #5
commit-bot: I haz the power
4 years, 5 months ago (2016-07-18 20:05:01 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/5a1b4b1725ec0685c99741620752e9f65db17d90
Cr-Commit-Position: refs/heads/master@{#406074}

Powered by Google App Engine
This is Rietveld 408576698