Merge changes from topic "aconfig-remove-format-aliases" into main
* changes:
aconfig: remove support for --format=text
aconfig: remove support for --format=verbose
aconfig: remove support for --format=bool
diff --git a/tools/aconfig/src/dump.rs b/tools/aconfig/src/dump.rs
index bcad064..37368ee 100644
--- a/tools/aconfig/src/dump.rs
+++ b/tools/aconfig/src/dump.rs
@@ -36,17 +36,6 @@
// protobuf formats
"protobuf" => Ok(Self::Protobuf),
"textproto" => Ok(Self::Textproto),
-
- // old formats now implemented as aliases to custom format
- "text" => Ok(Self::Custom(
- "{fully_qualified_name} [{container}]: {permission} + {state}".to_owned(),
- )),
- "verbose" => Ok(Self::Custom(
- "{fully_qualified_name} [{container}]: {permission} + {state} ({trace:paths})"
- .to_owned(),
- )),
- "bool" => Ok(Self::Custom("{fully_qualified_name}={state:bool}".to_owned())),
-
// custom format
_ => Ok(Self::Custom(value.to_owned())),
}
@@ -270,20 +259,6 @@
"{fully_qualified_name}={permission} + {state}",
"com.android.aconfig.test.enabled_ro=READ_ONLY + ENABLED"
);
-
- // aliases
- assert_dump_parsed_flags_custom_format_contains!(
- "text",
- "com.android.aconfig.test.enabled_ro [system]: READ_ONLY + ENABLED"
- );
- assert_dump_parsed_flags_custom_format_contains!(
- "verbose",
- "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]
diff --git a/tools/aconfig/src/main.rs b/tools/aconfig/src/main.rs
index 59f3677..6c4e241 100644
--- a/tools/aconfig/src/main.rs
+++ b/tools/aconfig/src/main.rs
@@ -114,7 +114,9 @@
Arg::new("format")
.long("format")
.value_parser(|s: &str| DumpFormat::try_from(s))
- .default_value("text"),
+ .default_value(
+ "{fully_qualified_name} [{container}]: {permission} + {state}",
+ ),
)
.arg(
Arg::new("filter")