- 17 Feb, 2021 1 commit
-
-
Edward Lesmes authored
Change-Id: I2263d8c1bc48ba0f64bcd5e665d8eb6ea451f1cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2693916 Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Josip Sokcevic <sokcevic@google.com> Reviewed-by:
Gavin Mak <gavinmak@google.com> Reviewed-by:
Josip Sokcevic <sokcevic@google.com>
-
- 03 Feb, 2021 1 commit
-
-
Edward Lesmes authored
Change-Id: I7aab38306e0699de465d0d2ab83dd37c4a099eee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2669149 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Josip Sokcevic <sokcevic@google.com>
-
- 02 Feb, 2021 1 commit
-
-
Edward Lesmes authored
Change-Id: I9dc9cda5a30833938fe13476deb76f4ed5770eab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2666668 Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Gavin Mak <gavinmak@google.com> Reviewed-by:
Gavin Mak <gavinmak@google.com>
-
- 30 Nov, 2020 1 commit
-
-
Sigurdur Asgeirsson authored
Without this change, git cl split will create a separate CL for every file that has per-file owners rules. This is almost always the wrong thing to do. With this change, it will at least aggregate all such files to their parent directory. This may under-split, but is perhaps nearer the mark. Change-Id: Iea7997b36d053713fa00f1261ea032e09d9c2818 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2555729 Auto-Submit: Sigurður Ásgeirsson <siggi@chromium.org> Commit-Queue: Dirk Pranke <dpranke@google.com> Reviewed-by:
Dirk Pranke <dpranke@google.com>
-
- 26 Aug, 2020 1 commit
-
-
Anthony Polito authored
Bug: 1120999 Change-Id: I8190a9d37f8dfcb59d9d51966fc67719592c2c8a R=sokcevic@google.com Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2378453Reviewed-by:
Josip Sokcevic <sokcevic@google.com> Commit-Queue: Anthony Polito <apolito@google.com>
-
- 31 Mar, 2020 1 commit
-
-
Edward Lesmes authored
Change-Id: I6ca167e34829cd7694372923917446c3843c879f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2129146Reviewed-by:
Josip Sokcevic <sokcevic@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 23 Mar, 2020 1 commit
-
-
Edward Lesmes authored
gclient_utils.AskForData will use raw_input on Python 2 and input on Python 3. Bug: 1063976 Change-Id: I93c059c4e4454d01787716b5a0a2641ad5253f53 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2116370 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com> Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org>
-
- 12 Mar, 2020 1 commit
-
-
Edward Lemur authored
Bug: 1042324 Change-Id: I72db082f086f69bf49256d0613c39dc92ae0a07f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2101471 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com>
-
- 09 Mar, 2020 1 commit
-
-
Edward Lemur authored
Also refactor code to eliminate multiple calls to _create_description_from_log. Change-Id: I113134fbd90f396bdb6d561ed0369ea5ee9c78ad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2090448Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 04 Mar, 2020 1 commit
-
-
Edward Lemur authored
file does not exist on Python 3. Change-Id: I9392fd797b730784cff4d1144c5a3ab1027c7dd8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2086493Reviewed-by:
Josip Sokcevic <sokcevic@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 29 Feb, 2020 1 commit
-
-
Edward Lemur authored
This reverts commit 68409634. Reason for revert: Breaks git-cl split. https://bugs.chromium.org/p/chromium/issues/detail?id=1054888 Original change's description: > Improve git cl split > > This CL changes the behavior of `git cl split` to split the change > by the size of the resulting CLs. For now, this is based on the number > of bytes changed, and not by the number of changed lines. Depending > on the shape of change, this may still produce more CLs than expected > (and possibly more than before). > > A future change will switch the split to be based on the number > of affected lines, and also introduce a mode to base the split > on the number of affected files. > > Bug: 998922 > Change-Id: I49f868972a61b89b426ef9e2ceedc733eacb4350 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1778744 > Commit-Queue: Yannic Bonenberger <yannic.bonenberger@gmail.com> > Reviewed-by: Dirk Pranke <dpranke@chromium.org> TBR=fdoray@chromium.org,dpranke@chromium.org,yannic.bonenberger@gmail.com,infra-scoped@luci-project-accounts.iam.gserviceaccount.com Bug: 998922 Change-Id: I466e1245d5f1c934443d850c48c42d3e694e71c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2080205Reviewed-by:
Anthony Polito <apolito@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 Lesmes authored
Changelist.GetChange signature was changed in https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2038050 Bug: 1042324, 1054880 Change-Id: I6ba63e7f2f6c13bd8e8bef4e76cd66ee0525bba2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2067623Reviewed-by:
Josip Sokcevic <sokcevic@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Edward Lesmes authored
Bug: 998922, 1054888 Change-Id: I4ae14700cbd6eb483f318f6ed9f4d1f30b58153e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2068492Reviewed-by:
Josip Sokcevic <sokcevic@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 23 Jan, 2020 1 commit
-
-
Yannic Bonenberger authored
This CL changes the behavior of `git cl split` to split the change by the size of the resulting CLs. For now, this is based on the number of bytes changed, and not by the number of changed lines. Depending on the shape of change, this may still produce more CLs than expected (and possibly more than before). A future change will switch the split to be based on the number of affected lines, and also introduce a mode to base the split on the number of affected files. Bug: 998922 Change-Id: I49f868972a61b89b426ef9e2ceedc733eacb4350 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1778744 Commit-Queue: Yannic Bonenberger <yannic.bonenberger@gmail.com> Reviewed-by:
Dirk Pranke <dpranke@chromium.org>
-
- 11 Oct, 2019 1 commit
-
-
Marc-Antoine Ruel authored
Ran: vi $(git grep --name-only iteritems | grep -v third_party) vi $(git grep --name-only itervalues | grep -v third_party) vi $(git grep --name-only 'print ' | grep -v third_party) and edited the files quickly with adhoc macros. Then ran in recipes/: ./recipes.py test train There was only a small subset of files that had been updated to use six.iteritems() and six.itervalues(). Since the dataset size that is being used in gclient is small (pretty much always below 200 items), it's better to just switch to .items() right away and take the temporary performance hit, so that we don't need to come back to rewrite the code. Recipe-Nontrivial-Roll: build Bug: 984182 Change-Id: I5faf11486b66b0d73c9098ab0f2ce1b15a45c53e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1854900 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Auto-Submit: Marc-Antoine Ruel <maruel@chromium.org>
-
- 06 May, 2019 1 commit
-
-
Raul Tambre authored
Ran "2to3 -w -n -f print ./" and manually added imports. Ran "^\s*print " and "\s+print " to find batch/shell scripts, comments and the like with embedded code, and updated them manually. Also manually added imports to files, which used print as a function, but were missing the import. The scripts still work with Python 2. There are no intended behaviour changes. Bug: 942522 Change-Id: Id777e4d4df4adcdfdab1b18bde89f235ef491b9f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1595684Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Dirk Pranke <dpranke@chromium.org> Auto-Submit: Raul Tambre <raul@tambre.ee>
-
- 14 Feb, 2019 1 commit
-
-
Takuto Ikuta authored
With default True. This CL was uploaded by git cl split. R=jochen@chromium.org Bug: 927178 Change-Id: Ie0a68ed7ca381e1bb33341badca153bf409bb376 Reviewed-on: https://chromium-review.googlesource.com/c/1470402Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
-
- 07 Sep, 2018 1 commit
-
-
Stephen Martinis authored
Bug: 878117 Change-Id: Ifb8af6bfcc6467ad9b79cf6292ebb8afc8f7e949 Reviewed-on: https://chromium-review.googlesource.com/1213588Reviewed-by:
John Budorick <jbudorick@chromium.org> Reviewed-by:
Ryan Tseng <hinoka@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org>
-
- 29 Aug, 2018 1 commit
-
-
Stephen Martinis authored
`git cl split` currently runs a cq dry run for every uploaded CL. This has overloaded our infrastructure a few times in the past. This CL changes the command to not dry run by default, and adds a flag to enable this. The flag has a warning about doing this, and tells the user to email infra-dev@chromium.org if they're going to be generating >~10 CLs. Bug: 878117 Change-Id: Ic865c09b188b8d4f202785f4763f7b7b8910c9cf Reviewed-on: https://chromium-review.googlesource.com/1191927Reviewed-by:
Andrii Shyshkalov <tandrii@chromium.org> Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org>
-
- 13 Dec, 2017 1 commit
-
-
Chris Watkins authored
git cl split previously had no way to find out what it would do before it went ahead and uploaded CLs. Now you can use --dry-run to get it to print info about each CL it would upload. Change-Id: If2ac74a89b0a8b30d86819e16ece46f487b1e7c7 Reviewed-on: https://chromium-review.googlesource.com/821336Reviewed-by:
Aaron Gable <agable@chromium.org> Commit-Queue: Chris Watkins <watk@chromium.org>
-
- 09 Nov, 2017 1 commit
-
-
Gabriel Charette authored
R=iannucci@chromium.org Bug: --send-email is not a command Change-Id: Ia6a034204510322d674e0a45a2ed8a43dbba242c Reviewed-on: https://chromium-review.googlesource.com/757131 Commit-Queue: Gabriel Charette <gab@chromium.org> Reviewed-by:
Robbie Iannucci <iannucci@chromium.org>
-
- 14 Jul, 2017 1 commit
-
-
Aaron Gable authored
The actual flag is --send-mail. Also includes a fix to publish these changes if a comment is included. Bug: 740950 Change-Id: I38ca0a35c1364c8364eb0ef301137c04daede40b Reviewed-on: https://chromium-review.googlesource.com/572033Reviewed-by:
Andrii Shyshkalov <tandrii@chromium.org> Commit-Queue: Aaron Gable <agable@chromium.org>
-
- 31 May, 2017 1 commit
-
-
Francois Doray authored
`git cl split` splits a branch into smaller branches and uploads CLs. Change-Id: Ic41cdabdd6241008ff48766e31a8d9d07995f2b0 Reviewed-on: https://chromium-review.googlesource.com/439706 Commit-Queue: Francois Pierre Doray <fdoray@chromium.org> Reviewed-by:
Aaron Gable <agable@chromium.org>
-