- 28 Feb, 2020 1 commit
-
-
Josip Sokcevic authored
Change-Id: I9607035c8ee5e5f4423b7526c67f384cfebb2f1d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2079708 Commit-Queue: Josip Sokcevic <sokcevic@google.com> Auto-Submit: Josip Sokcevic <sokcevic@google.com> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org>
-
- 27 Feb, 2020 3 commits
-
-
Edward Lemur authored
Bug: 1042324 Change-Id: Iae90e632e9c1b2c798575ab940d82a145b47204a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2070681 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Josip Sokcevic <sokcevic@google.com>
-
Edward Lemur authored
Remove input and output stream, that always pointed to sys.stdin and sys.stdout. Add tests for DoPostUploadExecuter. Bug: 1042324 Change-Id: Ib5d092706a242ae97ec877af5dbceef6db7cbb73 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2079411Reviewed-by:
Josip Sokcevic <sokcevic@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
Bug: 1056309, 1042324 Change-Id: I69815044c199f2321a66760bbdfe251ed916154a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2076037Reviewed-by:
Josip Sokcevic <sokcevic@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 25 Feb, 2020 3 commits
-
-
Edward Lemur authored
PRESUBMIT.py scripts are executed in presubmit_support.py using exec(). Since PRESUBMIT.py scripts might not be yet compatible with python 3, we have to execute presubmit_support.py using python 2. git_cl.py imports presubmit_support.py, and executes presubmit checks using presubmit_support as a module. This forces git_cl.py to be executed using python 2 to maintain compatibility for PRESUBMIT.py scripts. This change allows git_cl.py to be executed using python 3, while presubmit_support.py is executed using python 2. Similar changes for post-submit hooks and git-cl try masters will follow. Bug: 1042324 Change-Id: Ic3bb1c2985459baf6aa04d0cc65017a1c2578153 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2068945Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
Change-Id: Ib5625ae9d052eca9d6c6b64f12f64e6697aef7a8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2072874Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
Bug: 1051631 Change-Id: I5da32978bbed16bcc9854f15db91a901f5892cda Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2067402Reviewed-by:
Josip Sokcevic <sokcevic@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 24 Feb, 2020 1 commit
-
-
Edward Lemur authored
In Python 3 the semantics of `raise StopIteration` inside a generator function changed. Bug: 1002209 Change-Id: I51222a5006c4024b3a6a06d344423ee36870825a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2071056Reviewed-by:
Josip Sokcevic <sokcevic@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 22 Feb, 2020 1 commit
-
-
Edward Lemur authored
Useful when passing information to a subcommand via a temporary file. Bug: 1051631 Change-Id: I0b8deda921effd24a37109544e1e7ca22e00ba4e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2068942Reviewed-by:
Josip Sokcevic <sokcevic@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 21 Feb, 2020 2 commits
-
-
Edward Lemur authored
Bug: 1054787, 1051631 Change-Id: Ib36740a3f8158977897431c5b52e5ecbdaeb16b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2068506 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Josip Sokcevic <sokcevic@google.com>
-
Edward Lemur authored
Also, rename --all-files to --all_files to be consistent with other options, and remove reviewers, since it's never set in presubmit_support. Bug: 1042324 Change-Id: I34fa3162c56e39bcdd52ad1e19058357de934f1c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2067351Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 20 Feb, 2020 1 commit
-
-
Edward Lemur authored
Bug: 1051631 Change-Id: I20f68db7dd0a901ff53513998e148cf4ba2c3eaa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2065133Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 19 Feb, 2020 2 commits
-
-
Edward Lesmes authored
Mock GetChangeDetail instead of using self.calls to make the code easier to change. Bug: 1051631 Change-Id: I77361caccaf694644839825d890343864267688f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2062716 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com>
-
Edward Lemur authored
Bug: 1051631 Change-Id: I43460b72dfbc9c8210c2d8fdf5d29e876a5637f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2056648Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 14 Feb, 2020 1 commit
-
-
Edward Lemur authored
scm.GIT functions are tested, and using them makes it easier to mock git calls on git-cl tests. Bug: 1051631 Change-Id: If067aa3f71b7478cafd7985d3020dacc0c6a41c1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2055464Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 13 Feb, 2020 2 commits
-
-
Edward Lemur authored
scm: - Add methods to deal with git configs and tests for them. Will make it easier to mock git config calls as opposed to all git calls. - Add tests to FetchUpstreamTuple git-cl: - Mock out settings.GetChangeRoot() - Use scm.GIT.FetchUpstreamTuple() to reduce code duplication, and mock it out on tests. Bug: 1051631 Change-Id: I1a3220b4c0f3e6221b3c00550259a433c2775798 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2052552 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com>
-
Edward Lemur authored
Bug: 1051631 Change-Id: I0c8222f89477300e6a14296b72ecaa01d65ca32d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2052627Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 06 Feb, 2020 6 commits
-
-
Edward Lemur authored
Is not needed, and it's deprecated on Python 3. Change-Id: I040891ff74708c6606b8c216da3a0bb98d44d977 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2042281 Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Anthony Polito <apolito@google.com> Reviewed-by:
Anthony Polito <apolito@google.com>
-
Edward Lemur authored
- Merge GetDescription and FetchDescription, and remove unused `force` argument. - Merge UpdateDescription and UpdateDescriptionRemote. Bug: 1042324 Change-Id: Ia7a1b0aa1ea12a95bb6a19d9d3c9dd6aeb5bd2b8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2039968 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com>
-
Edward Lemur authored
Unused after https://skia-review.googlesource.com/c/skia/+/265209 Bug: 1042324 Change-Id: I43a04d659e4d851e7b954cdd7dc0525490e3244c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2039969 Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
The `author` parameter is never set. The `local_description` parameter was only set with the purpose of getting the local description, so separate it into its own method and call that instead. Use the absolute root when getting the list of modified files instead. Bug: 1042324 Change-Id: I8505172edaacee0656dda98ebbd5f8ebfacb3cef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2038050 Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
Change-Id: I94922e1abab8acac132e16d255e1a9b95771ea5c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2040572Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
This is a reland of 1c8026d2 Original change's description: > presubmit_support: Add --all-files options. > > Will be set when --all is passed to git-cl presubmit to execute > presubmit checks over all files on a repo. > > Bug: 1042324 > Change-Id: I2ebb7c3ec633f9d3a3774314799aa056bc2919c8 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2003071 > Reviewed-by: Anthony Polito <apolito@google.com> > Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Bug: 1042324 Change-Id: I452d98972f9b2c6d0a212029bac5aa6af57dce40 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2038055Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 05 Feb, 2020 1 commit
-
-
Edward Lesmes authored
This reverts commit 1c8026d2. Reason for revert: Description should be set from gerrit before parsing the changes. Original change's description: > presubmit_support: Add --all-files options. > > Will be set when --all is passed to git-cl presubmit to execute > presubmit checks over all files on a repo. > > Bug: 1042324 > Change-Id: I2ebb7c3ec633f9d3a3774314799aa056bc2919c8 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2003071 > Reviewed-by: Anthony Polito <apolito@google.com> > Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> TBR=ehmaldonado@chromium.org,apolito@google.com,infra-scoped@luci-project-accounts.iam.gserviceaccount.com Change-Id: I8c455a269cdd6eb3dea9379faf351a04644bd23f No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1042324, 1048885 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2038052Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 04 Feb, 2020 1 commit
-
-
Edward Lemur authored
Will be set when --all is passed to git-cl presubmit to execute presubmit checks over all files on a repo. Bug: 1042324 Change-Id: I2ebb7c3ec633f9d3a3774314799aa056bc2919c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2003071Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 03 Feb, 2020 2 commits
-
-
Edward Lemur authored
Mock time_time so that time.time is not called from ThreadPoolTest tests, so that the execution time doesn't change between executions. Bug: 1047631 Change-Id: I4f4ac30ca6a2ee3be25830c0e17a9f29a9c0c999 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2036477Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
Needed by Skia's PostUploadHooks, so that presubmit_support doesn't depend on git-cl internal classes. Bug: 1042324 Change-Id: If8f4e6ea035197667cd9705a4274590c4d11511e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2007670Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 31 Jan, 2020 1 commit
-
-
Josip authored
Change-Id: I0a4d16fef3aed6bba0c6015fe861a2f11558e11b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2029114 Commit-Queue: Josip Sokcevic <sokcevic@google.com> Reviewed-by:
Anthony Polito <apolito@google.com>
-
- 30 Jan, 2020 2 commits
-
-
Florian Mayer authored
Without this change, having an SSH remote configured makes `git cl archive` crash: Traceback (most recent call last): File "/usr/local/google/home/fmayer/depot_tools/metrics.py", line 267, in print_notice_and_exit yield File "/usr/local/google/home/fmayer/depot_tools/git_cl.py", line 5553, in <module> sys.exit(main(sys.argv[1:])) File "/usr/local/google/home/fmayer/depot_tools/git_cl.py", line 5535, in main return dispatcher.execute(OptionParser(), argv) File "/usr/local/google/home/fmayer/depot_tools/subcommand.py", line 252, in execute return command(parser, args[1:]) File "/usr/local/google/home/fmayer/depot_tools/metrics.py", line 252, in _inner return self._collect_metrics(func, command_name, *args, **kwargs) File "/usr/local/google/home/fmayer/depot_tools/metrics.py", line 232, in _collect_metrics gclient_utils.reraise(exception[0], exception[1], exception[2]) File "/usr/local/google/home/fmayer/depot_tools/metrics.py", line 205, in _collect_metrics result = func(*args, **kwargs) File "/usr/local/google/home/fmayer/depot_tools/git_cl.py", line 3786, in CMDarchive for cl, status in statuses File "/usr/local/google/home/fmayer/depot_tools/git_cl.py", line 3599, in get_cl_statuses cl.EnsureAuthenticated(force=False, refresh=True) File "/usr/local/google/home/fmayer/depot_tools/git_cl.py", line 1757, in EnsureAuthenticated if urllib.parse.urlparse(self.GetRemoteUrl()).scheme != 'https': File "/usr/lib/python2.7/urlparse.py", line 143, in urlparse tuple = urlsplit(url, scheme, allow_fragments) File "/usr/lib/python2.7/urlparse.py", line 201, in urlsplit i = url.find(':') AttributeError: 'NoneType' object has no attribute 'find' Change-Id: I6f794f11d3ed5e3dbedfe278ded660fca32abda6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2030505 Commit-Queue: Florian Mayer <fmayer@google.com> Reviewed-by:
Anthony Polito <apolito@google.com>
-
Josip authored
git cl upload gets a new flag --edit-description, which allows user to edit git commit message. It only works when updating existing CL. Bug: 739928 Change-Id: Ia501dc31f32e1887a937d5df39aed03745376827 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2020591Reviewed-by:
Andrii Shyshkalov <tandrii@google.com> Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Josip Sokcevic <sokcevic@google.com>
-
- 24 Jan, 2020 1 commit
-
-
Anthony Polito authored
Bug: 1043877 Change-Id: I5363dc1ad7f6b7e0ce9ce9885c86e357c3c65339 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2016162Reviewed-by:
Andrii Shyshkalov <tandrii@google.com> Commit-Queue: Anthony Polito <apolito@google.com>
-
- 17 Jan, 2020 2 commits
-
-
Edward Lemur authored
Will be used by presubmit_support to run checks over all files on a repo. Bug: 1042324 Change-Id: I872b4eb7f287f3a4b14d753cad73d0c5d7beb00c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2002961Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
Change-Id: I54f0fe3a2257166cba2a69659e5128e4ac9310bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2001707Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 15 Jan, 2020 1 commit
-
-
Raul Tambre authored
Example error during "git cl upload": Traceback (most recent call last): File "C:\Google\depot_tools\presubmit_support.py", line 220, in CallCommand returncode, stdout = self._RunWithTimeout(cmd, test.stdin, test.kwargs) File "C:\Google\depot_tools\presubmit_support.py", line 204, in _RunWithTimeout p = subprocess.Popen(cmd, **kwargs) File "C:\Google\depot_tools\subprocess2.py", line 143, in __init__ super(Popen, self).__init__(args, **kwargs) File "C:\Google\depot_tools\bootstrap-3_8_0b1_chromium_1_bin\python\bin\Lib\subprocess.py", line 390, in __init__ errread, errwrite) File "C:\Google\depot_tools\bootstrap-3_8_0b1_chromium_1_bin\python\bin\Lib\subprocess.py", line 640, in _execute_child startupinfo) TypeError: environment can only contain strings Bug: 984182 Change-Id: Id996d73a80845aaeaa061107ed123627091cb600 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1815872 Commit-Queue: Raul Tambre <raul@tambre.ee> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Auto-Submit: Raul Tambre <raul@tambre.ee>
-
- 14 Jan, 2020 1 commit
-
-
Edward Lemur authored
Change-Id: I730f364f30b0d211c2db6da0d6bb4f5ff3c948da Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1999280Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 13 Jan, 2020 3 commits
-
-
Edward Lemur authored
This is a reland of d3bfd23c Don't modify cwd for paths other than depot_tools. Original change's description: > presubmit_canned_checks: Run pylint on parent directory for depot_tools. > > On Windows, scripts on the current directory take precedence over PATH. > When pylint.bat calls vpython, it executes the vpython of the depot_tools > under test instead of the one in the bot. > As a workaround, run the tests from the parent directory instead. > > Change-Id: I7fa3f5b268df516194063ff39907ada18f7ef544 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1994216 > Reviewed-by: Anthony Polito <apolito@google.com> > Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> No-Presubmit: True Change-Id: I4236ec17b2642ed7f53844af532ec1f5234b80d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1995783 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com>
-
Dean Berris authored
This reverts commit d3bfd23c. Reason for revert: Breaks non-depot_tools based presubmits that use canned checks that expect the presubmit to be run in the sub-repository (for example, in Catapult where it's hosted in third_party/catapult instead of being directly in chromium). Original change's description: > presubmit_canned_checks: Run pylint on parent directory for depot_tools. > > On Windows, scripts on the current directory take precedence over PATH. > When pylint.bat calls vpython, it executes the vpython of the depot_tools > under test instead of the one in the bot. > As a workaround, run the tests from the parent directory instead. > > Change-Id: I7fa3f5b268df516194063ff39907ada18f7ef544 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1994216 > Reviewed-by: Anthony Polito <apolito@google.com> > Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> TBR=ehmaldonado@chromium.org,apolito@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: If989527a53be90e1ae32cb3fa30f5e4777824480 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1994753Reviewed-by:
Dean Berris <dberris@chromium.org> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Dean Berris <dberris@chromium.org>
-
Jamie Madill authored
Re-land fixes format always saying the folder was dirty. The --full option would skip setting the return value of the format command when used with --dry-run. This in turn would cause the presubmit check to always pass when --full was enabled in a repo by default. Was discovered in the ANGLE repository. Bug: angleproject:4241 Change-Id: I986646336d1857af2826863ef4becbf2b9ee3049 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1993910 Commit-Queue: Jamie Madill <jmadill@chromium.org> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org>
-
- 10 Jan, 2020 2 commits
-
-
Edward Lemur authored
On Windows, scripts on the current directory take precedence over PATH. When pylint.bat calls vpython, it executes the vpython of the depot_tools under test instead of the one in the bot. As a workaround, run the tests from the parent directory instead. Change-Id: I7fa3f5b268df516194063ff39907ada18f7ef544 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1994216Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
Change-Id: I8fd0034f6a6d7623792620f92208b25961fa174e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1990142Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-