- Feb 07, 2018
-
-
Treehugger Robot authored
-
Treehugger Robot authored
-
Treehugger Robot authored
-
Treehugger Robot authored
-
- Feb 06, 2018
-
-
Sundong Ahn authored
The module name of system_$(VER) prebuilt file is system_sdk_v$(VER). But this is inconsistent with sdk_v$(LOCAL_SDK_VERSION) in prebuilts/sdk. So, system_sdk_v$(VER) is changed to sdk_vsystem_$(VER) to use the same naming policy. Bug: 72031391 Test: build Change-Id: I3f18b7969dda208beeefdcaefadcc42db5c1577e
-
Ryan Prichard authored
-fsanitize=cfi requires LTO. Normally, the Clang driver automatically enables emutls, but it's broken with -flto, so work around it by passing -Wl,-plugin-opt=-emulated-tls. See https://buganizer.corp.google.com/issues/72706604#comment15 Enable the workaround with ctx.Device(), which is the same condition used in the existing I18acac41aac885fc6635fbd55f96ba7c845eb5e7 workaround. Bug: b/72706604 Bug: https://github.com/android-ndk/ndk/issues/498 Test: m libaudioflinger (on internal master); \ readelf -sW symbols/system/lib64/libaudioflinger.so | grep tlNBLogWriter check that __emutls_t.tlNBLogWriter exists check that no symbols of type TLS exist Change-Id: I2cf65574c52476843cc017ee176a7d6777e2ce0b
-
Stephen Hines authored
We can simply revert this patch to work on these warnings independently. Bug: http://b/72331526 Bug: http://b/72331524 Bug: http://b/72330874 Test: Build Change-Id: I9cec51578a1c6de33fb6f832c758b9916c61b735
-
Treehugger Robot authored
-
Colin Cross authored
Bug: 71715793 Test: m checkbuild Change-Id: Ibf78a5ec88e743f29504c91057d2cfced667c8b4
-
Jiyong Park authored
core_current is a pseudo SDK version which is a core Java API subset of the Android API. It is expected to be mainly used for external Java projects which are agnostic to Android; such as junit, guava, etc. A module built with this SDK version can only link to java modules of the same kind. It can't depend on modules built with other sdk (e.g. current) or without sdk. Bug: 72206056 Test: m -j Change-Id: I778e7b4fcb9456a12b418ffd633ea78e29951e84
-
Treehugger Robot authored
-
Colin Cross authored
Test: symbol_inject -i a.out -o a.out2 -s symbol -v value Change-Id: I16cd8facbae754f679bef07ab0ba23638286e1d7
-
Nan Zhang authored
Converting Doclava to Soong need tools.jar as one of its bootclasspaths. Test: m doclava Bug: b/70351683 Change-Id: I0ac65fcbe1eadfc57239cde2e83b80cf441a9b69
-
Stephen Hines authored
-
- Feb 05, 2018
-
-
Stephen Hines authored
-
- Feb 03, 2018
-
-
Stephen Hines authored
These files contain the flags and selection logic for our C/C++ toolchain. Test: N/A Change-Id: I860ea34c226ec01ae4127a18d80e1c8be6393a83
-
- Feb 01, 2018
-
-
Yi Kong authored
Bug: 68397894 Test: http://go/clang-4523590-testing Change-Id: Ief806250572fe2cc5f39b4c47ff74863acbbb26d
-
Yi Kong authored
Allow developers to disable LTO to reduce build time. Test: m checkbuild Test: DISABLE_LTO=true m checkbuild Bug: 62839002 Change-Id: I30642e3cfff60c4874a2f8797d08105b4e07cc6d
-
- Jan 30, 2018
-
-
Treehugger Robot authored
-
Dan Willemsen authored
-
- Jan 29, 2018
-
-
Nan Zhang authored
originally: .* -> /**/*.class now: .* -> /*.class .** -> /**/*.class Also add NinjaAndShellEscape to allow filtering inner classes with $ in the name. Test: unittests. Bug: b/71889972 Merged-In: Ifb8d7daa22bee02346885171edb15180af18f0de Change-Id: Ifb8d7daa22bee02346885171edb15180af18f0de (cherry picked from commit c61066684d5c8ddf600af602852c32fce6e6c6f4)
-
Pirama Arumuga Nainar authored
Bug: http://b/72642679 Store missing profile files and the modules that refer to them in the SOONG_MODULES_MISSING_PGO_PROFILE_FILE variable passed to Make. The contents of this variable will be written to $DIST_DIR/pgo_profile_file_missing.txt as part of the 'dist' target. Test: 'm dist' and verify creation of pgo_profile_file_missing.txt. Change-Id: I237cd0398be418be2b7db6fa65ece5ef347ecbc1
-
- Jan 27, 2018
-
-
Dan Willemsen authored
The prebuilts and defaults mutators start adding some dependencies, so for them to be able to reference imported modules, we need to run the namespace mutator earlier. Test: m nothing Test: Try to use a global defaults module from a namespace Change-Id: I6e853d20e32251d0fd33c8b2dcc26c0695b808fd
-
- Jan 26, 2018
-
-
Treehugger Robot authored
-
- Jan 25, 2018
-
-
Dan Willemsen authored
Bug: 38177569 Test: EXPERIMENTAL_USE_OPENJDK9=true m nothing check out/soong/Android-aosp_arm.mk Change-Id: If35e913c7203c12f4305e74919c0df76a8443253
-
Stephen Hines authored
Bug: http://b/68397894 Test: Build on darwin with new toolchain. Change-Id: I4d81efdc84d0b6f07482206430252ea63a4b081b
-
- Jan 24, 2018
-
-
Jaekyun Seok authored
Bug: 64195575 Test: succeeded building product.img with BOARD_PRODUCTIMAGE_PARTITION_SIZE, BOARD_PRODUCTIMAGE_FILE_SYSTEM_TYPE and PRODUCT_PRODUCT_VERITY_PARTITION. Change-Id: Icc4f8c16bc389fe20db680849f311d02df1299c3
-
Treehugger Robot authored
-
Dan Willemsen authored
We were previously setting GOROOT to "prebuilts/go/linux-x86" during the ninja executions when we were running Soong. But we can also run Soong during the main ninja execution, were GOROOT was unset. When the GOROOT was unset, the default GOROOT in our Go installation is "./prebuilts/go/linux-x86" (note the extra ./). This would cause g.bootstrap.goRoot to change between some soong runs, causing us to rebuild all go programs (and anything depending on them) more often than necessary. So instead, keep GOROOT undefined when running Soong. Everything that matters is using runtime.GOROOT(), which will fall back to the default. Continue setting $GOROOT for bootstrap.bash, otherwise it fails when there is no system provided go binary. What we give bootstrap.bash doesn't really matter, since we don't actually use the blueprint wrapper in Android. Test: m blueprint_tools; touch bionic/libc/tzcode/new.c; m blueprint_tools <doesn't rebuild everything> Change-Id: I82f30c7c3b5d25e5cbf28fe37a97fdb776c4a164
-
Andreas Gampe authored
Add support for an errorprone block in Java modules that accepts additional javacflags for the errorprone build. Sample: errorprone: { javacflags: ["-Xep:EqualsNaN:WARN"], }, Bug: 72004718 Test: m RUN_ERROR_PRONE=true Test: manual - add block with ERROR to a project Test: androidmk_test Change-Id: I502248fe76c26aa19102f413af72a7324c35b7f4
-
- Jan 23, 2018
-
-
Justin Yun authored
For the VNDK prebuilt modules that does not match the target arch, skip installing the module instead of marking the module to prevent installing. Bug: 72310137 Bug: 71787263 Test: Install VNDK snapshot v27 lunch aosp_arm64_ab-userdebug; m vndk_v27_arm64 - vndk libs must be installed m vndk_v27_arm - no vndk libs must be installed because target does not match OUT_DIR=out_clean m --skip-make Change-Id: I9df25d90c276ce5e0d94ec7f9bee32f9ce7231df
-
Pirama Arumuga Nainar authored
Bug: http://b/72343691 Change https://android-review.googlesource.com/c/572758, in addition to dependencies from the compiler flags, also marked all exported dependencies as implicit. This can cause lots of unnecessary recompiles. This change moves exported dependencies back as order-only dependencies. Test: 1. mma in art after changing profile_compilation_info.h triggers only a limited number of recompiles. 2. verify that changes to PGO profile files trigger recompiles. Change-Id: Icb0f4cd2b6da0add3b6e5206661e6aa7a577602f
-
Pirama Arumuga Nainar authored
Bug: http://b/63768402 If ANDROID_PGO_INSTRUMENT has "all" or "ALL", all PGO-enabled modules are built for profile generation. Test: Build with and without 'all' in ANDROID_PGO_INSTRUMENT Change-Id: I3b1a9b562775e80a4ab3965100341d9e8e4ffde9
-
Treehugger Robot authored
-
Logan Chien authored
This commit adds `extends: "name"` property and provides basic support to VNDK extensions. This is the simplest example: ``` cc_library { name: "libvndk", vendor_available: true, vndk { enabled: true, }, } cc_library { name: "libvndk_ext", vendor: true, vndk: { enabled: true, extends: "libvndk", }, } ``` A vndk extension library must extend an existing vndk library which has `vendor_available: true`. These two libraries must have the same `support_system_process` property. VNDK-ext libraries are installed to `/vendor/lib[64]/vndk` and VNDK-SP-ext libraries are installed to `/vendor/lib[64]/vndk-sp` by default. If there is a matching abi-dumps in `prebuilts/abi-dumps`, `header-abi-diff` will be invoked to check for ABI breakages. Bug: 38340960 Test: lunch aosp_walleye-userdebug && make -j8 # runs unit tests Test: lunch aosp_arm-userdebug && make -j8 # build a target w/o VNDK Test: Create a lsdump for a vndk lib, add an exported API to vndk lib, and build fails as expected. Test: Create a lsdump for a vndk lib, create an vndk extension lib with extra API, and build succeeds as expected. Test: Create libutils_ext, add an extra function to libutils_ext, and call it from a HIDL service. Change-Id: Iba90e08848ee99814405457f047321e6b52b2df0
-
Treehugger Robot authored
-
- Jan 22, 2018
-
-
Dan Albert authored
Test: build-ndk-prebuilts.sh with libc/libm included in the NDK Bug: https://github.com/android-ndk/ndk/issues/272 Change-Id: Ic398f541036f6c16f7dc58b9af984c60ed948495
-
- Jan 20, 2018
-
-
Treehugger Robot authored
-
Treehugger Robot authored
-
Ivan Lozano authored
This reverts commit 17df3c1b. Change-Id: Id5dab777da591c24113c3048b8606547a661afdd
-