Chromium Code Reviews Chromium Code Reviews
Issue 381043002:
Add a virtualenv-based python bootstrapping service to infra. (Closed)
Base URL: https://chromium.googlesource.com/infra/infra@master| OLD | NEW |
|---|---|
| 1 Contributing to the Chrome infra codebase | 1 Contributing to the Chrome infra codebase |
| 2 ========================================= | 2 ========================================= |
| 3 | 3 |
| 4 This document explains how to contribute to the Chrome infrastructure codebase. | 4 This document explains how to contribute to the Chrome infrastructure codebase. |
| 5 If you want to contribute to the Chromium browser, you're in the wrong place. | 5 If you want to contribute to the Chromium browser, you're in the wrong place. |
| 6 See | 6 See |
| 7 [http://dev.chromium.org/getting-involved](http://dev.chromium.org/getting-invol ved) | 7 [http://dev.chromium.org/getting-involved](http://dev.chromium.org/getting-invol ved) |
| 8 instead. You can find more information on the Chrome infrastructure | 8 instead. You can find more information on the Chrome infrastructure |
| 9 [here](http://dev.chromium.org/infra). | 9 [here](http://dev.chromium.org/infra). |
| 10 | 10 |
| (...skipping 10 matching lines...) Expand all Loading... | |
| 21 somewhere in your path) to run: | 21 somewhere in your path) to run: |
| 22 | 22 |
| 23 mkdir chrome_infra # or whatever name you please | 23 mkdir chrome_infra # or whatever name you please |
| 24 cd chrome_infra | 24 cd chrome_infra |
| 25 gclient config https://chromium.googlesource.com/infra/infra.git | 25 gclient config https://chromium.googlesource.com/infra/infra.git |
| 26 gclient sync | 26 gclient sync |
| 27 | 27 |
| 28 This will check out the base repository (infra/) and its dependencies. | 28 This will check out the base repository (infra/) and its dependencies. |
| 29 | 29 |
| 30 | 30 |
| 31 Bootstrapping Dependencies | |
| 32 -------------------------- | |
| 33 (See `bootstrap/README.md` for more details). | |
| 34 | |
| 35 Manually create a bootstrap virtualenv environment by running: | |
| 36 | |
| 37 `./bootstrap/bootstrap.py --deps_file bootstrap/deps.pst` | |
|
agable
2014/07/10 17:12:56
Same comment as above -- could be python -m.
iannucci
2014/07/11 02:14:44
Nack!
| |
| 38 | |
| 39 This is done for you automatically by `gclient sync` (or `gclient runhooks`). | |
| 40 | |
| 41 | |
| 31 Invoking tools | 42 Invoking tools |
| 32 -------------- | 43 -------------- |
| 33 | 44 |
| 34 Mixing modules and scripts in the same hierarchy can sometimes be a pain in | 45 Mixing modules and scripts in the same hierarchy can sometimes be a pain in |
| 35 Python, because it usually requires updating the Python path. The goal for | 46 Python, because it usually requires updating the Python path. The goal for |
| 36 infra/ is to be able to check out the repository and be able to run code right | 47 infra/ is to be able to check out the repository and be able to run code right |
| 37 away, without setting up anything. The adopted solution is to use __main__.py | 48 away, without setting up anything. The adopted solution is to use __main__.py |
| 38 files everywhere. | 49 files everywhere. |
| 39 | 50 |
| 40 Example: `python -m infra.services.lkgr_finder` will run the lkgr_finder script. | 51 Example: `python -m infra.services.lkgr_finder` will run the lkgr_finder script. |
| (...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 88 is mirrored on | 99 is mirrored on |
| 89 [https://chromium.googlesource.com/infra/third_party/httplib2.git](https://chr omium.googlesource.com/infra/third_party/httplib2.git)) | 100 [https://chromium.googlesource.com/infra/third_party/httplib2.git](https://chr omium.googlesource.com/infra/third_party/httplib2.git)) |
| 90 - add an entry into infra/DEPS pointing to the mirror, at a given revision | 101 - add an entry into infra/DEPS pointing to the mirror, at a given revision |
| 91 (not HEAD). | 102 (not HEAD). |
| 92 - add an 'import' entry into `infra/ext/__init__.py`. This is to enable | 103 - add an 'import' entry into `infra/ext/__init__.py`. This is to enable |
| 93 autocompletion for tools like pylint and jedi. | 104 autocompletion for tools like pylint and jedi. |
| 94 | 105 |
| 95 The two first steps can only be performed by a limited set of people. Send an | 106 The two first steps can only be performed by a limited set of people. Send an |
| 96 email to infra-dev@chromium.org to request that. | 107 email to infra-dev@chromium.org to request that. |
| 97 | 108 |
| OLD | NEW |
