Commit Graph

11830 Commits

Author SHA1 Message Date
Paul Duffin
e6cb60da57 Merge "Allow java_test.go tests to use any Android.bp file"
am: 45656aacab

Change-Id: I087cf543b7bc534e39b40f9e06e13a15487af109
2019-06-13 00:11:59 -07:00
Paul Duffin
c064604439 Merge "Add test for no_framework_libs:true"
am: 5b775ae8b4

Change-Id: I45889d705416212125ab466432df4f00e21a942d
2019-06-13 00:11:29 -07:00
Paul Duffin
45656aacab Merge "Allow java_test.go tests to use any Android.bp file" 2019-06-13 06:56:56 +00:00
Paul Duffin
5b775ae8b4 Merge "Add test for no_framework_libs:true" 2019-06-13 06:56:25 +00:00
Yi Kong
5c54831a17 Merge "Move darwin to llvm-ar"
am: 4821597065

Change-Id: I6a2529498bd8db1ae0c442d239464191b1362eb0
2019-06-12 23:15:20 -07:00
Yi Kong
4821597065 Merge "Move darwin to llvm-ar" 2019-06-13 06:04:29 +00:00
Colin Cross
4fb5563c20 Merge changes I89ad92f3,Ife1604c0
am: e51e2feafd

Change-Id: I262c2fcfa7df51b20c8bc15e851d6b7c7d4a8ed1
2019-06-12 20:58:53 -07:00
Colin Cross
e51e2feafd Merge changes I89ad92f3,Ife1604c0
* changes:
  Don't add dependencies on uses-library modules in unbundled builds
  Add implicit outputs to ErrorRule rules
2019-06-13 03:36:08 +00:00
Yi Kong
823532df31 Move darwin to llvm-ar
Test: build
Bug: 71618641
Change-Id: Ib45963bc9e295afe39752a270f7b5a1197335c69
2019-06-12 19:16:25 -07:00
Colin Cross
98a6f300fa Merge changes Ia17b2bcb,Ic71892c3,Id2b23b9e,I435ee7aa,I85112506, ...
am: 72ea641dff

Change-Id: I5f53e14e7d9ccfc3843d2d9e5c1d6d3bd9ca1b9b
2019-06-12 13:36:40 -07:00
Dan Willemsen
b29f7672a1 Merge "Convert cc aidl to rule builder"
am: 733728b42c

Change-Id: I10b1c99b47e5d83e0d24d5f353392f7dd380b5df
2019-06-12 13:34:02 -07:00
Colin Cross
3245b2cb83 Don't add dependencies on uses-library modules in unbundled builds
Unbundled builds may not have definitions of shared libraries, and
they are not dexpreopted so the dependencies are not used anyways.

Test: manual
Change-Id: I89ad92f3b073422734f824ac3a8a3b9baf995ccb
2019-06-12 20:20:21 +00:00
Colin Cross
4b69c497da Add implicit outputs to ErrorRule rules
When building with ALLOW_MISSING_DEPENDENCIES=true, Soong replaces
rules in modules that have missing dependencies with ErrorRule
rules that print an error.  The ErrorRules were not listing
implicit outputs, which could lead to dangling dependencies.

Test: manual
Change-Id: Ife1604c0a9a1159087b12568fd4c2b69517d81a7
2019-06-12 20:20:13 +00:00
Colin Cross
72ea641dff Merge changes Ia17b2bcb,Ic71892c3,Id2b23b9e,I435ee7aa,I85112506, ...
* changes:
  Consolidate *MutatorContext and ModuleContext into BaseModuleContext
  Add test for missing defaults modules with AllowMissingDependencies
  Capture missing dependency error rules
  Share buildDir for android/soong/android tests
  Reimplement AddMissingDependencies
  Add GenerateAndroidBuildActions to DefaultsModuleBase
2019-06-12 20:19:55 +00:00
Dan Willemsen
733728b42c Merge "Convert cc aidl to rule builder" 2019-06-12 20:08:46 +00:00
Jiyong Park
8e5d79cc67 Merge "Let genrule be mutatable by the apex mutator"
am: 7b8afc798a

Change-Id: Ide6e76e807860525be3bfd786fbce8a216cce29c
2019-06-12 11:50:53 -07:00
Treehugger Robot
7b8afc798a Merge "Let genrule be mutatable by the apex mutator" 2019-06-12 18:39:28 +00:00
Alexander Smundak
8975b8f376 Merge "Tune JVM, print full diagnostics on crash."
am: 90aa101152

