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

Issue 69243003: Revert 234233 "Add mojo_idl_generator.gypi" (Closed)

Created:
7 years, 1 month ago by Nico
Modified:
7 years, 1 month ago
Reviewers:
DaveMoore
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 234233 "Add mojo_idl_generator.gypi" Broke all windows try jobs. Example: http://build.chromium.org/p/tryserver.chromium/builders/win/builds/124746 FAILED: E:\b\depot_tools\python_bin\python.exe gyp-win-tool action-wrapper environment.x86 mojom_test_target_Generate_C___source_files_from_mojom_files_ff8ddd79354be39f4d49ff68f7050968.f7de501d36c9a2def832b8cb44ee34af.rsp Traceback (most recent call last): File "../mojo/public/bindings/mojom_bindings_generator.py", line 12, in <module> from parser import mojo_parser ImportError: cannot import name mojo_parser ninja: build stopped: subcommand failed. > Add mojo_idl_generator.gypi > NOT FOR REVIEW > BUG= > R=darin@chromium.org > > Review URL: https://codereview.chromium.org/53173008 TBR=davemoore@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=234277

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -566 lines) Patch
trunk/src/mojo/DEPS View 1 chunk +0 lines, -1 line 0 comments Download
trunk/src/mojo/mojo.gyp View 2 chunks +4 lines, -19 lines 0 comments Download
trunk/src/mojo/public/bindings/mojo_idl.py View 0 chunks +-1 lines, --1 lines 0 comments Download
trunk/src/mojo/public/bindings/mojom_bindings_generator.gypi View 1 chunk +0 lines, -46 lines 0 comments Download
trunk/src/mojo/public/bindings/mojom_bindings_generator.py View 1 chunk +0 lines, -46 lines 0 comments Download
trunk/src/mojo/public/bindings/sample/generated/sample_service.h View 0 chunks +-1 lines, --1 lines 0 comments Download
trunk/src/mojo/public/bindings/sample/generated/sample_service.cc View 0 chunks +-1 lines, --1 lines 0 comments Download
trunk/src/mojo/public/bindings/sample/generated/sample_service_internal.h View 0 chunks +-1 lines, --1 lines 0 comments Download
trunk/src/mojo/public/bindings/sample/mojom/sample_service.h View 1 chunk +0 lines, -132 lines 0 comments Download
trunk/src/mojo/public/bindings/sample/mojom/sample_service.cc View 1 chunk +0 lines, -253 lines 0 comments Download
trunk/src/mojo/public/bindings/sample/mojom/sample_service_internal.h View 1 chunk +0 lines, -41 lines 0 comments Download
trunk/src/mojo/public/bindings/sample/sample_service.idl View 0 chunks +-1 lines, --1 lines 0 comments Download
trunk/src/mojo/public/bindings/sample/sample_service.mojom View 1 chunk +30 lines, -32 lines 0 comments Download
trunk/src/mojo/public/bindings/sample/sample_test.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
Nico
7 years, 1 month ago (2013-11-11 20:53:50 UTC) #1
Nico
7 years, 1 month ago (2013-11-11 20:54:29 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r234277.

Powered by Google App Engine
This is Rietveld 408576698