removed alpm dependency (#1)

* removed alpm dependency

* fix typo in arg

Signed-off-by: innocentzero <[email protected]>

---------

Signed-off-by: innocentzero <[email protected]>
Co-authored-by: innocentzero <[email protected]>

Cargo.toml

@@ -26,7 +26,6 @@ toml = "0.8"
 derive_more = "0.99"
 itertools = "0.12"
 
-alpm = "3.0"
 # rust-apt = "0.7"
 
 [profile.release]

src/backend/apt.rs

@@ -10,7 +10,6 @@
 // #[derive(Debug, Copy, Clone, derive_more::Display)]
 // pub struct Apt;
 
-// #[allow(dead_code)]
 // #[derive(Debug, Clone)]
 // pub struct AptQueryInfo {
 //     explicit: bool,

src/backend/arch.rs

@@ -1,17 +1,15 @@
-use alpm::{Alpm, PackageReason};
 use anyhow::Result;
 use std::collections::BTreeMap;
 
-use crate::cmd::run_args;
+use crate::cmd::{command_found, run_args, run_args_for_stdout};
 use crate::prelude::*;
 
 #[derive(Debug, Clone, Copy, derive_more::Display)]
 pub struct Arch;
 
-#[allow(dead_code)]
 #[derive(Debug, Clone)]
 pub struct ArchQueryInfo {
-    reason: PackageReason,
+    pub explicit: bool,
 }
 
 pub struct ArchMakeImplicit;
@@ -24,16 +22,21 @@ impl Backend for Arch {
     type Modification = ArchMakeImplicit;
 
     fn query_installed_packages(_: &Config) -> Result<BTreeMap<Self::PackageId, Self::QueryInfo>> {
-        let alpm = match Alpm::new("/", "/var/lib/pacman") {
-            Ok(x) => x,
-            Err(_) => return Ok(BTreeMap::new()),
-        };
+        if !command_found("pacman") {
+            return Ok(BTreeMap::new());
+        }
 
-        Ok(alpm
-            .localdb()
-            .pkgs()
-            .iter()
-            .map(|x| (x.name().to_string(), ArchQueryInfo { reason: x.reason() }))
+        let explicit = run_args_for_stdout(["pacman", "--query", "--explicit", "--quiet"])?;
+        let dependency = run_args_for_stdout(["pacman", "--query", "--deps", "--quiet"])?;
+
+        Ok(dependency
+            .lines()
+            .map(|x| (x.to_string(), ArchQueryInfo { explicit: false }))
+            .chain(
+                explicit
+                    .lines()
+                    .map(|x| (x.to_string(), ArchQueryInfo { explicit: true })),
+            )
             .collect())
     }
 

src/backend/dnf.rs

@@ -8,7 +8,6 @@ use crate::prelude::*;
 #[derive(Debug, Copy, Clone, derive_more::Display)]
 pub struct Dnf;
 
-#[allow(dead_code)]
 #[derive(Debug, Clone)]
 pub struct DnfQueryInfo {
     user: bool,

src/backend/flatpak.rs

@@ -8,7 +8,6 @@ use crate::prelude::*;
 #[derive(Debug, Copy, Clone, derive_more::Display)]
 pub struct Flatpak;
 
-#[allow(dead_code)]
 #[derive(Debug, Clone)]
 pub struct FlatpakQueryInfo {
     explicit: bool,

src/backend/pip.rs

@@ -13,10 +13,9 @@ use crate::prelude::*;
 #[derive(Debug, Copy, Clone, derive_more::Display)]
 pub struct Pip;
 
-#[allow(dead_code)]
 #[derive(Debug, Clone)]
 pub struct PipQueryInfo {
-    explicit: bool,
+    pub explicit: bool,
 }
 
 impl Backend for Pip {
@@ -84,13 +83,11 @@ impl Backend for Pip {
 fn extract_package_names(stdout: String) -> Result<BTreeSet<String>> {
     let value: Value = serde_json::from_str(&stdout)?;
 
-    let result = value
+    Ok(value
         .as_array()
         .context("getting inner json array")?
         .iter()
         .map(|node| node["name"].as_str().expect("should always be a string"))
         .map(String::from)
-        .collect();
-
-    Ok(result)
+        .collect())
 }