Commit graph

62114 commits

Author SHA1 Message Date
Paul Duffin
b4c2b782d0 Merge "Support running bp2build tests in fixtures" am: de47ec777f
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2281252

Change-Id: I31483e54acadf30b2444ef127ebf2b5f9cd6fa3e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-02 16:34:12 +00:00
Paul Duffin
de47ec777f Merge "Support running bp2build tests in fixtures" 2022-12-02 15:56:33 +00:00
Parth Sane
22f2f9f372 Merge "Add support for AIDL traces in Priv apps" am: 0da806fe7a
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2311647

Change-Id: I7a3407cf00910481d759d6dc290b9434a18ccac6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-02 15:25:09 +00:00
Parth Sane
0da806fe7a Merge "Add support for AIDL traces in Priv apps" 2022-12-02 15:02:38 +00:00
Treehugger Robot
1ed3a7a0f7 Merge changes Ia4016240,Iba586155 am: a36589c4ca
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2309617

Change-Id: I14b21791d36601bc832c382391b003a1e9b4cd71
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-02 11:18:24 +00:00
Paul Duffin
4c0765a669 Support running bp2build tests in fixtures
Previously, the fixture test infrastructure was hard coded to
initialize itself in preparation for invoking the PrepareBuildActions()
method. That meant it could not be used for testing the bp2build and
apiBp2build modes which required different initialization and called a
different method.

This change extracts that behavior into a FixtureTestRunner and adds an
implementation that allows it to test the above two modes. It then uses
that to implement the runBp2BuildTestCaseWithSetup method.

The TestPrebuiltLibraryAdditionalAttrs was the only test which broke as
it supplied an invalid path to the MockFS (it does not support using a
trailing / to represent an empty build directory). So, it was modified
to add a couple of files inside those directories instead.

Test: m nothing
Change-Id: I6798a4f761160af8d1bfed81d46de9628bda3eb9
2022-12-02 11:14:17 +00:00
Treehugger Robot
a36589c4ca Merge changes Ia4016240,Iba586155
* changes:
  Allow choosing which soong_build invocations to run in the debugger
  Show module type in a module section of the Android-TARGET.mk
2022-12-02 10:36:07 +00:00
Jingwen Chen
9867290c71 Merge "Convert apex_available (for supported modules) to bazel tags." am: 8ec23f846d
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2321580

Change-Id: I813778da920431fe7a46384782ffbfc19e435444
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-02 07:41:55 +00:00
Jingwen Chen
8ec23f846d Merge "Convert apex_available (for supported modules) to bazel tags." 2022-12-02 07:22:10 +00:00
Sasha Smundak
4cbe83ad5e Allow choosing which soong_build invocations to run in the debugger
When `SOONG_DELVE_STEPS` enviromnment variable is set in addition
to `SOONG_DELVE`, only given soong_build invocations ('steps') are
run in the debugger.
Also, make bootstrapBlueprint code table-driven.

Test: treehugger
Change-Id: Ia4016240ca9e88c2a85c6d06851c5bba30e7b2b5
2022-12-01 21:12:11 -08:00
Sasha Smundak
5c4729df93 Show module type in a module section of the Android-TARGET.mk
The first line of each section will be
```
include $(CLEAR_VARS)  # <module type>
...
```

Bug: 257037252
Test: treehugger
Change-Id: Iba586155b682fe4e1e5817d8397eda8f9d9c8789
2022-12-01 21:12:11 -08:00
Kiyoung Kim
323f62701b Rename export_headers of cc_api_variant am: 62ed3dd5f8
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2324813

Change-Id: I80d71ed7fb64b76c50a69f14b7f7fe32434cad16
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-02 04:46:43 +00:00
Kiyoung Kim
62ed3dd5f8 Rename export_headers of cc_api_variant
Rename export_headers of cc_api_variant as export_include_dirs, because
the property only supports header directory.

Bug: 244244438
Test: Cuttlefish vendor build succeeded
Change-Id: I6e1468f51694150c70d3b0def2917f0ba74f7ea0
2022-12-02 10:31:48 +09:00
Treehugger Robot
7860257d28 Merge "Remove workaround for -fexceptions and stack MTE" am: a8ad378abc
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2322242

