feat(efiboot/list): allow to show a single namespace (#44)

efiboot/Cargo.toml

@@ -17,6 +17,7 @@ efivar = { version = "1.0.14", path = "../efivar", features = ["store"] }
 itertools = "0.10.5"
 paw = "1.0.0"
 structopt = { version = "0.3.26", features = ["paw"] }
+uuid = { version = "1.3.4" }
 
 [package.metadata.binstall]
 pkg-url = "{ repo }/releases/download/v{ version }/{ name }_{ target }{ archive-suffix }"

efiboot/src/cli/list.rs

@@ -1,10 +1,34 @@
-use efivar::VarManager;
+use efivar::{efi::VariableVendor, VarManager};
 
-pub fn run(enumerator: Box<dyn VarManager>) {
+fn list_all(enumerator: Box<dyn VarManager>) {
+    println!("{: >36} Variable", "Namespace");
     for var in enumerator
         .get_var_names()
         .expect("Failed to list variable names")
     {
-        println!("{}", var.short_name());
+        println!("{} {}", var.vendor(), var.variable());
+    }
+}
+
+fn list_namespace(enumerator: Box<dyn VarManager>, vendor: VariableVendor) {
+    println!("Variables in namespace {} :", vendor);
+    for var in enumerator
+        .get_var_names()
+        .expect("Failed to list variable names")
+    {
+        if var.vendor() == &vendor {
+            println!("{}", var.variable());
+        }
+    }
+}
+
+pub fn run(enumerator: Box<dyn VarManager>, namespace: Option<uuid::Uuid>, all: bool) {
+    if all {
+        list_all(enumerator);
+    } else {
+        list_namespace(
+            enumerator,
+            namespace.map_or(VariableVendor::Efi, VariableVendor::Custom),
+        );
     }
 }

efiboot/src/main.rs

@@ -16,7 +16,14 @@ enum Command {
         string: bool,
     },
     /// List known EFI variables
-    List,
+    List {
+        /// GUID of the namespace. Default: EFI standard namespace
+        #[structopt(value_name = "GUID")]
+        namespace: Option<uuid::Uuid>,
+        /// ignore --namespace and show all namespaces
+        #[structopt(short, long)]
+        all: bool,
+    },
 }
 
 #[derive(StructOpt)]
@@ -48,8 +55,8 @@ fn main(opts: Opt) {
         Command::Read { name, string } => {
             cli::read(manager, &name, string);
         }
-        Command::List => {
-            cli::list(manager);
+        Command::List { namespace, all } => {
+            cli::list(manager, namespace, all);
         }
     }
 }