aboutsummaryrefslogtreecommitdiffstats
path: root/ui
Commit message (Collapse)AuthorAgeFilesLines
* Allow remote execution of link actions.Ramy Medhat2020-04-231-0/+1
| | | | | | | | | | This CL adds a remoteexec package that allows adding a configurable RBE prefix to the template. Change-Id: I6b4ea773f6bae5706415a1830f822db7decca92f Test: built aosp crosshatch userdebug with and without RBE_CXX_LINKS. Merged-In: Ica920c3d7f79f2996210b9cbd448126451c1707c Bug: b/154820732
* Merge "Always keep rsp files" into rvc-devAnton Hansson2020-04-011-0/+1
|\
| * Always keep rsp filesDan Willemsen2020-03-271-0/+1
| | | | | | | | | | | | | | | | | | It's painful to debug without these. Bug: 151160048 Test: do a build; find out -name '*.rsp' Change-Id: I6fc25095442178c0bcbc1dea444ba51e9259c600 Merged-In: I6fc25095442178c0bcbc1dea444ba51e9259c600
* | Merge "Rename VTS to VTS10" am: 7182c45fe7 am: 4058983c0c am: 8989420fee am: ↵Dan Shi2020-03-311-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | 2d2a3643f5 Bug: 151896491 Test: none Change-Id: Iee9324c4edefef56171ecfac64fd11a9595e5f7c Merged-In: Iadad4ba96bf9a7f3b1d12c9d4eeb90fb1ed0841d Merged-In: Iee9324c4edefef56171ecfac64fd11a9595e5f7c (cherry picked from commit ee3ec387b0ca2d3ecb0df63cb25840874c295a1e)
* | Simplify message around nsjail errorDan Willemsen2020-03-281-4/+1
|/ | | | | | | | | | | | | We have known problems (docker; distros w/o user namespaces) which mean that we won't be turning this into a fatal error anytime soon. Also remove the bug report link, it's not useful to continue getting the same reports. Bug: 123210688 Test: treehugger Change-Id: I271871d68150417ac938f074d3730cad4518e327 Merged-In: I271871d68150417ac938f074d3730cad4518e327 (cherry picked from commit 1871d88e4f443494d936b5cbe79b01b25167a8ce)
* Cleanup vendor_boot artifacts in PRODUCT_OUT on installcleanWill McVicker2020-03-191-0/+3
| | | | | | | | | | | This makes sure that we don't keep old vendor_boot artifacts around after a clean. Signed-off-by: Will McVicker <willmcvicker@google.com> Fixes: 151957122 Bug: 151949371 Test: touch $OUT/vendor-ramdisk/test-file && make installclean Change-Id: I357929583f234212a998dceaa31506fce42e0255
* Stop using the host python3Dan Willemsen2020-03-071-22/+21
| | | | | | | | | | We've now got a prebuilt in prebuilts/build-tools Bug: 123089883 Test: treehugger Change-Id: Ic0fb09c8970f3a1c9f3b7b0303168520dcbfc3d8 Merged-In: Ic0fb09c8970f3a1c9f3b7b0303168520dcbfc3d8 (cherry picked from commit 669f769470ff4ac4c97b6dac3d07c6c32343e44a)
* Add CCACHE_DIR to allowed environment variablesJohn Eckerdal2020-02-181-0/+1
| | | | | | | | | | Trying to override the default directory for ccache by using the CCACHE_DIR environment variable fails unless it is in the allowed list. Bug: 149670916 Test: manual Change-Id: I8e7eea7a5c25d7ea5f0956fafc70d62522f3c4fc
* Remove empty directories when removing previously installed filesDan Willemsen2020-02-131-0/+15
| | | | | | | | | | | Apparently PackageManager gets confused when the apk disappears, but the directory still exists. Test: lunch aosp_arm-eng; m BasicDreams <remove BasicDreams from PRODUCT_PACKAGES> m nothing ls out/target/product/generic/system/app Change-Id: I486c0ddadde55fad226049d3fb0f09af88117da0
* Merge "Add additional variables to ninja variable whitelist"Treehugger Robot2020-01-281-0/+2
|\
| * Add additional variables to ninja variable whitelistKousik Kumar2020-01-281-0/+2
| | | | | | | | | | | | | | | | | | | | | | These variables control whether we accept cached-results / not and whether we should update the cache with locally executed results or not. I need accpet-cached whitelisted to purge the the RBE cache for the invalid cache entry we have set for the failure we say yesterday. Bug: b/148387048 Change-Id: I7344fc083f82e0b7bc11084376a267d19cf30bb8
* | Add support for experimentally enabling RBE support on specific rules.Ramy Medhat2020-01-282-0/+44
|/ | | | | | | | This CL adds RBE support to javac, r8, and d8 rules which is only enabled if respective environment variables are set. Test: an aosp_crosshatch build with and without the new variables. Change-Id: Ic82f3627944f6a5ee7b9f3228170c2709b1bfcb8
* Merge "Create a highmem pool and put metalava into it"Treehugger Robot2020-01-255-10/+52
|\
| * Create a highmem pool and put metalava into itColin Cross2020-01-245-10/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Create a highmem pool based on the total RAM and the number of CPUs, with an override via the NINJA_HIGHMEM_NUM_JOBS variable. Put metalava into the highmem pool. Ninja does not support nested pools, and when goma or RBE is enabled the maximum ninja parallelism is set very high with local jobs in a local pool. When both the local pool and highmem pool are enabled, the total number of local jobs will be as high as the sum of the sizes of the two pools. Keep the highmem pool limited to 1/16th of the local pool when remote builds are enabled to try to minimize the effect while still limiting highmem jobs. Fixes: 142644983 Test: m nothing, examine pools Test: m USE_GOMA=true nothing, examine pools Change-Id: Id79f11f44948992960ac34ecf831dacbe21bd332
* | Merge "Add total build time to metrics"Treehugger Robot2020-01-174-58/+78
|\ \ | |/ |/|
| * Add total build time to metricsColin Cross2020-01-164-58/+78
| | | | | | | | | | | | | | | | | | | | | | | | Determining the total build time from the existing metrics is tricky because some of the metrics are overlapping. For example, the "soong" metric includes "blueprint bootstrap", "environment check", "minibp", "bpglob", "minibootstrap", and "bootstrap". Create a new singleton "total" metric that covers the interesting parts of the build. Test: m nothing Change-Id: Iedcf5c9cf0d27252b694d35e6ff66ca1fcf734ac
* | Merge "Disallow system m4"Treehugger Robot2020-01-151-1/+0
|\ \ | |/ |/|
| * Disallow system m4Dan Willemsen2019-06-191-1/+0
| | | | | | | | | | | | | | | | | | Everything should be using the prebuilt now. Bug: 117561006 Test: treehugger Test: check bass logs for m4 uses Change-Id: Ic69d715d72d64bb0082e2f2c247113155d8b3de0
* | Merge "Switch to toybox gzip."Elliott Hughes2020-01-131-2/+0
|\ \
| * | Switch to toybox gzip.Elliott Hughes2019-12-201-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | (Although I was the one who added the gzcat reference, I don't believe we actually use it. And neither my Debian host nor toybox has gzcat -- as opposed to zcat -- anyway. So just quietly drop that.) Test: treehugger Change-Id: I95d94dab1daf72faedfa170770e8ebfaa37b70c6
* | | Merge "Sandbox soong_build by changing to root directory"Treehugger Robot2020-01-111-0/+1
|\ \ \
| * | | Sandbox soong_build by changing to root directoryColin Cross2020-01-111-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This relands I12a0f907753fefd1997ab8b4ea2ac331234093cf along with a fix to blueprint for absolute paths. Store the current working directory and then change to the root directory so that all file accesses must go through helpers in the android package that properly track dependencies. Change-Id: I24ac485677aa102eec1a2521d16820da6ee1ae77 Fixes: 146437378 Test: m checkbuild Test: m OUT_DIR=/tmp/out nothing
* | | | Merge "Revert "Sandbox soong_build by changing to root directory""Colin Cross2020-01-101-1/+0
|\| | |
| * | | Revert "Sandbox soong_build by changing to root directory"Colin Cross2020-01-101-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 05c25ccb4adb5329add700b533416c226cdbfa96. Reason for revert: broke absolute OUT_DIR Bug: 146437378 Change-Id: I523ed79d40e1c1ef040212ba794a7a084abea75d
* | | | Merge "Sandbox soong_build by changing to root directory"Colin Cross2020-01-101-0/+1
|\| | |
| * | | Sandbox soong_build by changing to root directoryColin Cross2020-01-091-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Store the current working directory and then change to the root directory so that all file accesses must go through helpers in the android package that properly track dependencies. Fixes: 146437378 Test: m checkbuild Change-Id: I12a0f907753fefd1997ab8b4ea2ac331234093cf
* | | | Add FLAG_compare to whitelisted environment variablesKousik Kumar2020-01-091-0/+1
|/ / / | | | | | | | | | | | | | | | | | | Bug: b/147419757 Test: Ran a compare build and tested that compare mode works again Change-Id: I743d5070673f3ec9d0c2b099019f88935f3894fc
* | | Allow EMMA_INSTRUMENT_FRAMEWORK during ninjaDan Willemsen2020-01-061-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | This is used by the art apex test. Bug: 147197813 Test: m EMMA_INSTRUMENT=true EMMA_INSTRUMENT_FRAMEWORK=true art-check-debug-apex-gen Change-Id: Id185cd35f16131f2c9a8afeba2e5b87834b0e620
* | | Likely fix SANITIZE_HOST=address buildsDan Willemsen2020-01-041-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | We're passing in ASAN_OPTIONS=detect_leaks=0 on the command line, and the build started failing after we limited the environment. Test: forrest of aosp-master/aosp_x86_64-eng Change-Id: I42c91897c7677e1a249412e5a8bc7bb1edb2f881
* | | Merge "Limit environment during ninja"Dan Willemsen2020-01-041-1/+3
|\ \ \
| * | | Limit environment during ninjaDan Willemsen2020-01-031-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See the change that added ALLOW_NINJA_ENV for more information. Test: m nothing; check out/soong.log for small list Test: ALLOW_NINJA_ENV=true m nothing; check out/soong.log Change-Id: I7761c6a07a7f8b0acee107e9c27c7739dd4b63ab
* | | | Don't print critical path parallelism ratio if elapsed time is 0.Colin Cross2020-01-041-2/+4
| | | | | | | | | | | | | | | | | | | | Test: m nothing, examine out/soong.log Change-Id: Ib23902c703d306337b2a5150d6d63b1e3d0f95b3
* | | | Merge "Remove files that should no longer be installed"Treehugger Robot2020-01-044-0/+196
|\ \ \ \ | |/ / / |/| | |
| * | | Remove files that should no longer be installedDan Willemsen2020-01-034-0/+196
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make now exports (partial) lists of what should be installed. Those lists are incomplete, but if something is removed from the list, we can remove it from the filesystem so that incremental builds work better. Test: built-in tests (m blueprint_tools) Test: remove a module from PRODUCT_PACKAGES, see the print and file removed Test: change the name of a cts test, see the old one removed from cts Change-Id: I6bc14c6a5af63383ea265e97ed8b875fa51af0ed
* | | | Add option to limit environment variables given to ninjaDan Willemsen2020-01-033-4/+79
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ninja does not track changes in environment variables, so we get potentially incorrect builds when environment variables change during incremental builds if some action was using one of them. Add a variable to limit exposure of these variables to ninja, and thus, all actions run by ninja. Kati and Soong can still read environment variables, they explicitly track which ones they read so that we can re-run them appropriately. This list is just the beginning, there's no good way to detect which environment variables are currently being used and to pass them through. So this initial change won't have a behavioral change, and we'll flip the switch and see what fails or who complains, flipping it off and on and adding to the list until we can make this always happen. Also adds a board-specific `BUILD_BROKEN_NINJA_USES_ENV_VARS := ...` list so that we can temporarily allow board-specific variables until they're fixed. Test: check out/soong.log Test: ALLOW_NINJA_ENV=false m nothing; check out/soong.log Test: set BUILD_BROKEN_NINJA_USES_ENV_VARS := OLDPWD ALLOW_NINJA_ENV=false m nothing; check out/soong.log Change-Id: I08e4834ce12100a577ef7d6a9a21b9e9d345cb93
* / / Add TotalRAM detectionDan Willemsen2019-12-275-0/+80
|/ / | | | | | | | | | | | | | | | | | | | | | | | | Colin originally wrote this for the highmem pool: https://android-review.googlesource.com/c/platform/build/soong/+/1168271 But while that's a larger change, this is useful to just be in the logs, and for use in multiproduct_kati to better limit the number of concurrent jobs. Test: check soong.log on linux and mac Change-Id: I0518d303a220d775f8d78dba9f660b2954e68e3e
* | Merge "Switch to toybox realpath."Elliott Hughes2019-12-201-1/+0
|\ \
| * | Switch to toybox realpath.Elliott Hughes2019-12-171-1/+0
| | | | | | | | | | | | Change-Id: I9f1d3df08d860e5bfa131a531e274c75b72148e7
* | | Merge "Switch the mac back to toybox xargs."Treehugger Robot2019-12-151-3/+0
|\ \ \
| * | | Switch the mac back to toybox xargs.Elliott Hughes2019-12-121-3/+0
| |/ / | | | | | | | | | | | | Test: make sdk Change-Id: I29a6b0404e248553ecfdfdeb263aa9cd0d2fdb3e
* / / Remove obsolete copy headersDan Willemsen2019-12-131-0/+39
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When LOCAL_COPY_HEADER lines are removed, we just removed the copy rule, we didn't actually remove the old header, so C/C++ files could still include the old headers during incremental builds. While we do consider LOCAL_COPY_HEADERS as near-obsolet, and it'll disappear as we move everything over to Soong, this could produce some unfortunate incremental build results while people are attempting to remove them. Fix this by ensuring that only the files currently listed in LOCAL_COPY_HEADERS exist in TARGET_OUT_HEADERS after we run Kati. Test: Remove a LOCAL_COPY_HEADERS entry; see the header removed Change-Id: I817305703a6996d50490d552623d7df019b608c9
* | Switch to our hermetic bc.Elliott Hughes2019-12-101-1/+0
| | | | | | | | | | Test: treehugger Change-Id: I12142a7f54190edd21e719bbd8f5df831ada6161
* | Add CUJ testsColin Cross2019-12-053-56/+183
| | | | | | | | | | | | | | | | | | Add a cuj_tests binary that runs through a sequence of critical user journey builds and collects metrics. Bug: 145688720 Test: cuj_tests Change-Id: I6bb1a3fc4a458fd8cea7b7c924f565d0cf3e6df3
* | Merge changes Id261b5a3,Ice6f34d0Treehugger Robot2019-12-053-15/+20
|\ \ | | | | | | | | | | | | | | | * changes: Fix errorProtoLog error messages Write BUILD_DATETIME_FILE after SetupOutDir
| * | Fix errorProtoLog error messagesColin Cross2019-11-261-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Use Printf instead of Println for formatted strings, and pass the filename instead of the proto. Test: cuj_tests Change-Id: Id261b5a34304a0caa61faa1f3bbc388aacdd25a6
| * | Write BUILD_DATETIME_FILE after SetupOutDirColin Cross2019-11-262-13/+18
| | | | | | | | | | | | | | | | | | | | | | | | Delay writing the BUILD_DATETIME_FILE until after the out directory has been created. Test: cuj_tests Change-Id: Ice6f34d003f93c26b5d2d0b64f92b11efe16c2d4
* | | Clean vts-core in host/out to ensure old package to be removed.Dan Shi2019-11-261-0/+1
|/ / | | | | | | | | | | Bug: None Test: m installclean Change-Id: I5aa451d87dc377e62dcf1b88baa58539fa661feb
* | Remove BASH_ENV from the build environmentDan Willemsen2019-11-181-0/+1
| | | | | | | | | | | | | | | | | | This runs an arbitrary shell script for all non-interactive bash sessions, and we've got lots of those. It was causing problems with Fedora 30, which triggers the `manpath` executable, which isn't available in our $PATH: https://groups.google.com/d/msg/android-building/WqEyZQuxYtY/zDXl9irOCAAJ Change-Id: If2f63bbbcc3accb8d239f93fa06823d301d69488
* | Merge "Switch to our own zipinfo(1)."Elliott Hughes2019-11-161-1/+0
|\ \
| * | Switch to our own zipinfo(1).Elliott Hughes2019-11-141-1/+0
| | | | | | | | | | | | | | | Test: treehugger Change-Id: If71e87e45a52959be9a316810fe79250771eb9d4
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-08 05:14:31 +0000