StrixOS/build
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

aconfig: remove support for --format=bool

Browse Source 
This CL is part of a larger effort to simplify maintenance by removing
format aliases: with the new --format option, clients of `aconfig dump`
can specify the format they need themselves.

Bug: 317044914
Test: m
Change-Id: I11efc5d7280402f503ddd05f2dc65bb15ffd8ac7
This commit is contained in:
Mårten Kongstad
2023-12-19 15:21:56 +01:00
parent 45e9ce5d49
commit 3d76d1afe0
1 changed files with 0 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
tools/aconfig/src/dump.rs
View File

@@ -45,7 +45,6 @@ impl TryFrom<&str> for DumpFormat {
"{fully_qualified_name} [{container}]: {permission} + {state} ({trace:paths})" "{fully_qualified_name} [{container}]: {permission} + {state} ({trace:paths})"
.to_owned(), .to_owned(),
)), )),
"bool" => Ok(Self::Custom("{fully_qualified_name}={state:bool}".to_owned())),
// custom format // custom format
_ => Ok(Self::Custom(value.to_owned())), _ => Ok(Self::Custom(value.to_owned())),
@@ -280,10 +279,6 @@ mod tests {
"verbose", "verbose",
"com.android.aconfig.test.enabled_ro [system]: READ_ONLY + ENABLED (tests/test.aconfig, tests/first.values, tests/second.values)" "com.android.aconfig.test.enabled_ro [system]: READ_ONLY + ENABLED (tests/test.aconfig, tests/first.values, tests/second.values)"
); );
assert_dump_parsed_flags_custom_format_contains!(
"bool",
"com.android.aconfig.test.enabled_ro=true"
);
} }
#[test] #[test]