| 2a2d3117e3912743527455ec2b7056eb66215ae1 |
|
15-Nov-2017 |
Igor Murashkin <iam@google.com> |
cpplint: Use upstream cpplint
Also use CPPLINT.cfg instead of passing in extra flags to cpplint.py.
This unifies the handling and allows us to remove cpplint_presubmit
without loss of functionality.
Bug: 68951293
Change-Id: I6ece835440e3ac3f84fadc544307a9a5126a3e1c
|
| cf2bd735fd85f19b7388a64b66d2b2f7169c2cdd |
|
30-Jul-2017 |
Colin Cross <ccross@android.com> |
Use absolute path for art/tools/cpplint.py --root
art/tools/cpplint.py still fails when run with --root=. if the
.git directory is missing. Replace --root=. with --root=$(pwd)
to give it an absolute path.
Bug: 34787652
Test: mv art/.git art/.git.bak; m -j cpplint-art-phony; mv art/.git.bak art/.git
Change-Id: I56c2acf54090e53a02d52fb12ffd2e5a862ede4f
|
| 06c42a571358b5e5adb69104b183af8f32f4c07d |
|
26-Jul-2017 |
Andreas Gampe <agampe@google.com> |
ART: Move openjdkjvmti to art/
Move libopenjdkjvmti out of the runtime directory. Let's not
pollute the runtime library.
Test: m test-art-host
Change-Id: Idb6b9cebcd61777bd3200437a2ae584a63a4a341
|
| 6744a5cedfca2964e19a204e9cb7acbe55937dda |
|
14-Jul-2017 |
Colin Cross <ccross@android.com> |
Always pass --root to cpplint
Mac builds were sometimes flaky when running cpplint.py because
the .git directory is sometimes removed when the disk is almost
full, which causes cpplint.py to fail to autodetect the project
root. Change "Pass --root to cpplint."
(I6d418979a9bdceefc7a2c19297a7cc9aacb3e83f) fixed a similar
problem when running inside of gitc, which doesn't have .git
directories, but use ANDROID_BUILD_TOP, which is only set if
lunch has been run. It also didn't set --root for some of the
phony targets.
Bug: 34787652
Test: m -j cpplint-art-phony
Change-Id: Ie0eb9d9d038f56a661750bb0f782321aba314a37
|
| 5e03a305edafb49a34b436d9a858281e53d390f2 |
|
13-Mar-2017 |
Andreas Gampe <agampe@google.com> |
ART: Refactor libopenjdkjvmti
Move jvmti.h to its own folder and fix up include setup.
Bug: 31455788
Test: m
Change-Id: I5beb66fc5d0178746216f61f9214cda54fc2c7ac
|
| 25c3d8d8cbefc8c4ada4094ac0296498e9dc9b96 |
|
22-Sep-2016 |
Sen Jiang <senj@google.com> |
Pass --root to cpplint.
cpplint looks for .git to detect root, but in gitc, there's no art/.git
unless I repo start in art/ first, but I'm not working on art and
checkbuild fails in gitc because of this.
Bug: None
Test: mma in art/ in gitc
Change-Id: I6d418979a9bdceefc7a2c19297a7cc9aacb3e83f
|
| 49948e9d4e66e68ea077031f9aa80b6c332bd673 |
|
12-Aug-2016 |
Alex Light <allight@google.com> |
Add skeleton of a jvmti plugin for art.
Test: mma test-art-host-run-test-901-hello-ti-agent
Change-Id: If6807b6238d57471e4ba0dd75c717721246443f6
|
| 9150b5fa5fa18257e2c4259ccdc42a368fdeeb8e |
|
12-Jan-2016 |
Alex Light <allight@google.com> |
Sort output of a find command in makefile
This should mean that there will be fewer spurious kati rebuilds of
the ninja files due to the output changing.
Change-Id: I26beb4ae1d95fe591c3854772b804718b41fe53c
(cherry picked from commit 80c0d7905ea2763caf4c561d9d033d03f1b3fc83)
|
| 80c0d7905ea2763caf4c561d9d033d03f1b3fc83 |
|
12-Jan-2016 |
Alex Light <allight@google.com> |
Sort output of a find command in makefile
This should mean that there will be fewer spurious kati rebuilds of
the ninja files due to the output changing.
Change-Id: I26beb4ae1d95fe591c3854772b804718b41fe53c
|
| 91de25fd88ed755be5d72db80c93da364d97e16f |
|
29-Oct-2015 |
Alex Light <allight@google.com> |
Make things generally quieter.
* Give run-test a --quiet flag that causes it to only print on errors.
* Give cpplint a --quiet flag that causes it to not print anything
when there are no errors.
* Add a ART_TEST_QUIET flag to build/Android.common_test.mk which
makes run-test targets much quieter when true (the default). With
this flag only failures will be printed.
* Make build/Android.cpplint.mk pass the new cpplint --quiet flag so
that only failures will be printed.
Before:
[ 96% 5715/5906] build test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-461-get-reference-vreg32
test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-448-multiple-returns32 RUNNING
/usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: building...
/usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: running...
/usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: succeeded!
test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-448-multiple-returns32 PASSED
After:
[ 96% 5715/5906] build test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-461-get-reference-vreg32
Change-Id: Idf6fce7f48a619f83254b48861dbd7f8eb4ebdbf
|
| e130ee6fe902315062ce17f8ce115b1dc506576d |
|
02-Jul-2014 |
Brian Carlstrom <bdc@google.com> |
Add a local elf.h
Change-Id: Ic2d449bfdd3c95e393c083c9e1022adc4e9c06b8
|
| afd9acc30bdd11cdd12d8209eb994cb371c65e33 |
|
17-Jun-2014 |
Ian Rogers <irogers@google.com> |
Multilib ART host.
Build ART for the host as a multilib project with dalvikvm32 and dalvikvm64
running as 32 or 64-bit repsectfully. Note, currently multilib host builds
are not the default, you make the so by setting BUILD_HOST_64bit=1.
Extend tests to execute in both 32 and 64-bit modes. By default both 32 and
64-bit tests are run, add 32 or 64 to the end of a test name to run it in
purely that flavor.
Given the extra spam, modify oat tests to only generate console output when
the test fails.
Change the test harness so that common commands are run when a test should be
skipped, when it passes or when it fails. Use these commands to generate a
summary of passing, skipped and failing tests. Tests will be skipped if they
are known to be broken or if a test has already failed. Setting the variable
TEST_ART_KEEP_GOING=true will force working tests not to be skipped.
In this change all tests running on the optimizing compiler are marked broken
due to breakages running them in a multilib environment.
Break apart Android.common.mk into its constituent parts, along with other
pieces of reorganization.
Stylistic nit, we refer to make rule targets as targets thereby overloading
the term target. While consistent with make's terminology, its confusing with
the Android notion of target. I've switched to just calling targets rules to
avoid confusion in host tests.
Change-Id: I5190fc3de46800a949fbb06b3f4c258ca89ccde9
|
| 43c86422e210a3883729ab90997711e79f26bccc |
|
18-Mar-2014 |
Nicolas Geoffray <ngeoffray@google.com> |
Fix lint error, and Makefile that could be confused with local files.
Change-Id: I780cc0d6593eadd6f82e1126d7ad445894af666c
|
| 7934ac288acfb2552bb0b06ec1f61e5820d924a4 |
|
26-Jul-2013 |
Brian Carlstrom <bdc@google.com> |
Fix cpplint whitespace/comments issues
Change-Id: Iae286862c85fb8fd8901eae1204cd6d271d69496
|
| 05eecea70b64095db2bc5f9f8aef7475964a9d7b |
|
19-Jul-2013 |
Brian Carlstrom <bdc@google.com> |
Enable running cpplint.py on art C++ files when using mm and mmm
Change-Id: I945bfc37c1e95310ee52e171faeeafc224934215
|