Age | Commit message (Collapse) | Author |
|
This patch moves all scripts that are not used any more to deprecated
directory. We do not need to migrated those scripts to python 3.
BUG=chromium:1011676, chromium:1051236
TEST=None
Change-Id: I2caac6204c82dcd21b2a121875a2f9851eaca322
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2051285
Commit-Queue: Zhizhou Yang <zhizhouy@google.com>
Tested-by: Zhizhou Yang <zhizhouy@google.com>
Auto-Submit: Zhizhou Yang <zhizhouy@google.com>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
In addition to fixing the lint errors, this also fixes the Python
formatting issues (ran tc_pyformat on nearly all the files).
BUG=chromium:570450
TEST=Ran all crosperf & bisect tool unit tests.
Ran afe_lock_machine.py (check machine status)
Ran full crosperf test (octane, speedometer, BootPerf) on alex.
Change-Id: Ic86f9192801ac67769f3de30f1c5f0d203ce0831
Reviewed-on: https://chromium-review.googlesource.com/471886
Commit-Ready: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Found all instances of 'raise Exception' and changed them to raise
something slightly more specific. Changed a few 'except Exception'
statements appropriately as well.
BUG=chromium:566255
TEST=None
Change-Id: If2666bd55838342bc71431e5e6ab2157c2c9e125
Reviewed-on: https://chrome-internal-review.googlesource.com/270731
Commit-Ready: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Reviewed-by: Yunlian Jiang <yunlian@google.com>
|
|
This CL finishes switching the subdirectory from 'utils' to
'cros_utils'. It changes all the remaining import statements to
use 'cros_utils'; it removes the 'cros_utils' symlink, and it
renames the 'utils' subdirectory to 'cros_utils'.
BUG=chromium:568195
TEST=ran crosperf & binary search tool unittests.
Change-Id: I7427f8bfb2ddac3a4b6108e46782039059684382
Reviewed-on: https://chrome-internal-review.googlesource.com/270396
Commit-Ready: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Reviewed-by: Cassidy Burden <cburden@google.com>
Reviewed-by: Luis Lozano <llozano@chromium.org>
|
|
Also move deprecated scripts to the 'deprecated' directory.
BUG=chromiumos:570464
TEST=tested scripts to make sure they still work.
Change-Id: I3442a86d898104591233a0849ea0bafb52ecf1f7
Reviewed-on: https://chrome-internal-review.googlesource.com/244221
Commit-Ready: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Reviewed-by: Yunlian Jiang <yunlian@google.com>
|
|
This gets rid of a lot of lint issues.
Ran by doing this:
for f in *.py; do echo -n "$f " ; if [ -x $f ]; then pyformat -i
--remove_trailing_comma --yapf --force_quote_type=double $f ; else
pyformat -i --remove_shebang --remove_trailing_comma --yapf
--force_quote_type=double $f ; fi ; done
BUG=chromium:567921
TEST=Ran simple crosperf run.
Change-Id: I59778835fdaa5f706d2e1765924389f9e97433d1
Reviewed-on: https://chrome-internal-review.googlesource.com/242031
Reviewed-by: Luis Lozano <llozano@chromium.org>
Commit-Queue: Luis Lozano <llozano@chromium.org>
Tested-by: Luis Lozano <llozano@chromium.org>
Reviewed-by: Yunlian Jiang <yunlian@google.com>
|
|
Cleaned up the interfaces for the RunCommand routines.
These were returning different types (int or tuple) depending on
the value of the return_ouput parameter.
Returning different unrelated types from a routine is bad practice.
Linter complains about this with several warnings like this:
"Attempting to unpack a non-sequence defined at line XY of
utils.command_executer"
BUG=chromium:566256
TEST=ran crosperf with a example experiment file
Ran run_tests.
Change-Id: Ibb83ab9322c87558077fc4937ef5c0686bbe5417
Reviewed-on: https://chrome-internal-review.googlesource.com/241459
Commit-Ready: Luis Lozano <llozano@chromium.org>
Tested-by: Luis Lozano <llozano@chromium.org>
Reviewed-by: Han Shen <shenhan@google.com>
|
|
BUG=None
TEST=None
CQ-DEPEND=CL:199664
Change-Id: I6b2b119960cf4368a5cbc594523b4d44407e91f9
Reviewed-on: https://chrome-internal-review.googlesource.com/163786
Reviewed-by: Don Garrett <dgarrett@google.com>
Commit-Queue: Don Garrett <dgarrett@google.com>
Tested-by: Don Garrett <dgarrett@google.com>
|
|
BUG=None
TEST=Minimal testing done. Need to let the nightly testing test this.
Change-Id: I8273d5e90e5fb3ed96d9782cae651d4144f135dc
Reviewed-on: https://chrome-internal-review.googlesource.com/144015
Reviewed-by: Han Shen <shenhan@google.com>
Tested-by: Luis Lozano <llozano@chromium.org>
Commit-Queue: Luis Lozano <llozano@chromium.org>
|
|
BUG=None
TESTS=Did several succesful remote builds
Change-Id: Icff4f20ec44c962f52cd4cec85c3e2ab70eeb4a8
Reviewed-on: https://gerrit-int.chromium.org/43077
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Commit-Queue: Luis Lozano <llozano@chromium.org>
Tested-by: Luis Lozano <llozano@chromium.org>
|
|
- Allow remote_gcc_build to return image even in the case where the build
bot testing has failed. Testing may have failed but an image may have been
generated.
- Move the handling of the manifest repo to a new module.
- Various clean up
BUG=None
TEST=Tested by running with several inputs. Used for vtable verification testing
Change-Id: I1abf01c56d1159dfae76c45e2de4c2a9e385d386
Reviewed-on: https://gerrit-int.chromium.org/42123
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Commit-Queue: Luis Lozano <llozano@chromium.org>
Tested-by: Luis Lozano <llozano@chromium.org>
|
|
This fixes the error when a version is given, the branch
it empty, it invokes the cbuildbot in a wrong way.
BUG=None
TEST=./remote_gcc_build.py -t lumpy-release -c ~/clang -p 51046
-d ~/local_tmp -r --chromeos_version R29-4165.0.0 passes.
Change-Id: I75e24dc1c820583ecb0a10183d9bc86bdec50fbb
Reviewed-on: https://gerrit-int.chromium.org/38595
Commit-Queue: Yunlian Jiang <yunlian@google.com>
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Tested-by: Yunlian Jiang <yunlian@google.com>
|
|
BUG=None
TEST=The script works without patch
Change-Id: I9ee5308ff889699b4b206417ba105db65f0cfb62
Reviewed-on: https://gerrit-int.chromium.org/35440
Reviewed-by: Han Shen <shenhan@google.com>
Tested-by: Yunlian Jiang <yunlian@google.com>
Commit-Queue: Yunlian Jiang <yunlian@google.com>
|
|
BUG=None
TEST=run remote_gcc_build.py with master branch and a source gcc dir.
it works.
Change-Id: I8fb7a91b5725749cc13db860fbcaf0f0fde2ae79
Reviewed-on: https://gerrit-int.chromium.org/35073
Reviewed-by: Han Shen <shenhan@google.com>
Tested-by: Yunlian Jiang <yunlian@google.com>
Commit-Queue: Yunlian Jiang <yunlian@google.com>
|
|
BUG=None
TEST=None
Change-Id: I7b4966580493544f1410f2c754e7a76cbfd04635
Reviewed-on: https://gerrit-int.chromium.org/35018
Reviewed-by: Luis Lozano <llozano@chromium.org>
Commit-Queue: Yunlian Jiang <yunlian@google.com>
Tested-by: Yunlian Jiang <yunlian@google.com>
|
|
BUG=None
TEST=Verified the options work as expected
Change-Id: Ib3e41c6499d5102d8f44c16bafb1b1eafc8184f1
Reviewed-on: https://gerrit-int.chromium.org/34996
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Commit-Queue: Luis Lozano <llozano@chromium.org>
Tested-by: Luis Lozano <llozano@chromium.org>
|
|
BUG=None
TEST=None
Change-Id: I3e7e92d049d6587b0695e8b3f2a525eb063dbc5f
Reviewed-on: https://gerrit-int.chromium.org/35005
Reviewed-by: Luis Lozano <llozano@chromium.org>
Commit-Queue: Yunlian Jiang <yunlian@google.com>
Tested-by: Yunlian Jiang <yunlian@google.com>
|
|
Need to consider that the board name may contain "-"
BUG=None
TEST=Run the script for several boards and verified the copy was happening correctly.
Change-Id: Ic7e34548d65962da11a4ca1cb575fdfb6769feed
Reviewed-on: https://gerrit-int.chromium.org/34400
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Commit-Queue: Luis Lozano <llozano@chromium.org>
Tested-by: Luis Lozano <llozano@chromium.org>
|
|
Got rid of stale copies of some tools like "crosperf" and moved all
files under v14 directory (that came from perforce) into the top
directory.
BUG=None
TEST=None
Change-Id: I408d17a36ceb00e74db71403d2351fd466a14f8e
Reviewed-on: https://gerrit-int.chromium.org/33887
Tested-by: Luis Lozano <llozano@chromium.org>
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Commit-Queue: Luis Lozano <llozano@chromium.org>
|