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

Merge changes from topic "aconfig-improve-dump-formats" into main

Browse Source 
* changes:
  aconfig: add 'verbose' dump format
  aconfig: improve the 'text' dump format
  aconfig: remove the 'debug' dump format
This commit is contained in:
Treehugger Robot
2023-07-20 12:35:57 +00:00
committed by Gerrit Code Review
parent 9726e805b3 ea4981407b
commit 08cb169cbe
1 changed files with 16 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
tools/aconfig/src/commands.rs
View File

@@ -213,7 +213,7 @@ pub fn create_device_config_sysprops(mut input: Input) -> Result<Vec<u8>> {
#[derive(Copy, Clone, Debug, PartialEq, Eq, ValueEnum)]
pub enum DumpFormat {
Text,
Debug,
Verbose,
Protobuf,
Textproto,
}
@@ -229,18 +229,27 @@ pub fn dump_parsed_flags(mut input: Vec<Input>, format: DumpFormat) -> Result<Ve
DumpFormat::Text => {
for parsed_flag in parsed_flags.parsed_flag.into_iter() {
let line = format!(
"{}/{}: {:?} {:?}\n",
"{}/{}: {:?} + {:?}\n",
parsed_flag.package(),
parsed_flag.name(),
parsed_flag.state(),
parsed_flag.permission()
parsed_flag.permission(),
parsed_flag.state()
);
output.extend_from_slice(line.as_bytes());
}
}
DumpFormat::Debug => {
DumpFormat::Verbose => {
for parsed_flag in parsed_flags.parsed_flag.into_iter() {
let line = format!("{:#?}\n", parsed_flag);
let sources: Vec<_> =
parsed_flag.trace.iter().map(|tracepoint| tracepoint.source()).collect();
let line = format!(
"{}/{}: {:?} + {:?} ({})\n",
parsed_flag.package(),
parsed_flag.name(),
parsed_flag.permission(),
parsed_flag.state(),
sources.join(", ")
);
output.extend_from_slice(line.as_bytes());
}
}
@@ -326,7 +335,7 @@ mod tests {
let input = parse_test_flags_as_input();
let bytes = dump_parsed_flags(vec![input], DumpFormat::Text).unwrap();
let text = std::str::from_utf8(&bytes).unwrap();
assert!(text.contains("com.android.aconfig.test/disabled_ro: DISABLED READ_ONLY"));
assert!(text.contains("com.android.aconfig.test/disabled_ro: READ_ONLY + DISABLED"));
}
#[test]