Merge "Remove the code for bazel when reading ReleaseAconfigValueSets." into main

This commit is contained in:
Yu Liu 2023-12-08 21:52:59 +00:00 committed by Gerrit Code Review
commit 65287d5dc2

View file

@ -210,22 +210,7 @@ func (c Config) ReleaseVersion() string {
// The aconfig value set passed to aconfig, derived from RELEASE_VERSION // The aconfig value set passed to aconfig, derived from RELEASE_VERSION
func (c Config) ReleaseAconfigValueSets() []string { func (c Config) ReleaseAconfigValueSets() []string {
// This logic to handle both Soong module name and bazel target is temporary in order to return c.config.productVariables.ReleaseAconfigValueSets
// provide backward compatibility where aosp and internal both have the release
// aconfig value set but can't be updated at the same time to use bazel target
var valueSets []string
for _, valueSet := range c.config.productVariables.ReleaseAconfigValueSets {
value := strings.Split(valueSet, ":")
valueLen := len(value)
if valueLen > 2 {
// This shouldn't happen as this should be either a module name or a bazel target path.
panic(fmt.Errorf("config file: invalid value for release aconfig value sets: %s", valueSet))
}
if valueLen > 0 {
valueSets = append(valueSets, value[valueLen-1])
}
}
return valueSets
} }
// The flag default permission value passed to aconfig // The flag default permission value passed to aconfig