diff options
| author | Colin Cross <ccross@android.com> | 2018-04-17 10:52:26 -0700 |
|---|---|---|
| committer | Colin Cross <ccross@android.com> | 2018-04-18 15:26:55 -0700 |
| commit | 9ccbc6036d9fda84c3cf7253a578c91fb60ace8d (patch) | |
| tree | 17653abf9ca6c5a7031f8cfa3442004803bbabaa /java | |
| parent | e20c1b19045c880e6dfe6bce7c00e7bce0c20318 (diff) | |
| download | build_soong-9ccbc6036d9fda84c3cf7253a578c91fb60ace8d.tar.gz build_soong-9ccbc6036d9fda84c3cf7253a578c91fb60ace8d.tar.bz2 build_soong-9ccbc6036d9fda84c3cf7253a578c91fb60ace8d.zip | |
Reverse RRO dir paths when passing to Make
LOCAL_RESOURCE_DIRS puts the highest priority overlay first, but
aapt2 expects the highest priority overlay last. Soong stores the
list in aapt2 order (low to high priority), but that means when it
exports to Make as LOCAL_SOONG_RRO_DIRS, which goes to
build_rro_package.mk and then package_internal.mk, it gets reversed
again and comes out backwards.
Bug: 78032566
Test: m checkbuild
Change-Id: If72bf929fbf1d126f9051a2f21ec1eb4e3030e6e
Merged-In: If72bf929fbf1d126f9051a2f21ec1eb4e3030e6e
(cherry picked from commit a140bb05f1eeafe046574454debd190c32e09888)
Diffstat (limited to 'java')
| -rw-r--r-- | java/androidmk.go | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/java/androidmk.go b/java/androidmk.go index 3658636f..13966ed7 100644 --- a/java/androidmk.go +++ b/java/androidmk.go @@ -195,7 +195,9 @@ func (app *AndroidApp) AndroidMk() android.AndroidMkData { } if len(app.rroDirs) > 0 { - fmt.Fprintln(w, "LOCAL_SOONG_RRO_DIRS :=", strings.Join(app.rroDirs.Strings(), " ")) + // Reverse the order, Soong stores rroDirs in aapt2 order (low to high priority), but Make + // expects it in LOCAL_RESOURCE_DIRS order (high to low priority). + fmt.Fprintln(w, "LOCAL_SOONG_RRO_DIRS :=", strings.Join(android.ReversePaths(app.rroDirs).Strings(), " ")) } if Bool(app.appProperties.Export_package_resources) { |