Commit graph

3807 commits

Author SHA1 Message Date
Jiyong Park
27e57d981e Merge "add exclude_shared_libs for vendor_available:true libs" am: d5696dd836
am: 939c02ee9a

Change-Id: I3bbd8b469196b6f21681fa05d71d01c8a994f591
2017-06-21 19:37:25 +00:00
TreeHugger Robot
094400150f Merge "Turn find emulator warnings into errors" into stage-aosp-master 2017-06-21 19:36:03 +00:00
Jiyong Park
939c02ee9a Merge "add exclude_shared_libs for vendor_available:true libs"
am: d5696dd836

Change-Id: Ib06214d951d00c9f6116c584bf410bbde110e5e1
2017-06-21 19:34:36 +00:00
Treehugger Robot
d5696dd836 Merge "add exclude_shared_libs for vendor_available:true libs" 2017-06-21 19:15:50 +00:00
Dan Willemsen
14b3f7f12b Turn find emulator warnings into errors
Test: build/soong/build_test.bash (multiproduct_kati)
Change-Id: Ieb65a457c8c1ed4bff44a01aa2cc11203172ba9b
Merged-In: I2f0338bd2b8d56717166c1805993efe1fa7f2819
(cherry picked from commit dc5b70f97c)
(cherry picked from commit e8a25d5577)
2017-06-21 17:44:59 +00:00
Dan Willemsen
eac5d59de1 Capture backtraces during build_test am: e7efcbe34e
am: a40ea06fd6

Change-Id: Ied4865cd34fd531824768319c818244e208cf12f
2017-06-21 17:20:13 +00:00
Dan Willemsen
a40ea06fd6 Capture backtraces during build_test
am: e7efcbe34e

Change-Id: I5f089eff83755722f535e73781b628ffc651803e
2017-06-21 17:16:42 +00:00
Dan Willemsen
e7efcbe34e Capture backtraces during build_test
We're seeing random ckati segfaults, and ASAN only hides the problem, so
attempt to capture more information.

Bug: 36182021
Test: build/soong/build_test.bash, kill -SIGSEGV <ckati>
Change-Id: I854ae846df0c47f69ec8d39b9a3d9e6f482e0854
2017-06-20 23:27:45 -07:00
Jiyong Park
44cf1a7779 add exclude_shared_libs for vendor_available:true libs
Adding a mechanism to conditionally exclude some shared library
dependencies when a lib is built for vendors.

Without this, some libraries cannot be earily marked as vendor_available
if they are depending on shared libs can shouldn't be marked as
vendor_available.

By using exclude_shared_libs with exclude_srcs (or __ANDROID_VNDK__
macro), we can eliminate the unnecessary dependency for vendors.

Bug: 62471389
Test: build

Change-Id: If94277b45c3769223cea371d0028e75277640356
2017-06-21 10:13:27 +09:00
Jeff Gaston
327ce4cfce Merge changes I0b67e89f,I60f02a8d am: 83cf1cee58
am: 5e640d701b

Change-Id: Icb09c447d8181d134884c21785fb70b5a33f9521
2017-06-20 19:42:23 +00:00
Jeff Gaston
5e640d701b Merge changes I0b67e89f,I60f02a8d
am: 83cf1cee58

Change-Id: I5dc95cedd553f3ac98778fee5762b38c937c5239
2017-06-20 19:38:54 +00:00
Jeff Gaston
83cf1cee58 Merge changes I0b67e89f,I60f02a8d
* changes:
  Revert "Revert "Run bpfix at the end of androidmk""
  Revert "Revert "Initial implementation of bpfix""
2017-06-20 19:29:41 +00:00
Jayant Chowdhary
2e617d5aff Black-list for clang LibTooling Cflags. am: 9677e8c6ad
am: 702e5f3d0e

Change-Id: I5eef5ba79844e7e7e7390ebb3e06b08f016cef78
2017-06-20 18:17:49 +00:00
Jayant Chowdhary
702e5f3d0e Black-list for clang LibTooling Cflags.
am: 9677e8c6ad

Change-Id: I95691c3eb46de2078d1c59b8abaa7ad5d27c19e2
2017-06-20 18:13:50 +00:00
Jayant Chowdhary
9677e8c6ad Black-list for clang LibTooling Cflags.
Add a list of flags which are not understood by clang LibTooling tools
and filter them out of the Cflags the tools are invoked with.

Test: In frameworks/av, make libmedia vendor_available (this invokes
header-abi-dumper on this module), mm -j64.

Bug: 62447349

Change-Id: I46f017212b89f4331145c999103d0ed44da0abaf
2017-06-19 19:43:03 -07:00
Jeff Gaston
c381fd0282 Revert "Revert "Run bpfix at the end of androidmk""
Bug: 38351765
Test: androidmk Android.mk > Android.bp

This reverts commit eb3186abbe.

Change-Id: I0b67e89f5096aae60c97eb6af4fd205509aac5c9
2017-06-19 15:52:25 -07:00
Jeff Gaston
aff66e55a9 Revert "Revert "Initial implementation of bpfix""
Bug: 38351765
Test: bpfix Android.bp

