Android Build Coastguard Worker
f232d983f5
Snap for 11794304 from 160a6c0310
to 24Q3-release
...
Change-Id: I174421d8b1ddca958adec7f65975e3a5b5c690c3
2024-05-02 23:00:41 +00:00
Cole Faust
160a6c0310
Improve error message when assigning select to nonconfigurable property am: 1c48101091
am: 6722022217
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3068366
Change-Id: I474a427a3c68303742f9779be67e0b46c9fed31f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-05-01 21:44:14 +00:00
Cole Faust
6722022217
Improve error message when assigning select to nonconfigurable property am: 1c48101091
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3068366
Change-Id: I8429704e4c614dc0d85fe0c1ab8d95e0a4f42957
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-05-01 21:15:42 +00:00
Android Build Coastguard Worker
069a18f8c3
Snap for 11784291 from e872f19792
to 24Q3-release
...
Change-Id: Id691fbdc8937dd0503e394f9dc3388829e1964cb
2024-04-30 23:01:24 +00:00
Cole Faust
1c48101091
Improve error message when assigning select to nonconfigurable property
...
Bug: 323382414
Test: m nothing --no-skip-soong-tests
Change-Id: Ideac33cd286566e14e7ff1ff579930add48a94db
2024-04-30 13:58:17 -07:00
Treehugger Robot
e872f19792
[automerger skipped] Merge "Empty merge of Android 24Q2 Release (ab/11526283) to aosp-main-future" into aosp-main-future am: b316cc9bc9
-s ours
...
am skip reason: Merged-In I7091effe7791c1f3ea57e775b37cd72f4ee13d03 with SHA-1 8ef7561b0c
is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/build/blueprint/+/27144984
Change-Id: Ie2ca9ab5e62c1cbdb1a0cabbfc013ba97220e4b4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-30 02:11:01 +00:00
Xin Li
d4fe814978
[automerger skipped] Empty merge of Android 24Q2 Release (ab/11526283) to aosp-main-future am: f672401e1b
-s ours
...
am skip reason: Merged-In I7091effe7791c1f3ea57e775b37cd72f4ee13d03 with SHA-1 8ef7561b0c
is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/build/blueprint/+/27144984
Change-Id: I242e315537d8b022f491b18837f94d91f26cfe0a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-30 02:10:26 +00:00
Treehugger Robot
b936384b5c
Merge changes from topics "fix_selects_appending", "refactor_selects" into main am: ca5ffdd3ce
am: b87170476c
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3062997
Change-Id: I54df335ce6965ed0cd410aa1708377ef257f370a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-30 02:00:27 +00:00
Treehugger Robot
b316cc9bc9
Merge "Empty merge of Android 24Q2 Release (ab/11526283) to aosp-main-future" into aosp-main-future
2024-04-30 01:58:42 +00:00
Treehugger Robot
b87170476c
Merge changes from topics "fix_selects_appending", "refactor_selects" into main am: ca5ffdd3ce
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3062997
Change-Id: Ic00ebdfb1adcc095910de739aa7c475d26cc1aa6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-30 01:47:33 +00:00
Treehugger Robot
ca5ffdd3ce
Merge changes from topics "fix_selects_appending", "refactor_selects" into main
...
* changes:
Refactor selects
Fix bugs when appending selects
Allow extending configurable propeties with non-configurable properties
2024-04-30 01:15:42 +00:00
Cole Faust
a52b058ccc
Refactor selects
...
In order to do less cloning, refactor selects so that all the
soong-visibile structs are immutable to soong and can be reused.
Additionally, refactor how the inner linked list of selects is managed,
so that the append/prepend/replace logic is simpler.
Bug: 323382414
Test: m nothing --no-skip-soong-tests
Change-Id: Iba5d27405decc1b0596590c3e0555daeb044bf9e
2024-04-29 13:23:30 -07:00
Xin Li
f672401e1b
Empty merge of Android 24Q2 Release (ab/11526283) to aosp-main-future
...
Bug: 337098550
Merged-In: I7091effe7791c1f3ea57e775b37cd72f4ee13d03
Change-Id: I1c6c3f6d848d7a67fbac426cd0347477b675a521
2024-04-29 11:46:46 -07:00
Android Build Coastguard Worker
0eb301b6cd
Snap for 11772614 from c1d603e975
to 24Q3-release
...
Change-Id: I9f44d68087181bf5876fd9666e34c8c2b9c2842d
2024-04-27 01:00:43 +00:00
Cole Faust
70b814a964
Fix bugs when appending selects
...
The biggest issue here is that Configurable objects needed to be
cloned before appended to one another, otherwise some Configurables
that were used in defaults could be edited after being applied to one
module and apply a different value to another module.
Also fix an issue where a select without a defined appendWrapper
always evaluated to nil.
I plan to make a followup refactor cl to make these things clearer,
but start with the bugfix.
Bug: 323382414
Test: m nothing --no-skip-soong-tests (see other cl in topic for tests)
Change-Id: Icf68d0ee1779c76bfb3d68db43b35d7e09bc0dd9
2024-04-25 15:38:56 -07:00
Cole Faust
4560bb086e
Allow extending configurable propeties with non-configurable properties
...
Sometimes modules add arch-variant properties in load hooks, to disable
modules by default on certain platforms for example. When changing the
property to a Configurable property, these load hooks would also need
to be changed in order to have a matching type for
ExtendMatchingProperties.
Since this can be kindof a pain to address everywhere, for now,
special case the extension functions to promote non-configurable
properties to configurable ones. We can remove this later when
everything switches to configurable properties.
Bug: 323382414
Test: go tests
Change-Id: Iac96587dbd60ccdd6aa667dd69a71ad252abe589
2024-04-25 15:31:00 -07:00
Haamed Gheibi
c1d603e975
Use relative GOPATH for microfactory golang builds. am: c10b49b5e9
am: ea82d0e22d
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3060443
Change-Id: I536d5701360a4c8cfbbf50253da83b8fc954c214
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-25 22:30:29 +00:00
Haamed Gheibi
ea82d0e22d
Use relative GOPATH for microfactory golang builds. am: c10b49b5e9
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3060443
Change-Id: If6558225f5f16a6750936c59598846485bffed20
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-25 21:47:39 +00:00
Haamed Gheibi
c10b49b5e9
Use relative GOPATH for microfactory golang builds.
...
Bug: b/336890676
Change-Id: I15ff839986d6c4997d5abccaa2ae8a9703d6cfb6
2024-04-25 04:44:01 +00:00
Android Build Coastguard Worker
263cc02caa
Snap for 11760402 from 1fbe12e4d7
to 24Q3-release
...
Change-Id: If4353e7361750dd34b7fd7c5a3b9a0358cde4895
2024-04-24 23:00:39 +00:00
Tomasz Wasilczyk
1fbe12e4d7
Merge "Add support for go:embed" into main am: 36b6322979
am: 78d056bd95
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3046017
Change-Id: I018f8bc58ecdcb76bb3a16eee13ef24342bc54a5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-24 13:42:11 +00:00
Tomasz Wasilczyk
78d056bd95
Merge "Add support for go:embed" into main am: 36b6322979
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3046017
Change-Id: I9e9ff27e457958a24630943e5f0305cf031c73e3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-24 13:25:05 +00:00
Tomasz Wasilczyk
36b6322979
Merge "Add support for go:embed" into main
2024-04-24 12:33:37 +00:00
Cole Faust
a476c072d0
Export ConfigurableCase and add constructors am: 185cb44bef
am: 5dac25b090
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3056565
Change-Id: I99ca8c4ce0e2fbb9c3ab678688bedebb59aae196
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-24 00:46:37 +00:00
Cole Faust
5dac25b090
Export ConfigurableCase and add constructors am: 185cb44bef
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3056565
Change-Id: Ic1cdf0b78cefcfc4298a740e798c2c3ff3405a22
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-24 00:29:10 +00:00
Cole Faust
185cb44bef
Export ConfigurableCase and add constructors
...
Some soong code sets arch-variant properties in order to control a
module's default behavior. I'll make this continue to work, but long
term the arch-variant properties should be replaced with selects,
so expose an API for creating select statements in soong code.
Bug: 323382414
Test: m nothing --no-skip-soong-tests
Change-Id: I6c65d6e112b6f826f1027777b6fdf36915d34b1d
2024-04-23 13:55:59 -07:00
Tomasz Wasilczyk
51b4b03520
Add support for go:embed
...
Bug: 302718225
Test: Build aosp/2798713
Change-Id: I8c09f1604aa1a702c639bba91cefd104f8f2f36d
2024-04-22 11:52:00 -07:00
Android Build Coastguard Worker
70dbe82629
Snap for 11740859 from 6f49c6448e
to 24Q3-release
...
Change-Id: I6251321c5f7567ccee4f962d4817a7da56012fa7
2024-04-20 01:00:39 +00:00
Colin Cross
6f49c6448e
Call TransitionMutator.IncomingTransition when adding dependencies later am: d7474dd743
am: 1e65fed726
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3017629
Change-Id: I27379e59e7316d0802a6d2f61ce252301aa10f8b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-18 19:28:10 +00:00
Colin Cross
1e65fed726
Call TransitionMutator.IncomingTransition when adding dependencies later am: d7474dd743
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3017629
Change-Id: I79ef8c4acf66ba3d11cbd4141ea0c2f9a147600b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-18 19:09:02 +00:00
Android Build Coastguard Worker
0d303c18c9
Snap for 11728787 from 87758cdfa0
to 24Q3-release
...
Change-Id: If3e1934154b43e2d282ad00fea250ac08e1515ee
2024-04-17 23:02:06 +00:00
Colin Cross
d7474dd743
Call TransitionMutator.IncomingTransition when adding dependencies later
...
Adding a dependency on a module that has already had a TransitionMutator
run on it may require adjusting the variation name based on the results
of IncomingTranstion. Store the variants that existed before the
TransitionMutator ran, find one that is a subset of the requested
variant, and call TranstionMutator.IncomingTransition to update the
value.
Bug: 319288033
Test: TestPostTransitionDeps
Change-Id: I690357f9792401a3edbc5ae9fdcb666495954fbc
2024-04-16 15:18:29 -07:00
Colin Cross
87758cdfa0
Merge "Move TransitionMutator to transition.go and add tests" into main am: 6ed94b7f85
am: ae74d5ecd7
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3042673
Change-Id: Ib54ac052f761100f1d4d19235e439cb96ee3371a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-16 20:48:07 +00:00
Colin Cross
ae74d5ecd7
Merge "Move TransitionMutator to transition.go and add tests" into main am: 6ed94b7f85
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3042673
Change-Id: I08ecacb507ae8b5fd19e444b2f28da23a0231ef7
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-16 20:30:33 +00:00
Colin Cross
6ed94b7f85
Merge "Move TransitionMutator to transition.go and add tests" into main
2024-04-16 20:14:53 +00:00
Cole Faust
91dce455c0
Support unpacking a variable to a configurable property am: 28357db9d0
am: ea224ecf22
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3040813
Change-Id: Iaf13baa636be6a7651bcf1ee86de31b20ef07974
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-16 18:52:34 +00:00
Cole Faust
2b5076bdb6
Support multi-variable selects and typed selects am: 3311debbb3
am: 6b6eb4344d
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3038314
Change-Id: I163d8d39a6709e810bdadb32cc3eda8495bacfa7
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-16 18:52:24 +00:00
Cole Faust
ea224ecf22
Support unpacking a variable to a configurable property am: 28357db9d0
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3040813
Change-Id: If6161e750f2252db52c6cfb6579099abc926120c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-16 18:27:45 +00:00
Cole Faust
6b6eb4344d
Support multi-variable selects and typed selects am: 3311debbb3
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3038314
Change-Id: I47723388f70cc767219112a00d25bb637915bc4d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-16 18:27:41 +00:00
Cole Faust
28357db9d0
Support unpacking a variable to a configurable property
...
Bug: 323382414
Test: m nothing --no-skip-soong-tests
Change-Id: I504e53c34305f5de0af8629bf21ac1ce94704732
2024-04-12 16:37:51 -07:00
Cole Faust
3311debbb3
Support multi-variable selects and typed selects
...
This adds support for selecting on multiple variables at once, so that
you can do AND/OR combindations of them. For example:
select((
arch(),
os(),
), {
("arm64", "linux"): ["libfoo64"],
(default, "linux"): ["libfoo"],
(default, "windows"): ["libfoowindows"],
(default, default): ["libbar"],
})
It also allows for select conditions to be boolean-typed. You can
write literal true and false without quotes to select on them. Currently
we don't have any boolean-typed variables though, so a fake one was
added for testing.
Bug: 323382414
Test: m nothing --no-skip-soong-tests
Change-Id: Ibe586e7b21865b8734027848cc421594cbd1d8cc
2024-04-12 16:33:01 -07:00
Colin Cross
95b3627f6f
Move TransitionMutator to transition.go and add tests
...
Bug: 319288033
Test: transition_test.go
Change-Id: Ia6fd96a69f559e3356155954fd021ec8d7293908
2024-04-12 14:58:58 -07:00
Android Build Coastguard Worker
b5b3deedeb
Snap for 11691493 from 40acd0417b
to 24Q3-release
...
Change-Id: I2e803d6ac8f0818833e6978d252522ec057815db
2024-04-09 23:01:08 +00:00
Cole Faust
40acd0417b
Rename Evaluate() to Get() and add GetDefault() am: 09fe90e407
am: 70c30f6ec8
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3028284
Change-Id: I226a299cc5c9af60ed8ef5d415541c739619cfe4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-08 21:33:48 +00:00
Cole Faust
97080d8421
Add android:replace_instead_of_append am: 2437d5edb9
am: e8602b8b22
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3025378
Change-Id: I271a96c3622853a364a11a10911d1ce50a1d7459
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-08 21:33:46 +00:00
Cole Faust
70c30f6ec8
Rename Evaluate() to Get() and add GetDefault() am: 09fe90e407
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3028284
Change-Id: Id10c8a2773f17f7a1d97d39c6f99f5b39d608728
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-08 21:16:18 +00:00
Cole Faust
e8602b8b22
Add android:replace_instead_of_append am: 2437d5edb9
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3025378
Change-Id: I0af8eab5aa136b05c0a97acdbe40380eccfd7f2d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-08 21:16:13 +00:00
Android Build Coastguard Worker
68a4e74c30
Snap for 11679388 from 760d00076c
to 24Q3-release
...
Change-Id: Ifa8fe8b3df60db9204182b60a7f405ec4ac9e76c
2024-04-06 01:00:42 +00:00
Cole Faust
760d00076c
Rename default select branch to 'default' keyword am: 0173a2268b
am: 05b4504d80
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3024123
Change-Id: Icc4fde6bbf2a306ae71fc518a1ca664460161ea3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-04 22:23:48 +00:00
Cole Faust
05b4504d80
Rename default select branch to 'default' keyword am: 0173a2268b
...
Original change: https://android-review.googlesource.com/c/platform/build/blueprint/+/3024123
Change-Id: I985f397d1a954c711170bc2a757439950b7271ad
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-04 22:04:48 +00:00