See https://bugs.chromium.org/p/chromium/issues/detail?id=834376
https://cs.chromium.org/chromium/infra/appengine/findit/libs/deps/deps_parser.py ehmaldonado@, mind updating the parser here like what you did for ClusterFuzz for the new DEPS format you are proposing?
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/e0b73c52e403fcd418e371fce1ca3661173f7ce3 commit e0b73c52e403fcd418e371fce1ca3661173f7ce3 Author: Edward Lesmes <ehmaldonado@chromium.org> Date: Fri Apr 20 00:08:01 2018 Make DEPS parser forward compatible. See [1] for details [1] https://docs.google.com/document/d/1GSTw4ysm5AYjNR6vMA7NJUxLK2VN9IpOEp_v0u-wyOE/edit?ts=5ad7d242 Bug: 834399 Change-Id: Idc26eaa9b0006bd69b2c0bf72e5ad00a1193f249 Reviewed-on: https://chromium-review.googlesource.com/1020411 Reviewed-by: Shuotao Gao <stgao@chromium.org> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> [modify] https://crrev.com/e0b73c52e403fcd418e371fce1ca3661173f7ce3/appengine/findit/libs/deps/test/deps_parser_test.py [modify] https://crrev.com/e0b73c52e403fcd418e371fce1ca3661173f7ce3/appengine/findit/libs/deps/deps_parser.py
Comment 1 by st...@chromium.org
, Apr 18 2018Components: Tools>Test>FindIt
Owner: ehmaldonado@chromium.org