This reverts commit a8cc9c53fa.

Change-Id: I60f02a8dd920346aa17b9044f834ffe94fa693c6
2017-06-19 15:52:15 -07:00
Ivan Lozano
3efff5ffc2 Merge "Add support for misc_undefined diagnostics" am: f47b048782
am: 18795595aa

Change-Id: Ie4e2d32ce81881feb18d1e04713ce9c2f4a71fb5
2017-06-19 20:32:06 +00:00
Ivan Lozano
18795595aa Merge "Add support for misc_undefined diagnostics"
am: f47b048782

Change-Id: I6574621b22481a4d2dde5b95e4f2415cfb523faf
2017-06-19 20:26:35 +00:00
Ivan Lozano
f47b048782 Merge "Add support for misc_undefined diagnostics" 2017-06-19 20:16:48 +00:00
Ivan Lozano
651275ba90 Add support for misc_undefined diagnostics
Setting Sanitize.Diag.Undefined to true does not enable diagnostics for
all possible UBSan sanitizers that might be declared in Sanitize.Misc_undefined
(specifically unsigned-integer-overflow and nullability-*). This adds an
equivalent Sanitize.Diag.Misc_undefined string array to allow diagnostics to
be enabled in these sanitizers.

Bug: 62546512
Test: make works and device boots.
Test: Tested an Android.bp generated binary with the new field enabling
unsigned-integer-overflow diagnostics.

Change-Id: I9aa7cb2555ab4f94ae1aa290715a617718d936ff
2017-06-19 10:11:56 -07:00
Dan Willemsen
e8a25d5577 Turn find emulator warnings into errors
Test: build/soong/build_test.bash (multiproduct_kati)
Change-Id: Ieb65a457c8c1ed4bff44a01aa2cc11203172ba9b
Merged-In: I2f0338bd2b8d56717166c1805993efe1fa7f2819
(cherry picked from commit dc5b70f97c)
2017-06-17 00:28:09 +00:00
Chih-Hung Hsieh
81a6cb3295 Add OWNERS in build/soong
* Owners are selected from top CL approvals or owners.
  They will be suggested to review/approve future CLs.
* OWNERS files are recognized by the new find-owners plugin, see .md files in
  https://gerrit.googlesource.com/plugins/find-owners/+/master/src/main/resources/Documentation/

Test: build/make/tools/checkowners.py -c -v OWNERS
Change-Id: I5184ff6ddec602de238eb6bde033503fad1f8887
2017-06-15 13:49:57 -07:00
Yoshisato Yanagisawa
0ebcb38c4b Add JAVAC_WRAPPER support am: 572324ac33
am: bd8194497f

Change-Id: Ida34bbc4854a6bcbcbaa38e242dd8627c2d87c20
2017-06-14 03:12:13 +00:00
Yoshisato Yanagisawa
bd8194497f Add JAVAC_WRAPPER support
am: 572324ac33

Change-Id: I911414379160439d97896a8f79c04a5c19713db5
2017-06-14 03:07:43 +00:00
Yoshisato Yanagisawa
572324ac33 Add JAVAC_WRAPPER support
To run javac with AndroidGomaStaticRule, let me make JAVAC_WRAPPER
also show up to javac/builder.go.

Test: USE_GOMA=true JAVAC_WRAPPER=gomacc m -j32
Bug: b/62334576

Change-Id: I19e1e4315f71d8706800da810b5212f5964171a4
2017-06-14 01:04:50 +00:00
Dan Albert
ec0acf308f Merge "Require that NDK symbol files end with .map.txt." am: 2a60959b65
am: 79a9014c9b

Change-Id: Ic1f74682754f17fc1bf6166729c0eb7f3e4da270
2017-06-14 00:38:05 +00:00
Dan Albert
79a9014c9b Merge "Require that NDK symbol files end with .map.txt."
am: 2a60959b65

Change-Id: I77115c6b7f76906dd2ac65c87b774c8cb3a15a81
2017-06-14 00:34:03 +00:00
Treehugger Robot
2a60959b65 Merge "Require that NDK symbol files end with .map.txt." 2017-06-14 00:21:45 +00:00
Jeff Gaston
cf3e77bb4d Only check for ninja stuckness after it's been running for a bit am: a6697e8b7f
am: 41a88d181d

Change-Id: Id8f501af2cad0d87236eed40a25dc2063cd85f91
2017-06-14 00:02:30 +00:00
Jeff Gaston
41a88d181d Only check for ninja stuckness after it's been running for a bit
am: a6697e8b7f

Change-Id: I28d3b9d0f83aefbe8d7f9f2cb07a9c126276742b
2017-06-13 23:58:29 +00:00
Dan Albert
15be0c6a36 Require that NDK symbol files end with .map.txt.
We're making an NDK API Council +1 a requirment for changes to these
files, so we need to know that all symbol files will match a given
file pattern.

Test: Changed a symbol file to foo.txt, `make ndk` failed
Test: make ndk
Bug: http://b/35870541
Change-Id: I29690fb97746ad682e57845fea3f1114cfd77598
2017-06-13 15:14:56 -07:00
Jeff Gaston
a6697e8b7f Only check for ninja stuckness after it's been running for a bit
Bug: 62580037
Test: rm -rf out && m -j showcommands | grep "ninja may be stuck" || echo ok