Change-Id: Ifb154653db358f38d488d27479aaec96de70355f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-02 01:23:32 +00:00
Treehugger Robot
a8ad378abc Merge "Remove workaround for -fexceptions and stack MTE" 2022-12-02 00:52:41 +00:00
Treehugger Robot
cc701fff9c Merge "Allow hwasan variant for vendor_snapshot_static modules" am: 6721ab5d40
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2314109

Change-Id: I5256ece2e93fca05c04e45ba00a5d1d04ef8f137
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-02 00:49:11 +00:00
Treehugger Robot
33738d9c0e Merge "Print error code when bazel invocation fails" am: 591c5ee0fb
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2323957

Change-Id: I9cac27c590dbd38637a7983aaae26b1516dad57c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-02 00:49:00 +00:00
Treehugger Robot
6721ab5d40 Merge "Allow hwasan variant for vendor_snapshot_static modules" 2022-12-02 00:27:22 +00:00
Treehugger Robot
591c5ee0fb Merge "Print error code when bazel invocation fails" 2022-12-02 00:10:02 +00:00
Treehugger Robot
a2a80fe6df Merge "soong: add os name to cc_genrule" am: 5195178376
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2322616

Change-Id: I84228714ba317d69272279e82d74f15d0c1a4858
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 23:42:47 +00:00
Treehugger Robot
5195178376 Merge "soong: add os name to cc_genrule" 2022-12-01 22:55:33 +00:00
Liz Kammer
f9420ac643 Merge changes Ia165e4be,I63fe74fb am: f843436455
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2260886

Change-Id: I31b85e9060f26b168a66ade0bea8be9ff7c4e745
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 22:36:57 +00:00
Florian Mayer
06aca9b448 Remove workaround for -fexceptions and stack MTE
The upstream change was merged in https://r.android.com/2322236

Bug: 174878242
Change-Id: Ia23d4a9d6ef54ed2c286e0b1becec1f35df75160
2022-12-01 14:23:23 -08:00
Liz Kammer
f843436455 Merge changes Ia165e4be,I63fe74fb
* changes:
  Update sanitize to not modify user properties
  Extend sanitize_test.go
2022-12-01 22:10:18 +00:00
Sasha Smundak
0e87b186cb Print error code when bazel invocation fails
Also, fix compiler warnings about deprecated ioutil.WriteFile/ReadFile
and about redundant variables

Test: treehugger
Change-Id: Iaf8f89b0e991dae8c5da403d4a1ec3a270f1ae29
2022-12-01 13:37:48 -08:00
Treehugger Robot
86cf825146 Merge "Limit propagating san config of shared to fuzzer" am: b8ae9cd936
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2279408

Change-Id: Ic0c66fef7088ef184e0cf0f999e04656b0d22459
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 20:55:08 +00:00
Treehugger Robot
b8ae9cd936 Merge "Limit propagating san config of shared to fuzzer" 2022-12-01 20:33:50 +00:00
Patrick Rohr
5307b3c790 soong: add os name to cc_genrule
While converting from gn to bp we stumbled upon genrules that have different cmd's for different targets. cc_genrule supports different architectures, but it does not give us any information about the OS types (some genrules are use for both host and android targets).

Change-Id: Ida531ad8b59d87a91d6faedbfe8fee9f75640bd1
Test: TH
2022-12-01 20:01:03 +00:00
Usta (Tsering) Shrestha
943de16b6b Merge "metric namespace for symlink_forest" am: c61e3c2789
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2320458

Change-Id: Ic8c32180a1d1273ba7d8a4c72499f53d7720a07d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 19:48:24 +00:00
Usta (Tsering) Shrestha
c61e3c2789 Merge "metric namespace for symlink_forest" 2022-12-01 19:18:39 +00:00
Kevin Dagostino
2090f1104c Merge "Use existing BUILD files for test-related tools." am: dfc72a8ca2
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2309887

Change-Id: Iad01bbdf1329e24b890b456fd01bad286d953df6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 08:04:04 +00:00
Kevin Dagostino
dfc72a8ca2 Merge "Use existing BUILD files for test-related tools." 2022-12-01 07:32:59 +00:00
Jingwen Chen
c4c34e1eab Convert apex_available (for supported modules) to bazel tags.
The tags will then be read by an aspect applied from apex attributes to
validate that the deps are explicitly opt-in to the appropriate apex.

