- 25 Nov, 2019 1 commit
-
-
Edward Lemur authored
Using bootstrapped vpython causes problems like [1] when used on platforms it was not compiled for (e.g. Fedora). [1] https://chromium-review.googlesource.com/c/infra/infra/+/1929031 Change-Id: I4e62cd1a4e13eee58143695f54ecf80d3473a862 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1930181Reviewed-by:
Anthony Polito <apolito@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
- 19 Nov, 2019 2 commits
-
-
John Budorick authored
This reverts commit b2bfd55f. Reason for revert: not responsible for crbug.com/1025659 Original change's description: > Revert "depot_tools: Ensure Python is bootstrapped before reading python3_bin_reldir.txt" > > This reverts commit bf26b167. > > Reason for revert: spec revert for https://bugs.chromium.org/p/chromium/issues/detail?id=1025659 > > Original change's description: > > depot_tools: Ensure Python is bootstrapped before reading python3_bin_reldir.txt > > > > Bug: 1017812 > > Change-Id: I7013e5445ff5917fab813a54fed416d9770d2684 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1913269 > > Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> > > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > > TBR=dpranke@chromium.org,ehmaldonado@chromium.org,apolito@google.com > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: 1017812 > Change-Id: I8e10894a2e835c20ece7d555c21f6ce1f76528a5 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1922949 > Reviewed-by: John Budorick <jbudorick@chromium.org> > Commit-Queue: John Budorick <jbudorick@chromium.org> Bug: 1017812 Change-Id: I251c6b1a3e3a7b9ac9d21dabe745466c668a8246 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1924494Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
John Budorick authored
This reverts commit bf26b167. Reason for revert: spec revert for https://bugs.chromium.org/p/chromium/issues/detail?id=1025659 Original change's description: > depot_tools: Ensure Python is bootstrapped before reading python3_bin_reldir.txt > > Bug: 1017812 > Change-Id: I7013e5445ff5917fab813a54fed416d9770d2684 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1913269 > Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> > Reviewed-by: Dirk Pranke <dpranke@chromium.org> TBR=dpranke@chromium.org,ehmaldonado@chromium.org,apolito@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1017812 Change-Id: I8e10894a2e835c20ece7d555c21f6ce1f76528a5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1922949Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org>
-
- 15 Nov, 2019 1 commit
-
-
Edward Lemur authored
Bug: 1017812 Change-Id: I7013e5445ff5917fab813a54fed416d9770d2684 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1913269 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org>
-
- 12 Nov, 2019 1 commit
-
-
Edward Lemur authored
pythond and python3 are still downloaded from cipd, but python3 is no longer shim'd into depot_tools on Linux, so is not available on PATH. Modify PATH in vpython and vpython3 on Linux to point to the python3 download directory, so that the version downloaded from cipd is used. Bug: 1017812 Change-Id: I70bc8e7d460f60dcbfa5e141127edecfc2b44744 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1893982 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Anthony Polito <apolito@google.com> Reviewed-by:
Mike Frysinger <vapier@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org>
-
- 10 May, 2019 1 commit
-
-
Robert Nagy authored
on non-linux systems bash is not the default shell (by default) and lives in a completely different path Change-Id: I6580c776dda1b60b3d5c5f5f1ce4d6694ac45fe9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1605902Reviewed-by:
Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
-
- 21 Feb, 2018 1 commit
-
-
Robert Iannucci authored
This will allow users on unsupported platforms to successfully bypass vpython via the recognized environment variable. R=tandrii@chromium.org, vadimsh@chromium.org Change-Id: Id63019443321d9f63550fc11e7a4ae3f684b17e7 Reviewed-on: https://chromium-review.googlesource.com/927695 Commit-Queue: Robbie Iannucci <iannucci@chromium.org> Reviewed-by:
Andrii Shyshkalov <tandrii@chromium.org>
-
- 28 Jul, 2017 1 commit
-
-
Dan Jacques authored
Have the "update_depot_tools" script also do a CIPD tool sync. This will ensure that users and systems have access to tooling at sync-time rather than just-in-time loading them at execution time. Update the tool boostraps to suppress any sort of syncing logs, if it does happen. This will ensure that users who execute the tools don't see unexpected output. BUG=chromium:748651 TEST=local - Tested on Mac and Windows. Change-Id: I1aad897d885a07beeac40a372a658681720efd2a Reviewed-on: https://chromium-review.googlesource.com/591229 Commit-Queue: Daniel Jacques <dnj@chromium.org> Reviewed-by:
Robbie Iannucci <iannucci@chromium.org>
-
- 26 Jul, 2017 1 commit
-
-
Dirk Pranke authored
This reverts commit 4d92fe43. Reason for revert: This moved the logging from invoking `led` or `vpython` to whenever someone would run something that would end up invoking `update_depot_tools`. It's good that we are calling this when we run update_depot_tools, but we probably should've suppressed the logging there in at least the success case, because now things are even more confusing. See crbug.com/748651. Original change's description: > [bootstraps] Sync at gclient, suppress output. > > Have the "update_depot_tools" script also do a CIPD tool sync. This will > ensure that users and systems have access to tooling at sync-time rather > than just-in-time loading them at execution time. > > Update the tool boostraps to suppress any sort of syncing logic, if it > does happen. This will ensure that users who execute the tools don't se > unexpected output. > > BUG=None > TEST=local > - Tested on Mac and Windows. > > R=dpranke@chromium.org, iannucci@chromium.org > > Change-Id: I8efce8c73cc4e82ffdf5067ba9b917119a81e843 > Reviewed-on: https://chromium-review.googlesource.com/581494 > Commit-Queue: Daniel Jacques <dnj@chromium.org> > Reviewed-by: Robbie Iannucci <iannucci@chromium.org> TBR=iannucci@chromium.org,dpranke@chromium.org,dnj@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: None Change-Id: I2485c9dd2e48a8dbdeebfff5da9d4c708e0edcb7 Reviewed-on: https://chromium-review.googlesource.com/585867Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Dirk Pranke <dpranke@chromium.org>
-
- 24 Jul, 2017 1 commit
-
-
Dan Jacques authored
Have the "update_depot_tools" script also do a CIPD tool sync. This will ensure that users and systems have access to tooling at sync-time rather than just-in-time loading them at execution time. Update the tool boostraps to suppress any sort of syncing logic, if it does happen. This will ensure that users who execute the tools don't se unexpected output. BUG=None TEST=local - Tested on Mac and Windows. R=dpranke@chromium.org, iannucci@chromium.org Change-Id: I8efce8c73cc4e82ffdf5067ba9b917119a81e843 Reviewed-on: https://chromium-review.googlesource.com/581494 Commit-Queue: Daniel Jacques <dnj@chromium.org> Reviewed-by:
Robbie Iannucci <iannucci@chromium.org>
-
- 10 May, 2017 1 commit
-
-
Dan Jacques authored
BUG=None TEST=None R=sergeyberezin@chromium.org Change-Id: I2bb22ad002b82767c6a7efb68412f9c1dbf57c27 Reviewed-on: https://chromium-review.googlesource.com/502228Reviewed-by:
Sergey Berezin <sergeyberezin@chromium.org> Reviewed-by:
Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
-
- 01 May, 2017 1 commit
-
-
Sergey Berezin authored
BUG=717208 R=dnj@chromium.org Change-Id: If937a382be9aa4d8eb5f957386e8b1b28cc1c3ac Reviewed-on: https://chromium-review.googlesource.com/492086Reviewed-by:
Daniel Jacques <dnj@chromium.org> Reviewed-by:
Nodir Turakulov <nodir@chromium.org> Commit-Queue: Sergey Berezin <sergeyberezin@chromium.org>
-