aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Split logtags implementations for cc and javaColin Cross2017-12-077-81/+16
| | | | | | | | | | | | | | | | Logtags files in cc and java are treated fundamentally differently. In cc, they are not used for compiling at all, but need to be passed to Make to be combined into the global logtags list, and logtag files are listed in a logtags property. In java they are listed in srcs and produce generated code that is compiled in, and so shouldn't also need to be listed in a logtags property. Move the logtags property to cc and export it to Make from there, and have java extract logtags files from srcs to be exported to Make. Test: m checkbuild Change-Id: I31d49289efe72db60d2f33566df771b4a3ebc8a0
* Merge "Remove 6 directories from WarningAllowed*Projects."Chih-hung Hsieh2017-12-071-6/+0
|\
| * Remove 6 directories from WarningAllowed*Projects.Chih-Hung Hsieh2017-12-071-6/+0
| | | | | | | | | | Test: normal build Change-Id: I44381a2a7994d9f74329b0516d2bb303b5866a66
* | Merge "Add HostToolPath() for Python binary"Treehugger Robot2017-12-071-0/+8
|\ \
| * | Add HostToolPath() for Python binaryNan Zhang2017-12-061-0/+8
| | | | | | | | | | | | | | | | | | | | | So in the genrule, Python binary module can be used as tool. Test: manually write a test genrule. Change-Id: Idfd3af4c1002dd608e1bdffa203e01c802f1bf21
* | | Merge changes from topic "soong_dexpreopt"Colin Cross2017-12-065-8/+38
|\ \ \ | | | | | | | | | | | | | | | | | | | | * changes: Add more dex_preopt properties Run gofmt -w
| * | | Add more dex_preopt propertiesColin Cross2017-12-063-6/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move dex_preopt to dex_preopt.enabled, and add dex_preopt.app_image, dex_preopt.profile_guided, and dex_preopt.profile. These values will be passed back to Make if provided to control dex preopting. Test: m checkbuild Change-Id: I54a4b1de697a08be20ab65d2a5dc43ce0046692d
| * | | Run gofmt -wColin Cross2017-12-062-2/+2
| | | | | | | | | | | | | | | | | | | | Test: none Change-Id: I2b503e06a4ae8e72a08f6bfb64692dfd33e2b7e2
* | | | Merge "Update pom2mk to allow duplicate module names if rewritten"Treehugger Robot2017-12-061-13/+22
|\ \ \ \ | |/ / / |/| | |
| * | | Update pom2mk to allow duplicate module names if rewrittenAlan Viverette2017-12-051-13/+22
| | | | | | | | | | | | | | | | | | | | | | | | Fixes: 70162730 Test: ./update_current.py -s -t 4482279 Change-Id: I874d7bfb50fd2d2bc488f5458cfe57b2e0d4d4e5
* | | | Merge "Revert "Default to building with OpenJDK 9 -target 1.8.""Tobias Thierer2017-12-062-15/+3
|\ \ \ \ | | |_|/ | |/| | | | | | | | | | | | | | am: c7efdd9b36 Change-Id: Icdbace15dcd4cc5f337b575e98db6265fdf967f2
| * | | Merge "Revert "Default to building with OpenJDK 9 -target 1.8.""Tobias Thierer2017-12-062-15/+3
| |\ \ \
| | * | | Revert "Default to building with OpenJDK 9 -target 1.8."Tobias Thierer2017-12-052-15/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After this revert CL topic, the default toolchain for Android goes back to being OpenJDK 8. This revert is being prepared ahead of time in case of any problems with original change topic, https://r.android.com/#/q/topic:bug_69449021 Bug: 69449021 Test: Treehugger This reverts commit 0ae8b548af38c8b8fccea7cdef11d54740d3bb6a. Change-Id: Ief7646a94f1a264085cd299b4327d244b78a1537
* | | | | Merge "Build with OpenJDK 9 -target 1.8 by default except for errorprone."Tobias Thierer2017-12-062-3/+15
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: cf8d83de82 Change-Id: Ibe7792469a125f78ef98240f63f2d08047d0e4f1
| * | | | Merge "Build with OpenJDK 9 -target 1.8 by default except for errorprone."Tobias Thierer2017-12-062-3/+15
| |\| | |
| | * | | Build with OpenJDK 9 -target 1.8 by default except for errorprone.Tobias Thierer2017-12-052-3/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before this CL topic, the build toolchain for .java source files used OpenJDK 8, targeting 1.8 (v52 class files) by default. This CL topic switches the default to OpenJDK 9, but still targeting 1.8 (v52 class files) by default. If USE_ERROR_PRONE is set to true, then the default remains OpenJDK 8. Code in the Android platform should generally be unaffected, but if host tools that are now compiled and run using OpenJDK 9 are causing problems for your team, then let me know. To manually switch back to the old behavior for now (continue using OpenJDK 8), run this command in your shell: export EXPERIMENTAL_USE_OPENJDK9=false Bug: 69449021 Test: Treehugger Test: "make core-oj", checked that compilation now uses OpenJDK 9 javac -target 1.8 Test: Checked that this is still compiled using OpenJDK 8. export EXPERIMENTAL_USE_OPENJDK9=false make core-oj Change-Id: Ic87e9bb2a2e5da0ff13a2e51845b5365901c1507
* | | | | Split java_binary modules into common and binary variantsColin Cross2017-12-065-52/+119
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 6b4a32d771 Change-Id: I6da3e0b26b39e079769dbaed65903416f9fa1ebb
| * | | | Split java_binary modules into common and binary variantsColin Cross2017-12-065-52/+119
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a common_first multilib type and use it for java.Binary so that the java part is compiled as a "common" arch type but the wrapper script is installed as a "linux_glibc" arch type. This allows java_binary to be used as a tool dependency for a genrule. Bug: 68397812 Test: TestJavaBinary Change-Id: I809060839ce8878300da3fb76426ceb1ea6b0e8e
* | | | | Make (Source|Output)Path.Join("foo").Rel() return "foo"Colin Cross2017-12-061-5/+15
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 0db5568612 Change-Id: I6a19550ef017532b2623491938ff000a0e31ac29
| * | | | Make (Source|Output)Path.Join("foo").Rel() return "foo"Colin Cross2017-12-061-5/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows TestModule.Output("foo") to be used to find a file called "foo" that was installed using ctx.InstallFile. Test: soong tests Change-Id: I04833c9ee8ac5baa6b6afd35715d1191c1622a78
* | | | | Add java_genrules to use jars as inputs and outputsColin Cross2017-12-064-26/+146
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 5425090a4f Change-Id: I0b6f3b8f04bb5ced5df7b9a985ed5298e8ba2e1e
| * | | | Add java_genrules to use jars as inputs and outputsColin Cross2017-12-064-26/+146
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a java_genrule that has the right multilib flags to be a dependency of a java rule. Make java libraries implement SourceFileProducer so that their classes jar can be used as an input to a java_genrule. Allow libs and static_libs dependencies to be a java_genrule. Test: TestJarGenrules Change-Id: Ib1b31ef9c0b7e72eeed2c9ecc4ce8a1088e0b1c9
* | | | | Make TestModule.Output print a list of all outputs on errorColin Cross2017-12-061-1/+4
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 9cf27db425 Change-Id: Id3e6b3e4b2faee411753d4e34ce59a3cee2651ac
| * | | | Make TestModule.Output print a list of all outputs on errorColin Cross2017-12-061-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Debugging when writing tests using TestModule.Outputs is unnecessarily hard, it panics when an incorrect output path is given but doesn't provide any help to figure out why. Follow the pattern used by TestContext.ModuleForTests and print the list of valid output paths on failure. Test: m checkbuild Change-Id: I50e8e2dfc2070bd538d47cf6495a489f727b1564
* | | | | Allow warnings in libbufferhub.Chih-Hung Hsieh2017-12-061-0/+1
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 809d3f8f63 Change-Id: Ia4c668bbf02fc0a7af7fec12e1f1f1ec3aaabf58
| * | | | Allow warnings in libbufferhub.Chih-Hung Hsieh2017-12-051-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * This directory still has warnings. Test: normal build Change-Id: Ie322ebf38d828d63e9115308b8e71769fc2a452a
* | | | | Reduce WarningAllowedProjects.Chih-Hung Hsieh2017-12-051-7/+0
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 8ca48361ef Change-Id: Id48dfe471ff908df3e3ec6bc4dab8d7e3466fbaf
| * | | | Reduce WarningAllowedProjects.Chih-Hung Hsieh2017-12-051-7/+0
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | * Warnings were fixed or suppressed in these projects. Test: normal build Change-Id: I24ef1b59d6c5ccd50979b7d792764de9d6da7e30
* | | | Merge changes from topic "soong-namespaces"Jeff Gaston2017-12-053-17/+59
|\| | | | | | | | | | | | | | | | | | | | | | | am: 73c2099534 Change-Id: I75fa176c2fa545811bf14500c23f55d7426aa439
| * | | Merge changes from topic "soong-namespaces"Jeff Gaston2017-12-0511-7/+1212
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * changes: Make ninja file deterministic even with dup module names require namespaces to be declared only in files named Android.bp Document Soong namespaces Revert "Revert "Soong support for namespaces""
| | * | | Make ninja file deterministic even with dup module namesJeff Gaston2017-12-043-17/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 65683273 Test: ./build/soong/scripts/diff_build_graphs.sh \ --products=aosp_arm '' '' Change-Id: Ie01ff579d69e94b12363f53aec5d25810211c451
* | | | | require namespaces to be declared only in files named Android.bpJeff Gaston2017-12-052-16/+54
|\ \ \ \ \ | | |/ / / | |/| | | | | | | | | | | | | | | | | | am: 5c3886de5a Change-Id: I41b69b310e9051f07533962f2c90c2ef311051fc
| * | | | require namespaces to be declared only in files named Android.bpJeff Gaston2017-12-042-16/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 65683273 Test: m -j nothing # which runs unit tests Change-Id: I5edf0e0482809f5ac9fb9dfff342fb404e1c52da
* | | | | Document Soong namespacesJeff Gaston2017-12-051-0/+32
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 44c0cd8543 Change-Id: Ice8ea9a9682121d0530d59aa482f50f0b21737ed
| * | | | Document Soong namespacesJeff Gaston2017-12-041-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 65683273 Test: Read README.md Change-Id: I179c085e123858ae425515ac6bbf4b272c245a2e
* | | | | Revert "Revert "Soong support for namespaces""Jeff Gaston2017-12-0510-6/+1099
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 088e29ed38 Change-Id: Id969fc05530600bbde27282c3035ad75e6e46a0e
| * | | | Revert "Revert "Soong support for namespaces""Jeff Gaston2017-12-0410-6/+1099
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This mostly reverts commit 178d5fefc0cea9d0f031c0bdee125b9d960f32c3 and mostly reapplies change I6d3e52ef62c4cabe85b9a135a54de0e1a6aab29c . Bug: 65683273 Test: build/soong/scripts/diff_build_graphs.sh \ --products=aosp_arm \ 'build/blueprint:work^ build/soong:work^' \ 'build/blueprint:work build/soong:work' # and see that the only changes were: # 1. adding some new files # 2. changing some line numbers Test: m -j nothing # which runs unit tests Change-Id: I32baae00277a547fdcdd1c2219fe6625ee0e45d7
* | | | Merge "Have diff_build_graphs.sh print the number of lines in the diff"Jeff Gaston2017-12-051-1/+1
|\ \ \ \ | | |/ / | |/| | | | | | | | | | | | | | am: 7c25d88e6b Change-Id: I809ac491b2a6b9bca8c9683df421c21dbd6b12e1
| * | | Merge "Have diff_build_graphs.sh print the number of lines in the diff"Treehugger Robot2017-12-051-1/+1
| |\ \ \ | | |/ / | |/| |
| | * | Have diff_build_graphs.sh print the number of lines in the diffJeff Gaston2017-11-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: b/69133815 Test: build/soong/diff_build_graphs.sh \ 'build/soong:work^' 'build/soong:work' Change-Id: Iaeb66ba53e1b3d7ddd4b4551b3ac29b12ce227c5
* | | | Install VNDK snapshot libraries for system buildJustin Yun2017-12-057-8/+178
|\| | | | | | | | | | | | | | | | | | | | | | | am: 7154928c93 Change-Id: I5320a9aa79cfac4d30ade0db1d2c901d03055c1a
| * | | Install VNDK snapshot libraries for system buildJustin Yun2017-12-057-8/+178
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When BOARD_VNDK_VERSION := <VNDK version>, or PRODUCT_EXTRA_VNDK_VERSIONS includes the needed <VNDK version> list, the prebuilt VNDK libs in prebuilts/vndk/ directory will be installed. Each prebuilt VNDK module uses "vndk_prebuilt_shared" for shared VNDK/VNDK-SP libs. Following is the sample configuration of a vndk snapshot module: vndk_prebuilt_shared { name: "libfoo", version: "27", vendor_available: true, vndk: { enabled: true, }, arch: { arm64: { srcs: ["arm/lib64/libfoo.so"], }, arm: { srcs: ["arm/lib/libfoo.so"], }, }, } The Android.bp for the snapshot modules will be auto-generated by a script. Bug: 38304393 Bug: 65377115 Bug: 68123344 Test: set BOARD_VNDK_VERSION := 27 copy a snapshot for v27 build with make command Change-Id: Ib93107530dbabb4a24583f4d6e4f0c513c9adfec
* | | | Merge "No need to dump modules added -Werror."Chih-Hung Hsieh2017-12-052-3/+0
|\| | | | | | | | | | | | | | | | | | | | | | | am: 5760c9a12c Change-Id: I465d2534cdbf351aed1c7c06966518f199391ea0
| * | | Merge "No need to dump modules added -Werror."Treehugger Robot2017-12-052-3/+0
| |\ \ \
| | * | | No need to dump modules added -Werror.Chih-Hung Hsieh2017-12-042-3/+0
| | | |/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | * This reduces size of output file wall_werror.txt. * Stop collecting SOONG_MODULES_ADDED_WERROR. Bug: 66996870 Test: normal build Change-Id: Ic6d87f6ee2b36b90ad0de4ac4f6f176334aba55e
* | | | Make setup_go_workspace_for_soong work on MacDan Willemsen2017-12-051-1/+9
|\| | | | | | | | | | | | | | | | | | | | | | | am: 932ecc1a97 Change-Id: Ic981d1a982dadd4604797e470ea1bfe427987be7
| * | | Make setup_go_workspace_for_soong work on MacDan Willemsen2017-12-041-1/+9
| |/ / | | | | | | | | | | | | Test: build/soong/scripts/setup_go_workspace_for_soong.sh on Mac and Linux Change-Id: Idfcad3ace6f3ff4b68a438b8f1e0d94ff35573e0
* | | Fix builds outside of makeColin Cross2017-12-041-4/+12
|\| | | | | | | | | | | | | | | | | am: e15ddaf4ae Change-Id: I2d313af3ec290d38b32361785b4a41d508e4aae4
| * | Fix builds outside of makeColin Cross2017-12-041-4/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Builds outside of make use custom config files that may not set some variables, which is causing nil pointer derefernces. Use wrapper functions that check for nil for the new config variables. Test: build/soong/scripts/build-ndk-prebuilts.sh Change-Id: I0f837094532aeb2ecdbe401affa55a7535075bf6
* | | Fix apps buildColin Cross2017-12-032-2/+2
|\| | | | | | | | | | | | | | | | | am: 42f3a76f00 Change-Id: I335bfc9b17f3564e1d4714706983406374994e7d