Create a shared python lib to parse DEPS file |
||||||
Issue descriptionCurrently, there are quite a few apps that need to parse DEPS file: Findit, Predator, ChromimiumDash, Pinpoint, ClusterFuzz, etc. It would be great if the infra could have a standalone python lib that could be reused by all the apps mentioned above. As gclient uses ast parsing (IIUC), it is also better to avoid exec for security reason.
,
Nov 13 2017
,
Nov 15 2017
Yep, this is definitely something that we want to provide. We already *almost* provide it: use-cases which have access to a subprocess environment can run "gclient flatten". But yes, better would be to have a library which consumes DEPS files (including over the network, pulling contents from googlesource) and produces a dependency tree. How upset would people be if it were in Go, since we try not to write any new code in Python? (I know, doing it in Go is a long shot anyway, since DEPS files are literally python, but I'd like to discuss it anyway.)
,
Nov 15 2017
From what I know, the apps I mentioned above are all in python. So a python library would be appreciated!
,
Nov 16 2017
Put differently, a Go library would be worse than a command-line interface :).
,
Nov 20 2017
,
Mar 10 2018
Any updates for this bug? recently the DEPS parser for Findit and Predator failed to parse the buildspec DEPS, I can do a hacky fix but think it might be much cleaner that we can have a such a shared and well-maintained lib to parse DEPS file.
,
Mar 12 2018
DEPS files are now parsed via gclient_eval.py, a stand-alone DPES-file-parser in depot_tools. However: 1) it isn't executable, it's just a library. 2) it's in depot_tools, so importing it from elsewhere is non-trivial. 3) we don't guarantee that its API is stable yet.
,
Mar 19 2018
We, the chrome crash team, has a job that exports blame/source info to our frontend http://crash, and our job depends on parsing DEPS file. Our code is in C++ but we could use a python parser.
,
Mar 19 2018
,
Jun 27 2018
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by st...@chromium.org
, Nov 13 2017