Change-Id: I8ff1bd216b5f8349ce9e06e5465a9f8d0663f8c0
2017-06-13 14:15:02 -07:00
Jeff Gaston
09af8c537b Make absolute OUT_DIR work with sbox am: 193f2fb092
am: be95116215

Change-Id: I2ddcc757466e8e5608d5cdb3366fbe3c51ad91c6
2017-06-13 18:06:34 +00:00
Jeff Gaston
be95116215 Make absolute OUT_DIR work with sbox
am: 193f2fb092

Change-Id: I8df526750666e92e7c863a79e4ed8fad7ff64fc1
2017-06-13 18:02:34 +00:00
Jeff Gaston
193f2fb092 Make absolute OUT_DIR work with sbox
Test: make OUT_DIR=/tmp/abspath
Test: make OUT_DIR=relative-out
Bug: 35562758

Change-Id: I688d5f6117b194440c5d01c1040033b5671187c2
2017-06-12 16:36:17 -07:00
Nan Zhang
b128d27b65 Fix the issue that /tmp/soong_python_test* isn't being removed. am: aac67d38b2
am: 78a4dd0173

Change-Id: I3eb91407ea3f4cf9f45ec91206cf448d6241a1f6
2017-06-12 19:59:53 +00:00
Nan Zhang
78a4dd0173 Fix the issue that /tmp/soong_python_test* isn't being removed.
am: aac67d38b2

Change-Id: I348801f343318e3a7e135087282db75ae4f88665
2017-06-12 19:56:53 +00:00
Nan Zhang
aac67d38b2 Fix the issue that /tmp/soong_python_test* isn't being removed.
Bug: 62499851
Test: go test
Change-Id: I84bd66cd1410dd9fc6add3ba5626d8ec4fe702d7
2017-06-12 10:49:42 -07:00
Jeff Gaston
cf3f15d5d3 Easier debugging of sbox am: f49082afab
am: 195360b9c6

Change-Id: I1b772a054c17abc9f8de54f3ae5d014b6db6debc
2017-06-09 22:21:00 +00:00
Jeff Gaston
bdbcdda527 Have Soong try to enforce that genrules declare all their outputs. am: efc1b412f1
am: fe4bf0f9a7

Change-Id: Ib9276c8ed8c72103cc1cc6ead64c976cfaf22eb3
2017-06-09 22:20:57 +00:00
Jeff Gaston
195360b9c6 Easier debugging of sbox
am: f49082afab

Change-Id: I0a9dcc748966a7db457a108868c1d13046de0d0a
2017-06-09 22:17:59 +00:00
Jeff Gaston
fe4bf0f9a7 Have Soong try to enforce that genrules declare all their outputs.
am: efc1b412f1

Change-Id: I1130224e412109810ad6b10127aeebf5797431d6
2017-06-09 22:17:56 +00:00
Jeff Gaston
f49082afab Easier debugging of sbox
Cleanup of some failure messages
Also, this leaves the temp directory untouched if a declared output was not created

Bug: 35562758
Test: make

Change-Id: I8ef1315af80eb327752501f12a331dbdf52ba3e9
2017-06-09 17:57:33 +00:00
Jeff Gaston
efc1b412f1 Have Soong try to enforce that genrules declare all their outputs.
This causes Soong to put the outputs of each genrule into a temporary
location and copy the declared outputs back to the output directory.
This gets the process closer to having an actual sandbox.

Bug: 35562758
Test: make

Change-Id: I8048fbf1a3899a86fb99d71b60669b6633b07b3e
2017-06-09 17:57:18 +00:00
Dan Willemsen
63c18370f5 Merge "Add pom2mk, a maven repo -> Android.mk tool" am: 6b78fa8c01
am: 97e8052c6d

Change-Id: Id8a3f5c33f6223a2d27bb3da467b0a693c3736fc
2017-06-08 18:56:35 +00:00
Dan Willemsen
97e8052c6d Merge "Add pom2mk, a maven repo -> Android.mk tool"
am: 6b78fa8c01

Change-Id: I7c9da93aa590efef002c55011d66d45aaa03c200
2017-06-08 18:53:35 +00:00
Dan Willemsen
6b78fa8c01 Merge "Add pom2mk, a maven repo -> Android.mk tool" 2017-06-08 18:41:49 +00:00
Evgenii Stepanov
8acc3d9596 Don't add -lrt on Darwin for sanitized targets.
am: 3045195f56

Change-Id: I1bce53a06e8314e46f86f2da046d7fbe248713ce
2017-06-08 15:54:48 +00:00
Dan Willemsen
2902fa7148 Add pom2mk, a maven repo -> Android.mk tool
This is still fairly simplistic, but good enough to test some build
system changes.

Bug: 33381544
Test: run, inspect output
Change-Id: Ia5c19570493116dca01cb65605cdf20becf8c1d0
2017-06-07 23:25:16 -07:00