The semantics of this is the same as the Soong apex_available property.

This also adds the "base_apex_name" attr to override_apex conversion so
that apex_available checks continue to work on the non-Google and non-Go
apex names.

Test: presubmits
Fixes: 218841706
Change-Id: I19b3637da9aa47573b252d74f62b601fbdc3784d
2022-12-01 06:25:25 +00:00
Treehugger Robot
d2a86c728d Merge "Support testing for resource shrinking" am: cdea74d96c
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2235281

Change-Id: I97a324800dff7802286b38dc97412f624a951635
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 06:25:20 +00:00
Treehugger Robot
cdea74d96c Merge "Support testing for resource shrinking" 2022-12-01 06:09:29 +00:00
Usta (Tsering) Shrestha
93b2a9b99f metric namespace for symlink_forest
Change-Id: Ifaa734a222bf37bccc3a74724f597f2d55200d6b
Test: run a build and inspeck event keys in bp2build_metrics.pb
Bug: 260936908
2022-12-01 06:00:27 +00:00
Hsin-Yi Chen
42b1ab1f5e Merge "Deprecate ABI reference dumps in gzip" am: 365dcdf655
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2321584

Change-Id: I09906e29b58c6d99f1cc37c7413b86c290d7d9d6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 05:52:24 +00:00
Hsin-Yi Chen
365dcdf655 Merge "Deprecate ABI reference dumps in gzip" 2022-12-01 05:18:05 +00:00
Treehugger Robot
ddb524dc7b Merge "Reland "Update clang version to clang-r475365"" am: c6bea2d26e
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2322236

Change-Id: If791b62e2110711df694d0be36570292b8ed15b2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 04:48:02 +00:00
Treehugger Robot
c6bea2d26e Merge "Reland "Update clang version to clang-r475365"" 2022-12-01 04:08:08 +00:00
Kiyoung Kim
d71c54b443 Merge "Support NDK variant of cc_api_variant" am: 48c4b601b9
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2319690

Change-Id: I01556220aef6ababbed618c7297036d7b60548a1
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 03:09:37 +00:00
Kiyoung Kim
48c4b601b9 Merge "Support NDK variant of cc_api_variant" 2022-12-01 02:45:33 +00:00
Hsin-Yi Chen
b07cb4e881 Merge "Refactor the build rules for ABI diff" am: 09a88fa528
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2306142

Change-Id: Ic91b798f6aff0b900f7bde1f2eb8019b06d8a3fb
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 02:34:09 +00:00
Chih-hung Hsieh
7dfa0b166d Merge "Reland "Enable sizeof-array-div warning"" am: dd8eaf8dbe
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2322303

Change-Id: I7790e1ba8363bd13f702a8dde1197d1fd9253157
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 02:33:57 +00:00
Hsin-Yi Chen
09a88fa528 Merge "Refactor the build rules for ABI diff" 2022-12-01 02:21:07 +00:00
Chih-hung Hsieh
dd8eaf8dbe Merge "Reland "Enable sizeof-array-div warning"" 2022-12-01 02:00:58 +00:00
Chih-hung Hsieh
ecd88699fa Reland "Enable sizeof-array-div warning"
This reverts commit 6be24baec4.

Reason for revert: warnings fixed in source files

Change-Id: I223ff6343c41ac957485726a3de006a84b9369df
2022-12-01 01:02:10 +00:00
Treehugger Robot
76c0d4cf9b Merge "Change condition to match go/thirdpartylicenses" am: b0e1044222
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2313438

Change-Id: I51818a9b8ef92d3b75fecc4fb771ad3ba2d1dd49
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-11-30 22:42:20 +00:00
Treehugger Robot
b0e1044222 Merge "Change condition to match go/thirdpartylicenses" 2022-11-30 22:10:50 +00:00
Yabin Cui
29f248b086 Reland "Update clang version to clang-r475365"
This reverts commit a558be6472.
Reason for revert: broken build was fixed

Bug: 253033919
Test: build
Change-Id: If6adf2ece00081eb2eeec7895461293751f94be1
2022-11-30 13:34:35 -08:00