- 30 Jan, 2020 1 commit
-
-
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 4 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>
-
Jamie Madill authored
This reverts commit fc132e61. Reason for revert: Causes all presubmit checks to return dirty format because of a non-zero diff output. Original change's description: > git-cl: Fix format --dry-run not working with --full. > > 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: Ie6cb423a6818c1e26781d77938a0dd22c02b4c16 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1987835 > Commit-Queue: Jamie Madill <jmadill@chromium.org> > Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org> > Auto-Submit: Jamie Madill <jmadill@chromium.org> TBR=ehmaldonado@chromium.org,jmadill@chromium.org Change-Id: I0a4c51117ab20606f6dbb1f7a3ba40f87da1e939 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: angleproject:4241 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1993907Reviewed-by:
Jamie Madill <jmadill@chromium.org> Commit-Queue: Jamie Madill <jmadill@chromium.org>
-
Jamie Madill authored
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: Ie6cb423a6818c1e26781d77938a0dd22c02b4c16 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1987835 Commit-Queue: Jamie Madill <jmadill@chromium.org> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Auto-Submit: Jamie Madill <jmadill@chromium.org>
-
- 08 Jan, 2020 2 commits
-
-
Edward Lemur authored
Remove default value from optional arguments, since they are always passed. In particular, the default value for `out` was a text stream (sys.stdout), but the value passed to `out` is always a binary stream, which is confusing. Get rid of the `err` argument, since it is always sys.stderr, and only used for testing. Change-Id: Ia289e9a97b968a0c802fc2f419397c1e494f713c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1986064Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
Change-Id: I544aa85b040544508df8ed16310def982356fbcc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1949417 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com>
-
- 06 Jan, 2020 3 commits
-
-
Edward Lemur authored
Change-Id: I3a188b34ae5f62649108afe08fe0e389a408c2ac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1947933 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com>
-
Edward Lemur authored
Change-Id: I7a434d4420573c6e5fc014455b011984f4d681b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1952159Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Dan Beam authored
Bug: 1028829 Change-Id: Iafc40952e402884639543436281e93942c0257a8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1959813 Auto-Submit: Dan Beam <dbeam@chromium.org> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org>
-
- 19 Dec, 2019 1 commit
-
-
Edward Lesmes authored
Bug: 1024683 Change-Id: I1c30473699c1bd6b198188b53b632e3617fed335 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1929653Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 10 Dec, 2019 1 commit
-
-
Aaron Gable authored
Now that the CQ has been enabled for both the Beta and Stable branches, git-drover should use the commit queue instead of directly submitting the CL. To that end, this CL replaces the usage of `git cl land` with `git cl set-commit`. Change-Id: I0d0c10115acc98eafe0c069825de5499b8ade3e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1960697 Auto-Submit: Aaron Gable <agable@chromium.org> Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Reviewed-by:
Garrett Beaty <gbeaty@chromium.org>
-
- 06 Dec, 2019 1 commit
-
-
Jeffrey Yasskin authored
BUG=1023031 Change-Id: I4bf3e33932af40600646f070f057a7c8c0661f33 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1954624 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Auto-Submit: Jeffrey Yasskin <jyasskin@chromium.org>
-
- 04 Dec, 2019 2 commits
-
-
Kevin Marshall authored
Adds error handling logic for pre-existing tags (which can occur if "archive" is CTRL-C aborted midway through) and for undeletable branches (which can happen if they are currently checked out in a working dir). Change-Id: I27b6da9f5860c307f49cbeabb1b0ccf9cfb28eb6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1930023 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Auto-Submit: Kevin Marshall <kmarshall@chromium.org>
-
Edward Lemur authored
Change-Id: Ife5826fc824f1304bc1a8ae09c0a659583f0ffb1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1949408 Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 02 Dec, 2019 2 commits
-
-
Anthony Polito authored
Bug: 896110 Change-Id: Ibaf36df48d9ea25e8ee633ca505fa7f7c2063df4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1944639Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Anthony Polito <apolito@google.com>
-
Brian Sheedy authored
Switches the yapfignore implementation in git_cl to use fnmatch instead of glob. This ends up bringing the .yapfignore file parsing in line with yapf's actual implementation, namely allowing uses such as '*pb2.py' instead of having to list out each directory manually. Bug: 1027953 Change-Id: Ibb1cb4252c546de6f1b1af720c4c29ffd0f7be0c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1938026 Commit-Queue: Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Quinten Yearsley <qyearsley@chromium.org>
-
- 26 Nov, 2019 1 commit
-
-
Edward Lemur authored
git hyper_blame might use a subprocess' stdin for its stdout, which is opened to accept byte input. The text must be encoded before printing to stdout to avoid unicode errors. Bug: 1028709 Change-Id: If2a270a7f3f69a818d367616f6732245de364db9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1937500Reviewed-by:
Andy Perelson <ajp@chromium.org> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 25 Nov, 2019 2 commits
-
-
Edward Lemur authored
Bug: 1026371 Change-Id: Iddef64ac2d3a9f97fdb0540a048dfb9dc5679b5c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1925407 Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
Bug: 1009816 Change-Id: I2fac9f03b6dd275563375d8ac4d1e71b5188e468 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1925325Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 21 Nov, 2019 3 commits
-
-
Edward Lemur authored
Bug: 1024683 Change-Id: I01245d34ad1fa1142ce726c80bf5b61567bafe35 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1925038Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
Bug: 1002209 Change-Id: I64aecc5e06be13237b73ae568a1beae046ba4962 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1925324Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lemur authored
Bug: 1009809 Change-Id: Ia02460a24f4ac048bd7f0072e70856f9ee22a997 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1925314 Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 19 Nov, 2019 1 commit
-
-
Edward Lemur authored
Running gclient_smoketests times out on windows when using git daemon, so use a local directory as remote instead. Bug: 1024683 Change-Id: I6ca506d74de463d914317f176eefbe74996298c5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1879723 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com>
-
- 15 Nov, 2019 3 commits
-
-
Edward Lemur authored
Useful when runnig the same test both on Python 2 and Python 3. Change-Id: I6074e4cbd64318985cf438a2d491182b7b9ccc7b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1913259Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Anthony Polito authored
Also add a better warning if 'gclient sync -D' fails due to a modified file. I've tested this code via deleting a DEP and running gclient sync -D. Bug: 981149 Change-Id: I97035ac238d163ccb1684c3ee423c223ed0f6299 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1891830Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Anthony Polito <apolito@google.com>
-
Anthony Polito authored
Bug: 1025293 Change-Id: I41ca2fba3855ecad3036e94a3bdc39b399c1a276 Recipe-Nontrivial-Roll: build_limited_scripts_slave Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1918609 Commit-Queue: Anthony Polito <apolito@google.com> Auto-Submit: Anthony Polito <apolito@google.com> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org>
-
- 14 Nov, 2019 1 commit
-
-
Bruce Dawson authored
git cl owners orders owners by score with alphabetization being the tie breaker. This leads to some owners being suggested far more often than others. Adding a tiny amount of randomization to the scoring leads to an even distribution of equally qualified reviewers. Less qualified reviewers will still be sorted into distinct buckets - the randomness is too small to do anything except break ties. The tests were updated so that they can tolerate the randomness, but only for breaking ties. Bug: 1024083 Change-Id: If7d39d1b3bbd980b80b46ab3f62c65215309bdc8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1913642 Commit-Queue: Bruce Dawson <brucedawson@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org>
-
- 12 Nov, 2019 1 commit
-
-
Edward Lemur authored
Bug: 1002209 Change-Id: I90de660afd901e544e5557f3af3a56cef4d6adaf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1891667 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com>
-
- 11 Nov, 2019 1 commit
-
-
Edward Lemur authored
This is a reland of c87ed606 Fix on the reland: Encode change description before writing to temporary file. Original change's description: > git-cl: Fix some python3 compatibility errors. > > Also, fix bug in git cl status where the 'updated' field was used to compare messages, even though > it doesn't exist (see [1]). This CL modifies it to use 'date', which does exist. > > [1] https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#change-message-info > > Bug: 1002209 > Change-Id: I5a5e1193b8502c3ad35d94808ea178cad7f44ac6 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1891259 > Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> > Reviewed-by: Anthony Polito <apolito@google.com> Bug: 1002209 Change-Id: I2f0fc779ea3e361e6d32ee7ce1e53cbddb5cfdea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1898702 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com>
-
- 05 Nov, 2019 1 commit
-
-
danakj authored
There are 3 layers modified, starting from the bottom up: 1. git_cache.py populate Now takes a --no-fetch-tags option. If specified, the cache will not fetch updated tags from the server by passing --no-tags to git fetch. This prevents the server from sending all the tag refs. In chromium this prevents significant time bottlenecks dealing with 10k+ tags. 2. bot_update.py bot_update has to deal with multiple git repos, it has the root repo that is checked out through git-cache, as well as repos synched via DEPS and gclient. The script now takes a --no_fetch_tags option. If specified, the git-cache populate call will include --no-fetch-tags. Otherwise, it won't. So this controls (for chromium) if fetches to the src.git server are passed --no-tags. 3. bot_update/api.py This is the entry point for recipes and also has to deal with multiple git repos. The behaviour at this point is not changed from the default. A |no_fetch_tags| parameter is added to ensure_checkout() that defaults to False. This CL is a refactor with no intended behavior change. The next step will be to change the chromium build repo to pass True for ensure_checkout(no_fetch_tags) on chromium trybots. This represents solution #2 in https://docs.google.com/document/d/1hDIunJjjfpmr50y3YnZ4o3aq1NZF4gJa1TS0p7AIL64/edit# Bug: 1019824 Change-Id: I935430603299daa9e301a95a5184c0ce486fd912 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1894352Reviewed-by:
Aaron Gable <agable@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Commit-Queue: danakj <danakj@chromium.org> Auto-Submit: danakj <danakj@chromium.org>
-
- 04 Nov, 2019 1 commit
-
-
Edward Lesmes authored
This reverts commit c87ed606. Reason for revert: Causes a regressions for CL descriptions on Windows Original change's description: > git-cl: Fix some python3 compatibility errors. > > Also, fix bug in git cl status where the 'updated' field was used to compare messages, even though > it doesn't exist (see [1]). This CL modifies it to use 'date', which does exist. > > [1] https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#change-message-info > > Bug: 1002209 > Change-Id: I5a5e1193b8502c3ad35d94808ea178cad7f44ac6 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1891259 > Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> > Reviewed-by: Anthony Polito <apolito@google.com> TBR=ehmaldonado@chromium.org,apolito@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1002209 Change-Id: I004f202b12c6b99cb6b24cb12a14fba7898569e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1898547Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-