Merge "Fix whitespace in soong_config_set calls"

This commit is contained in:
Treehugger Robot
2023-05-02 22:26:06 +00:00
committed by Gerrit Code Review
3 changed files with 7 additions and 1 deletions

View File

@@ -394,6 +394,10 @@ def _soong_config_namespace(g, nsname):
def _soong_config_set(g, nsname, var, value):
"""Assigns the value to the variable in the namespace."""
_soong_config_namespace(g, nsname)
if type(value) == "string":
# Trim right spaces, because in make the variable is set in an $(eval),
# which will ignore trailing spaces.
value = value.rstrip(" ")
g[_soong_config_namespaces_key][nsname][var]=value
def _soong_config_append(g, nsname, var, value):

View File

@@ -54,6 +54,7 @@ def init(g, handle):
rblf.soong_config_append(g, "NS1", "v2", "def")
rblf.soong_config_set(g, "NS2", "v3", "abc")
rblf.soong_config_set(g, "NS2", "v3", "xyz")
rblf.soong_config_set(g, "NS2", "v4", "xyz ")
rblf.mkdist_for_goals(g, "goal", "dir1/file1:out1 dir1/file2:out2")
rblf.mkdist_for_goals(g, "goal", "dir2/file2:")

View File

@@ -144,7 +144,8 @@ assert_eq(
"v2": "def"
},
"NS2": {
"v3": "xyz"
"v3": "xyz",
"v4": "xyz"
}
},
{k:v for k, v in sorted(ns.items()) }