aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Dedup exported flags from dependencies"android-n-iot-preview-4Treehugger Robot2017-05-172-0/+70
|\
| * Dedup exported flags from dependenciesColin Cross2017-05-172-0/+70
| | | | | | | | | | | | | | | | Soong command lines have gotten very long due to hidl modules reexporting lots of libraries. Dedup the include dir flags. Test: m -j checkbuild Change-Id: I6ada1251012da42344e2c00ae66001a649023d2c
* | Add python_test_host module.Nan Zhang2017-05-178-60/+226
| | | | | | | | | | | | bug: 31676493 Test: created py_test modules in real folder. and ran 'mma'. Change-Id: I22aa2fad74b11e4a31ea7a4c4a4f0ea64cd3fc94
* | Merge "Fix the issue "\n" didn't get interpreted properly."Treehugger Robot2017-05-172-2/+5
|\ \
| * | Fix the issue "\n" didn't get interpreted properly.Nan Zhang2017-05-162-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added params to WriteFile rule that allow users to turn on the specific 'echo' options, such as "enable the interpretation of backslash escapes". Bug: b/38261000 Test: tested with real data files. Change-Id: I5145056b5c2522a3a72b9436055ec95d2fbd94c0
* | | Merge changes I9512642d,I6548889c,I8db5198fTreehugger Robot2017-05-165-77/+216
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | * changes: Add build_test.sh, split common parts of soong_ui.bash Allow specifying a build variant Improve multiproduct_kati output
| * | | Add build_test.sh, split common parts of soong_ui.bashDan Willemsen2017-05-153-59/+126
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move common microfactory functions to cmd/microfactory/microfactory.bash so that they can be used to build both soong_ui for normal build and multiproduct_kati for build system tests. Test: m -j Test: build/soong/build_test.bash Change-Id: I9512642d846ce54d05a027b6d33a2b3029b3f90b
| * | | Allow specifying a build variantDan Willemsen2017-05-151-1/+3
| | | | | | | | | | | | | | | | | | | | Test: multiproduct_kati --variant userdebug Change-Id: I6548889caf868cad6270110c038753d029247572
| * | | Improve multiproduct_kati outputDan Willemsen2017-05-152-17/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It now uses the same output style as ninja, overwriting status lines in smart terminals. Test: multiproduct_kati Test: multiproduct_kati | cat Change-Id: I8db5198ffdc5ebc5503241ac492379753d92978e
* | | | Merge changes from topics 'soong_ui_installclean', 'soong_ui_version', ↵Dan Willemsen2017-05-167-17/+367
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'soong_ui_clean' * changes: Move auto installclean to soong_ui Move version checking from Make into soong_ui Move clean/clobber to soong_ui
| * | | | Move auto installclean to soong_uiDan Willemsen2017-05-153-3/+103
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This way kati won't need to be run as often (either initially, or when switching products with the same device). Bug: 35970961 Test: m clean; m -j blueprint_tools; m -j blueprint_tools; m -j blueprint_tools Test: lunch aosp_arm-eng; m -j blueprint_tools; lunch full-eng; m -j blueprint_tools; <repeat> Change-Id: Ie9fca3c8f1dd412459ea47c7090c7c5fdb0bcf6e
| * | | | Move version checking from Make into soong_uiDan Willemsen2017-05-156-14/+253
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When kati keeps state around, it has to regenerate the ninja file every time the state is changed. So move the java version checking into soong_ui, where we can parallelize it with other operations instead of only checking it occasionally. Bug: 35970961 Test: Put java7 in PATH, m -j Test: Put java8-google in PATH, m -j Test: Put a space in TOP, m -j Test: OUT_DIR=<case-preserving fs> m -j Test: OUT_DIR=<path with space> m -j Test: DIST_DIR=<path with sapce> m -j Change-Id: I3245c8dd6d856240d17d54cb05d593dc9df71a27
| * | | | Move clean/clobber to soong_uiDan Willemsen2017-05-151-0/+11
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So that we don't have to load up all the makefile state just to remove the output directory. Starting from a completely empty out directory: kati: 16s soong_ui: 2.0s From a minimal out directory (m -j blueprint_tools): kati: 3.8s soong_ui: 0.4s Test: m -j clean Test: m -j clobber Change-Id: Ibeec6fbfa29387750342a752a55336caca4b3992
* | | | Merge "soong: add cortex-a73 to microarchitecture list"Christopher Ferris2017-05-161-0/+2
|\ \ \ \
| * | | | soong: add cortex-a73 to microarchitecture listJake Weinstein2017-05-151-0/+2
| | |/ / | |/| | | | | | | | | | | | | | | | | | * Was missed in I3325f60add7f424f8cca53c22919fc481ef5e787 Change-Id: I24d1ef38922cb8f46c9136018cce6a22293ca95e
* | | | Merge "Support for LOCAL_AIDL_INCLUDES in androidmk"Jeff Gaston2017-05-161-1/+1
|\ \ \ \
| * | | | Support for LOCAL_AIDL_INCLUDES in androidmkJeff Gaston2017-05-151-1/+1
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now it converts to: aidl: { include_dirs: [ "...", ] } Bug: 37747936 Test: run androidmk against an Android.mk file having LOCAL_AIDLE_INCLUDES Change-Id: If9f4c3ee9969610f1e0e0f147ee6a70102b63edc
* | | | Merge "Revert "Revert "Ensure environment dependencies are correct"""Treehugger Robot2017-05-163-10/+46
|\ \ \ \
| * | | | Revert "Revert "Ensure environment dependencies are correct""Dan Willemsen2017-05-153-10/+46
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 4068a5db6c60d890e4d49379d600fd34ee94fdec. Now the Mac xcode-select and xcrun goes through Config.HostSystemTool, which will grab $PATH through Config.Getenv Test: m -j (on mac) Change-Id: I2632c4fdb2ec961e59944cf02ff165e0fd3c869d
* | | | Merge "Clarify paths.go somewhat"Treehugger Robot2017-05-163-40/+40
|\ \ \ \ | |_|/ / |/| | |
| * | | Clarify paths.go somewhatJeff Gaston2017-05-153-40/+40
| |/ / | | | | | | | | | | | | | | | | | | Bug: 35562758 Test: make Change-Id: I4cf05ffdd0962186f490467829e3576b4e269c47
* / / Add cortex-a73 support.Christopher Ferris2017-05-152-5/+23
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | Also, update flags for cortex-a53. Bug: 37647380 Test: Built target with cortex-a73 for both 32 bit/64 bit. Test: Built target with cortex-a53.cortex-a57 for both 32 bit/64 bit. Test: Ran bionic unit tests, ran art target tests. (cherry picked from commit 1d9aa26d445cd5407aea0831e6b67fb37dfc1d05) Change-Id: I3325f60add7f424f8cca53c22919fc481ef5e787
* / Clean OUT_DIRDan Willemsen2017-05-121-1/+3
|/ | | | | | | | | | | | | | Otherwise Make and soong_ui disagree about where the soong.variables file should be. For example, when "OUT_DIR=out/": Error dumping make vars: Failed to parse make line: "make: Nothing to be done for `out/soong/soong.variables'." We would need to pass out//soong/soong.variables into make, since it doesn't attempt to match clean/unclean paths. Bug: 37553659 Test: OUT_DIR=out/ m -j Change-Id: Iebeea4868b96b11e8b08735f272e22be5db1d701
* Prettify soong ninja build descriptionsColin Cross2017-05-1016-187/+249
| | | | | | | | | | | | | | | | | Descriptions currently look like: [ 0% 4/29328] cc out-soong/.intermediates/external/clang/lib/Sema/libclangSema/android_arm_armv7-a-neon_denver_static_core/obj/external/clang/lib/Sema/SemaCodeComplete.o This is not very helpful - most of the characters are used to show the output path, which contains useful information like target architecture, but also contains most of the path to the source files twice, and less useful information like the exact variant name used by soong. Make the descriptions look like: [ 0% 3/29329] //external/clang/lib/Sema:libclangSema clang++ SemaTemplateInstantiate.cpp This is //path/to/module:modulename tool relative/path/to/source/file Test: builds, looks pretty Change-Id: I3087aa7d4eb1860ef6239d77407b8b35445616d7
* Merge changes If90975c8,I2a873fc6Colin Cross2017-05-104-35/+59
|\ | | | | | | | | | | * changes: Use ctx.ModuleBuild for darwin ar Run gofmt
| * Use ctx.ModuleBuild for darwin arColin Cross2017-05-093-32/+56
| | | | | | | | | | Test: builds Change-Id: If90975c8545158012bc6201acadd136363c21260
| * Run gofmtColin Cross2017-05-091-3/+3
| | | | | | | | | | Test: builds Change-Id: I2a873fc65cf414a14d6b58ef4ed6b9f5e57d75f7
* | Merge "Fix genrule documentation"Treehugger Robot2017-05-101-1/+1
|\ \
| * | Fix genrule documentationDan Willemsen2017-05-091-1/+1
| |/ | | | | | | Change-Id: Ieffb2bd7d739527db129ff300d8aec70d82c278a
* | Merge "soong: add __clang_analyzer__ to clang-tidy"Treehugger Robot2017-05-101-0/+4
|\ \
| * | soong: add __clang_analyzer__ to clang-tidyGeorge Burgess IV2017-05-041-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have code that acts slightly differently when the static analyzer is running, so that it can produce more accurate diagnostics (e.g. less false positives). It uses __clang_analyzer__ to detect the static analyzer. When the static analyzer is run via clang-tidy, __clang_analyzer__ doesn't get defined. Bug: None Test: WITH_TIDY=1 m. clang-tidy now acts as expected in code made for the static analyzer Change-Id: I460ff410640524633c0a60b71d34927d17e1ed98
* | | Merge "Work-around for crt objects with sdk: current"Treehugger Robot2017-05-104-2/+11
|\ \ \
| * | | Work-around for crt objects with sdk: currentJayant Chowdhary2017-05-094-2/+11
| | | | | | | | | | | | | | | | | | | | | | | | Test: convert libcups from Android.mk->Android.bp, mm -j64. Change-Id: I4fddcfa5806d0ce27a229c37a078b0d93e9082a8
* | | | Merge "Add bootable/recovery to root.bp"Treehugger Robot2017-05-101-0/+1
|\ \ \ \ | |_|_|/ |/| | |
| * | | Add bootable/recovery to root.bpSteven Moreland2017-05-091-0/+1
| |/ / | | | | | | | | | | | | Test: soong finds things in bootable/recovery Change-Id: I88f012ab344046d2307a6f08dc618ad424fe870e
* | | Merge "androidmk: support cfi, diag, and misc_undefined properties"Colin Cross2017-05-092-66/+46
|\ \ \ | |/ / |/| |
| * | androidmk: support cfi, diag, and misc_undefined propertiesColin Cross2017-05-082-66/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support more sanitize related properties. Also remove integer, since it isn't a supported property, and rewrite it to be deterministic by not going through splitBpList. This reapplies Ie8fba06c36fc121912c65a77ee495ceef4962df0. Bug: 37547301 Test: androidmk_test.go Change-Id: Id637ffdb7d36bed8a2c2fe76bd6f54662709e3b3
* | | Soong: Change Sanitize.Recover orderAndreas Gampe2017-05-081-5/+5
|/ / | | | | | | | | | | | | | | | | | | Add Sanitize.Order cflags after the general sanitizer options, so it can override the global -fno-sanitize-recover=all. Bug: 38128476 Test: m SANITIZE_HOST=address Test: manual test Change-Id: Ied5600507b66bd25d9d2a8bba428162ed4eb01e4
* | Declare dependency on sanitizer runtime librariesColin Cross2017-05-083-2/+21
| | | | | | | | | | | | | | | | | | Moving the last users of a sanitizer runtime library to soong was causing the runtime library to stop getting installed. Declare the dependency so make keeps installing it. Test: builds Change-Id: Ieeb9ad5c04ac8df0d1a74239da393dac5cab2b03
* | Add product_sdk_version.cflags propertyColin Cross2017-05-081-0/+1
| | | | | | | | | | | | | | libwilhelm needs PRODUCT_SDK_VERSION=%d. Test: builds Change-Id: Ie3caf9e4fdcb8e241c8743b1675520e93cf20cae
* | Merge "Revert "androidmk: support cfi, diag, and misc_undefined properties""Colin Cross2017-05-082-58/+50
|\ \
| * | Revert "androidmk: support cfi, diag, and misc_undefined properties"Colin Cross2017-05-082-58/+50
| | | | | | | | | | | | | | | | | | This reverts commit a8688c465294dc0016c56855922238527d36cc1b. Change-Id: Ida4050f3ddc8303d614f2bd2a01470ed5459fe2c
* | | Merge "Revert "Ensure environment dependencies are correct""Dan Willemsen2017-05-082-17/+1
|\ \ \
| * | | Revert "Ensure environment dependencies are correct"Dan Willemsen2017-05-082-17/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit e78473c33ce9b4a827e814229836d136581ad80e. Reason for revert: Broke mac builds, which rely on $PATH to find xcrun Change-Id: I58711b94d472ab828823afaf53394a1aa869f3f5
* | | | Merge "Ensure environment dependencies are correct"Dan Willemsen2017-05-082-1/+17
|\| | |
| * | | Ensure environment dependencies are correctDan Willemsen2017-05-052-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Store the os Environment on init, then clear it so that it's only available via a call to Config.Getenv(). That way we're guaranteed to know about all dependencies. Test: Add some os.Getenv/Config.Getenv calls and ensure correct behavior Test: cs/ shows no os.Getenv / os.LookupEnv / os.ExpandEnv calls that would be affected Change-Id: Ic519e0c871f340e752df03b8af0599d47c1d91d8
* | | | Merge changes I7270fef0,Ie8fba06cTreehugger Robot2017-05-083-51/+62
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | * changes: Automatically add frameworks/rs includes to modules with .rs files androidmk: support cfi, diag, and misc_undefined properties
| * | | | Automatically add frameworks/rs includes to modules with .rs filesColin Cross2017-05-081-1/+4
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | C++ sources generated from .rs files need includes in frameworks/rs and frameworks/rs/cpp. Add them automatically instead of requiring modules to add them manually. Test: builds Change-Id: I7270fef0b36c1956475d6d83ab8aa2e3c362e688
| * | | androidmk: support cfi, diag, and misc_undefined propertiesColin Cross2017-05-082-50/+58
| |/ / | | | | | | | | | | | | | | | | | | | | | Support more sanitize related properties. Bug: 37547301 Test: androidmk_test.go Change-Id: Ie8fba06c36fc121912c65a77ee495ceef4962df0
* / / Ignore disabled modules when generating ndk sysrootDan Willemsen2017-05-051-0/+4
|/ / | | | | | | | | | | | | | | | | | | | | When linux_bionic is turned on, there are two host modules for the preprocessed ndk headers, but only one is enabled. So exit early for the disabled one. Bug: 31559095 Test: Diff out/soong/build.ninja before/after, no change. Test: Turn on linux_bionic, no longer see a panic. Change-Id: I204a884a83ccf21c96088e97c19a0cffe029ec99
generated by cgit v1.2.3 (git 2.25.1) at 2024-06-27 09:47:24 +0000