tools/android/checkstyle/checkstyle.py throwing exception in presubmit bot |
|||
Issue descriptionOn this CL: https://chromium-review.googlesource.com/c/517913/ https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Ftryserver.chromium.linux%2Fchromium_presubmit%2F451119%2F%2B%2Frecipes%2Fsteps%2Fpresubmit%2F0%2Fstdout Error: Could not find or load main class com.puppycrawl.tools.checkstyle.Main
,
May 30 2017
,
May 31 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/da2bfbbb47cacfdf9487c9e14f4afef0945729b7 commit da2bfbbb47cacfdf9487c9e14f4afef0945729b7 Author: John Budorick <jbudorick@chromium.org> Date: Wed May 31 01:49:12 2017 [android] Revert hooks_os change for checkstyle. Checkstyle gets run on Java changes on chromium_presubmit, which uses a linux checkout. Bug: 727875 Change-Id: I640418439735e91fbf5b18b3d5f5a72e5c00648e Reviewed-on: https://chromium-review.googlesource.com/517978 Reviewed-by: Bo Liu <boliu@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#475730} [modify] https://crrev.com/da2bfbbb47cacfdf9487c9e14f4afef0945729b7/DEPS
,
Oct 9 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/bbdcc4931afb03517d3c7f6e6192eafc244a350c commit bbdcc4931afb03517d3c7f6e6192eafc244a350c Author: John Budorick <jbudorick@chromium.org> Date: Mon Oct 09 18:57:09 2017 Restrict checkstyle JAR download to android and linux. Bug: 727875 , 772741 Change-Id: Iac36fc9426b2926c78334bedb8bfa0aaec57ab5a Reviewed-on: https://chromium-review.googlesource.com/707300 Reviewed-by: Bo <boliu@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org> Reviewed-by: Nico Weber <thakis@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#507432} [modify] https://crrev.com/bbdcc4931afb03517d3c7f6e6192eafc244a350c/DEPS
,
Oct 9 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by jbudorick@chromium.org
, May 30 2017