aboutsummaryrefslogtreecommitdiff
path: root/remote_gcc_build.py
AgeCommit message (Collapse)Author
2017-04-08[toolchain-utils] Fix remaining lint errors in toolchain-utils.Caroline Tice
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>
2016-07-27[crosperf] Clean up exceptions.Caroline Tice
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>
2016-07-25[toolchain-utils] Finish switching utils/ to cros_utils/.Caroline Tice
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>
2016-01-22Fix cros lint errors.Caroline Tice
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>
2015-12-16Run pyformat on all the toolchain-utils files.Luis Lozano
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>
2015-12-15crosperf: RunCommand should return one type of object.Luis Lozano
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>
2014-05-26Rename chromite.buildbot -> chromite.cbuildbot.Don Garrett
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>
2013-09-04Modified the script to use the new internal and external gerrit repositories.Luis Lozano
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>
2013-08-19Fixed typo in last commitLuis Lozano
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>
2013-08-01Changes to allow getting images from the remote bot when the testing failsLuis Lozano
- 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>
2013-05-23remote_gcc_build: fix branch setting error.Yunlian Jiang
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>
2013-04-10Remove the extra -g in command line if no patchYunlian Jiang
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>
2013-04-10Remove the branch prefix to make it workYunlian Jiang
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>
2013-04-03Change default value of branch, fix a branch name.Yunlian Jiang
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>
2013-04-03Fixed a couple of minor issues in the handling of "patch" and "branch" optionsLuis Lozano
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>
2013-04-03Handle the case where the branch is not there yet.Yunlian Jiang
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>
2013-03-26Fixed small issue in replacing sysroot functionality.Luis Lozano
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>
2013-03-15Cleaned up directory after copy of tools from perforce directoryLuis Lozano
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>
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-05 06:30:46 +0000