Change-Id: I43b5beb72799c7b929c3915d5c993878263edb6d
2019-06-12 08:53:53 -07:00
Alexander Smundak
90aa101152 Merge "Tune JVM, print full diagnostics on crash." 2019-06-12 15:33:06 +00:00
Paul Duffin
baccf7e122 Allow java_test.go tests to use any Android.bp file
Previously, the tests had to use the Android.bp file in the root
directory of the mock file system. That prevented adding tests that are
dependent on the location of the Android.bp file, e.g. ones that use
no_standard_libs. This change will process any Android.bp in the mock
filesystem.

Bug: 134566750
Test: m
Change-Id: I6fb057a473a18e87bd1a89507e78ceb3fd171eb5
2019-06-12 14:08:13 +01:00
Paul Duffin
fad7980075 Add test for no_framework_libs:true
Bug: 134566750
Test: m droid
Change-Id: Ie3c5c0730d4505c7932952eb20b5ccafd806a73a
2019-06-12 14:08:13 +01:00
Paul Duffin
9ec3254d13 Remove unnecessary use of sdk_version: "core_current"
am: b826a2f269

Change-Id: Ibdef47ca2af28f43ee3340641e828044f089cb44
2019-06-12 03:06:58 -07:00
Paul Duffin
28630e0c47 Add tests for core library neverallow rules
am: b6c6bdd1c2

Change-Id: I038cac7caec29afb74bda6c2f3e57f7aafe42632
2019-06-12 03:06:54 -07:00
Paul Duffin
b826a2f269 Remove unnecessary use of sdk_version: "core_current"
Modules that specify no_standard_libs: true should not specify the
sdk_version property as well because the former supercedes the latter.

Bug: 134566750
Test: m
Change-Id: I1d8663dc3a25baf23a1b136c7f35cf130cd7543b
2019-06-12 10:50:24 +01:00
Paul Duffin
b6c6bdd1c2 Add tests for core library neverallow rules
Bug: 134566750
Test: m
Change-Id: I65e2afdf726937c29fc4f62a317024dc133254bf
2019-06-12 10:50:24 +01:00
Jiyong Park
fc752ca1ac Let genrule be mutatable by the apex mutator
genrule modules were not mutated by the apex mutator. As a result, a
module that is mutated for an apex have depended on the genrule modules
host_bionic_linker_{asm|flags} that are not mutated for the APEX.

This in turn caused an inconsistency problem because the
host_bionic_linker_* genrules use the dynamic linker that isn't mutated
for APEX as an input, while the outputs of the genrules are used by
modules mutated for APEX. The inconsistency caused b/135008828.

Fixing this issue by correctly let genrule to be mutated by the apex
mutator.

Bug: 135008828
Test: checkout master-art-host
ALLOW_MISSING_DEPENDENCIES=true DIST_DIR=out/dist
./art/tools/dist_linux_bionic.sh -j50 showcommands
com.android.runtime.host
is passing

Change-Id: I92d90400c61d3951f6202d932832d0cb63c95a7d
2019-06-12 13:28:11 +09:00
Oliver Nguyen
85cefe661b Merge "Package coverage files as a zip."
am: 7268fd494d

Change-Id: Iac5315b45fa03a0b0c2d4abeb33f21bf93aa374a
2019-06-11 19:21:57 -07:00
Treehugger Robot
7268fd494d Merge "Package coverage files as a zip." 2019-06-12 02:00:01 +00:00
Oliver Nguyen
c743414d42 Package coverage files as a zip.
Test: make NATIVE_COVERAGE=true COVERAGE_PATHS="*"
Change-Id: I955212a20ea19c854ebc88635252a1d174ea8f7f
2019-06-11 16:04:11 -07:00
Elliott Hughes
e37424dc51 Merge "Switch to toybox date."
am: 633352f03f

Change-Id: Ic25d65226a1651b20cac835a0134a39975970137
2019-06-11 14:26:43 -07:00
Elliott Hughes
633352f03f Merge "Switch to toybox date." 2019-06-11 20:49:52 +00:00
Sasha Smundak
26c6d9e9fb Tune JVM, print full diagnostics on crash.
JVM occasionally (0.5% time) crashes on entering a method with SIGSEGV.
It might be due to excessive load on the host machine. Reduce the number
of the compiler threads to 6, and do not start all the GC threads
immediately. If JVM crashes, copy full diagnostics to stderr.

