Commit graph

58223 commits

Author SHA1 Message Date
Treehugger Robot
ffd6014a68 Merge "Revert^2 "Promote ExtraText lint warning to an error""" 2022-08-12 06:43:08 +00:00
Yu Liu
5d866ed81a Move testing.go to srcs from testSrcs.
This is needed so that tests in other packages can depend on it.

Bug: 232520859
Test: Manually run m nothing to verify that all tests passed
Change-Id: Ief64e8d0e4030431b26cb5526c9f8beab94cad23
2022-08-11 13:00:00 -07:00
Cole Faust
d57e8b2c2a Add lint test property
Some libraries are only used for tests, but
are not test module types. These modules get warnings
about @VisibleForTesting usages when they really
shouldn't. Expose a test flag that module authors
can use to make lint treat a module as test code.

Bug: 235339747
Test: Manually tested applying it to SystemUI-tests
Change-Id: I1356749a669dc80a7725605d7159da27c9a211b4
2022-08-11 12:26:51 -07:00
Yu Liu
e1eada0b77 Merge "Support bp2build conversion of hidl_interface." am: 4b32269d0a
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2137952

Change-Id: I62a5d52b8de273fa3cb92cd3910d41964c47aa99
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-11 18:21:57 +00:00
Matthew Maurer
e3d32f2671 Merge "Revert^4 "rust: Only allow bindgen to produce rlibs."" am: 18fa8f641a
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2182575

Change-Id: I3c4725d139f3052aa31376da9b426be65e2abd68
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-11 18:21:21 +00:00
Yu Liu
4b32269d0a Merge "Support bp2build conversion of hidl_interface." 2022-08-11 18:15:58 +00:00
Matthew Maurer
18fa8f641a Merge "Revert^4 "rust: Only allow bindgen to produce rlibs."" 2022-08-11 17:50:17 +00:00
Cole Faust
2aa68684ba Revert^2 "Promote ExtraText lint warning to an error""
This reverts commit 0b29c62498.

Reason for revert: Issue that caused breakage has been fixed on stage-aosp-master. I'll test this cl on go/abtd for the stage-aosp-master branch before submitting.

Change-Id: I90f1941f67ebf9c40402713fa7762a99e368c17f
2022-08-11 17:21:45 +00:00
Alan Viverette
d3be6f5885 Add support for prepending text from a file
Fixes: 241826272
Test: ./update_prebuilts.py -x <bid>
Change-Id: I47146fc8a4f9120891ee28d15b006448d66aaff7
2022-08-11 11:41:38 -04:00
Mike McTernan
8883d52ce0 Revert "Promote ExtraText lint warning to an error" am: 0b29c62498
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2183156

Change-Id: I03abf2ef079a4b2c371eaf291b54e1d3fbf449b4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-11 11:02:51 +00:00
Mike McTernan
0b29c62498 Revert "Promote ExtraText lint warning to an error"
This reverts commit 7868b9e942.

Reason for revert: DroidMonitor: Potential culprit for Bug b/242013483 - verifying through ABTD before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted.

Bug: 242013483
Change-Id: Ib73f862f34ca2c8ede7e4b4b9403d1eed00d0426
2022-08-11 10:21:38 +00:00
Matthew Maurer
e94f3e73dd Revert^4 "rust: Only allow bindgen to produce rlibs."
199c188502

Change-Id: I28b353fa89354cc572806b2274169342a473a03c
2022-08-10 20:25:50 +00:00
Yu Liu
df4413a01f Support bp2build conversion of hidl_interface.
Bug: 232520859
Test: Manual tests
Change-Id: I0a16f3c0a953dbc58c196aa9d2fedd2063c2a875
2022-08-10 10:06:04 -07:00
Usta Shrestha
7085918868 Merge "bootstrap glob file list in one place" am: 50e02aaf76
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2179779

Change-Id: Idde110c94a94dc2109de442b97556f9b3e07a6cf
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-10 16:10:53 +00:00
Usta Shrestha
a9d544f44c Merge "soong_ui to reject "showcommands"" am: 8c7c58ae65
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2179778

Change-Id: I4bc11585ec9c17aeadb015188135f8dbbdcd3f41
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-10 16:10:42 +00:00
Usta Shrestha
9bef91a075 Merge "hardening of test for aquery.go" am: c471c4b2fc
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2178179

Change-Id: I96b5e3f8385e04fe5199c7a465f02c7a016684b4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-10 16:10:09 +00:00
Usta Shrestha
e9db2347b8 Merge "cosmetic: unnecessary method indirection" am: e77dc9668e
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2178174

Change-Id: I6fc8c308fb5d0b04afa5ffd10d250b8e3ff8f21b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-10 16:09:31 +00:00
Usta Shrestha
50e02aaf76 Merge "bootstrap glob file list in one place" 2022-08-10 15:07:17 +00:00
Usta Shrestha
8c7c58ae65 Merge "soong_ui to reject "showcommands"" 2022-08-10 15:07:08 +00:00
Usta Shrestha
c471c4b2fc Merge "hardening of test for aquery.go" 2022-08-10 15:06:55 +00:00
Usta Shrestha
e77dc9668e Merge "cosmetic: unnecessary method indirection" 2022-08-10 15:05:09 +00:00
Romain Jobredeaux
2773d751ff Merge "Space out the bp2build allowlist for easier merging." am: 136c0d110d
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2168382

Change-Id: I096671588a6fae6183cc172cea6877adaa8a8290
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-10 14:39:56 +00:00
Romain Jobredeaux
136c0d110d Merge "Space out the bp2build allowlist for easier merging." 2022-08-10 14:20:10 +00:00
Pierre-Clément Tosi
6f630ae6f2 Support prefix_symbols for prebuilt libraries
Add support for prefix_symbols from binary and object targets to allow
rules to prefix symbols from a checked-in library before linking against
it, namespacing the resulting symbols and avoiding potential clashes.

Bug: 237371962
Test: Android.bp with cc_prebuilt_library_static::prefix_symbols
Change-Id: Iaf9cdd12637d5a259ae4c4827ff20e4f911699a4
2022-08-10 09:36:11 +01:00
Treehugger Robot
d03ea3d4f1 Merge "Introduce cc_api_library" am: 3a4088dfd1
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2162587

Change-Id: I0902c1a7757db36b7fd4564c356f5f6046d3d1af
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-10 04:35:55 +00:00
Treehugger Robot
3a4088dfd1 Merge "Introduce cc_api_library" 2022-08-10 04:12:32 +00:00
Treehugger Robot
2a20239c70 Merge "Use aapt2" am: 258d296698
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2178198

Change-Id: I9fa8b617a0854826f6f4da5a66618d17338c5772
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-10 02:17:45 +00:00
Treehugger Robot
258d296698 Merge "Use aapt2" 2022-08-10 01:21:11 +00:00
Kiyoung Kim
487689eaee Introduce cc_api_library
Introduce cc_api_library, which reflects imported API from other
inner-tree. This cc_api_library module type will later refer from
the other ninja module which generates stub library from the interface
description.

Tested environment :
* original libc definition has been removed temporarily, to ensure that
  imported api stub library is being used from build
* Added new definition of libc as below
 cc_api_library {
  name: "libc",
  arch: {
    x86: {
      src: "libs/x86/libc.so",
    },
    x86_64: {
      src: "libs/x86_64/libc.so",
    },
  },
  header_libs: [
    "libc_headers",
  ],
  export_header_lib_headers: ["libc_headers"],
  min_sdk_version: "9",
  vendor_available: true,
 }

Bug: 236087698
Test: `ALLOW_MISSING_DEPENDENCIES=true m vendorimage` succeeded
Change-Id: I67070b0f3561aa2afd73b6c1c0fdf4255218baac
2022-08-10 10:12:18 +09:00
Treehugger Robot
5457488c9a Merge "Promote ExtraText lint warning to an error" am: 69f5b684d4
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2180436

Change-Id: Ie5c874e675795f1ca68205c18a60005a22e9f0e6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-10 00:29:21 +00:00
Treehugger Robot
69f5b684d4 Merge "Promote ExtraText lint warning to an error" 2022-08-10 00:01:46 +00:00
Usta Shrestha
9b3724ad74 bootstrap glob file list in one place
Test: m nothing
Bug: N/A
Change-Id: I2aa4132726b1491db767ccbe695bd98cda08ff50
2022-08-09 18:07:36 -04:00
Usta Shrestha
96ff7227a0 soong_ui to reject "showcommands"
deprecated flag that simply adds 5s to a build
the `ctx.Done()` signal handling is unnecessary because of clean-up hooks already in place

Test: m libc
Bug: N/A
Change-Id: Ica12d727264af1e61e97393bc8cbeba9f78cbc83
2022-08-09 18:07:25 -04:00
Usta Shrestha
e70f28c4fa hardening of test for aquery.go
add a depset that only files under ../bazel_tools/dep

Test: aquery_test.go
Bug: N/A
Change-Id: I324cdd8a4c8951465f1dd856916d9b36acf2d0fc
2022-08-09 18:07:16 -04:00
Usta Shrestha
59417a19c8 cosmetic: unnecessary method indirection
Test: m nothing
Bug: N/A
Change-Id: Ie750d6556a12fb077d4592b1b7cb3e8edc8ef80e
2022-08-09 18:07:12 -04:00
Treehugger Robot
e6fb48270c Merge "Add libapexutil to denylist" am: 8c9a3f2596
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2179776

Change-Id: Ieab95664fed76bbe30e927385a4aca642d36e9b6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-09 21:07:39 +00:00
Treehugger Robot
8c9a3f2596 Merge "Add libapexutil to denylist" 2022-08-09 20:36:26 +00:00
Cory Barker
c541de62f8 Merge "Revert^2 "Update the way we build AFL++ fuzz binaries"" am: 285112d3fc
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2178196

Change-Id: I8f331312d2978ecc17b91f2b1c9da418f56f46c0
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-09 20:14:41 +00:00
Cory Barker
285112d3fc Merge "Revert^2 "Update the way we build AFL++ fuzz binaries"" 2022-08-09 19:49:04 +00:00
Romain Jobredeaux
5a634d3364 Space out the bp2build allowlist for easier merging.
Test: b build packages/apps/Music
Change-Id: Icd762dc26932d3ea58234e474c46f7fb66c9967c
2022-08-09 14:51:36 -04:00
Sasha Smundak
fa056e72d1 Go extractor should succeed am: 8df2a5c2fb
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2178200

Change-Id: Ifb3520522b0153379e9a8386aa932222d953e710
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-09 18:15:41 +00:00
Romain Jobredeaux
20f7fbc61e Merge "Bp2Build converter for android_library_import and android_library." am: aaf4f5ec8a
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2165698

Change-Id: I7b1a2b36507a3354bbe35769fdfebfdeb859066a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-09 18:14:50 +00:00
Sasha Smundak
8df2a5c2fb Go extractor should succeed
Test: treehugger
Fixes: 241561896
Change-Id: I182ae375be33f355275d1c2fd54e2cb1ac913354
2022-08-09 17:55:47 +00:00
Romain Jobredeaux
aaf4f5ec8a Merge "Bp2Build converter for android_library_import and android_library." 2022-08-09 17:46:59 +00:00
Liz Kammer
6a824f6027 Add libapexutil to denylist
Depends on unconverted module apex-info-list-tinyxml

Test: ci
Test: b build //system/...
Change-Id: I56cba9aa193d7285d49a3fba92b07bb92241aad0
2022-08-09 13:26:23 -04:00
Cole Faust
7868b9e942 Promote ExtraText lint warning to an error
It was previously demoted due to preexisting occurances
of it in the tree, but those have all been removed now.

Bug: 236431222
Test: m lint-check
Change-Id: Ic3501c43a09b10bc8a2944b0caa6ebbec663a008
2022-08-09 09:51:44 -07:00
Liz Kammer
2afec9aa1d Denylist test_fips because it depends on adb am: 3d13adc9d1
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2179775

Change-Id: If18bf2957bb81e7b373f1ba8d6461b4ad853eb6b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-09 15:47:25 +00:00
Liz Kammer
3d13adc9d1 Denylist test_fips because it depends on adb
Test: CI
Test: b build //external/boringssl/...
Change-Id: I278b8229cea347acef2edd9e56e2f0853f7780e0
2022-08-09 09:26:54 -04:00
Treehugger Robot
36b436e14c Merge "update bp2build tests for cc_yasm + add to CI" am: 68572c83bf
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2177842

Change-Id: Ieb104fcf9a575ed7c913948c42309cdcb60c9d34
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-09 01:54:32 +00:00
Treehugger Robot
68572c83bf Merge "update bp2build tests for cc_yasm + add to CI" 2022-08-09 01:49:37 +00:00