Bug: 132766811
Test: treehugger
Change-Id: Ib7bfd568f6d32456bf623cbb615ffc990c326a34
2019-06-11 13:30:13 -07:00
Elliott Hughes
bc31b9190a Switch to toybox date.
Test: treehugger
Change-Id: I558870092c29946419b9e15c33a6ee8ee7c6088f
2019-06-11 10:09:08 -07:00
Roland Levillain
299abf19c3 Merge "Have symlink_preferred_arch: true honor the stem property."
am: d7607c1c99

Change-Id: I741c846028da53facf11ae5a73cb36c4f650109b
2019-06-11 06:56:02 -07:00
Roland Levillain
d7607c1c99 Merge "Have symlink_preferred_arch: true honor the stem property." 2019-06-11 13:45:25 +00:00
Pete Gillin
071d0d6799 Merge "Don't specify -encoding twice in Jdiff javadoc invocations."
am: a53ae43180

Change-Id: I6bda6b1b57cc7b08141cb9adb2d6cdb48012f872
2019-06-11 04:31:41 -07:00
Pete Gillin
a53ae43180 Merge "Don't specify -encoding twice in Jdiff javadoc invocations." 2019-06-11 11:21:37 +00:00
Inseob Kim
6011640688 Merge "Separate exported includes out of flags"
am: b879fb6b4b

Change-Id: Id66ff859751816eae1c97ec247bbffe057cc2e86
2019-06-10 22:07:20 -07:00
Treehugger Robot
b879fb6b4b Merge "Separate exported includes out of flags" 2019-06-11 04:57:47 +00:00
Dan Willemsen
2758050dc4 Merge "Refactor .aar resource compilation"
am: 588857b418

Change-Id: I85532e6de6a8514e64c4304df711856584c2caef
2019-06-10 21:11:54 -07:00
Dan Willemsen
2cb02169d4 Merge "Add dependency to version script when linking stub libraries"
am: d1fcc4a93d

Change-Id: I3b9010ddaa6ff810d883ad7cffbc56cb513546dc
2019-06-10 21:11:11 -07:00
Treehugger Robot
588857b418 Merge "Refactor .aar resource compilation" 2019-06-11 04:03:51 +00:00
Treehugger Robot
d1fcc4a93d Merge "Add dependency to version script when linking stub libraries" 2019-06-11 03:59:42 +00:00
Josh Gao
1697e781bb Merge "Revert "Revert "Revert "Revert "Enable lld for windows"""""
am: a1738e2c00

Change-Id: I4d2b7446b8bbb02938f5bf56c4c8a627d8afad9b
2019-06-10 18:56:23 -07:00
Josh Gao
a1738e2c00 Merge "Revert "Revert "Revert "Revert "Enable lld for windows""""" 2019-06-11 01:43:42 +00:00
Inseob Kim
6364f4732a Merge "Fix sysprop_library build err when using filegroup"
am: 4b0cc8c3c5

Change-Id: I231370f6e2925a2fa8cf6b17106779300d85a095
2019-06-10 18:43:41 -07:00
Inseob Kim
6937844c29 Separate exported includes out of flags
Exported includes have been maintained along with other C/C++ flags.
This makes dependencies unclear, and users have had to parse flags to
get exported directories. This separates exported includes and
exported flags, thus making data more structured and explicit.

Bug: 132818174
Test: m
Change-Id: I5c25ac2224988c4a67e4db6fd6e4d39090b74041
2019-06-11 10:37:56 +09:00
Treehugger Robot
4b0cc8c3c5 Merge "Fix sysprop_library build err when using filegroup" 2019-06-11 01:21:44 +00:00
Dan Willemsen
939408aa22 Add dependency to version script when linking stub libraries
This isn't an effective issue with local builds currently, since the
version script is generated from the same rule as the sources used to
compile the objects that are also used in the link command. But if we
ever separated those paths or adopted restat, we could miss this
dependency.

This is also required for my RBE build to actually expose this file to
the link step.

Test: treehugger
Change-Id: I32bbb18cf7edddc88759d4f445d081868f3e9b44
2019-06-10 18:02:25 -07:00
Colin Cross
dc35e21124 Consolidate *MutatorContext and ModuleContext into BaseModuleContext
Following 99bdb2ab4fd5168a71a20aecf10611425be47ec4 in build/blueprint,
move more methods into BaseModuleContext.

This reapplies I9f8df94f1ae2b55d3cccf7b9468247f3e7cb2ebd after fixing
missing errors thrown for missing defaults modules when
AllowMissingDependencies == true.

Test: m checkbuild
Test: defaults_test.go
Change-Id: Ia17b2bcbf2bac6889c419b2e73953946f6aa40ad
2019-06-10 16:30:35 -07:00