Merge upstream changes

Author: vejnar

Cargo.lock

@@ -14,7 +14,7 @@ include = ["src/**/*", "LICENSE", "README.md", "CHANGELOG.md", "help"]
 
 
 [dependencies]
-alpm = "2.1.3"
+alpm = "2.2.1"
 alpm-utils = "1.1.2"
 aur-depends = "1.0.3"
 aur-fetch = "0.10.0"
@@ -63,3 +63,6 @@ generate = ["alpm/generate"]
 static = ["alpm/static"]
 mock = ["async-trait"]
 container = []
+
+#[patch.crates-io]
+#alpm = { path = "../alpm.rs/alpm" }

Cargo.toml

@@ -289,6 +289,9 @@ separated list that is quoted by the shell.
 This enables fm review mode, where PKGBUILD review is done using the file
 manager specified by command.
 
+This allows the PKGBUILDs and related files to be edited. Changes won't
+persist unless commited.
+
 .TP
 .B \-\-fmflags <flags>
 Passes arguments to file manager. These flags get passed to every instance where

man/paru.8

@@ -262,6 +262,9 @@ guaranteed to work with a custom \fBsudo\fR command.
 This enables fm review mode, where PKGBUILD review is done using the file
 manager specified by command.
 
+This allows the PKGBUILDs and related files to be edited. Changes won't
+persist unless commited.
+
 .TP
 .B MFlags = Flags...
 Passes arguments to makepkg. These flags get passed to every instance where

man/paru.conf.5

@@ -621,7 +621,7 @@ msgstr "    -p --print            Imprimir pkgbuild en stdout"
 
 #: src/help.rs:91
 msgid "    -c --comments         Print AUR comments for pkgbuild"
-msgstr "   -c --comments         Mostrar comentarios de AUR para pkgbuild"
+msgstr "    -c --comments         Mostrar comentarios de AUR para pkgbuild"
 
 #: src/help.rs:93
 msgid "upgrade specific options:"

po/es.po

@@ -339,7 +339,7 @@ msgstr "het parsen van de gegenereerde srcinfo door makepkg is mislukt"
 
 #: src/install.rs:171 src/install.rs:442
 msgid "Resolving dependencies..."
-msgstr "Afhaneklijkheden oplossen..."
+msgstr "Afhankelijkheden oplossen..."
 
 #: src/install.rs:220 src/install.rs:225
 msgid "failed to download sources"

po/nl.po

@@ -4,6 +4,7 @@ use anyhow::Result;
 use std::ffi::OsStr;
 use std::io::Write;
 use std::path::{Path, PathBuf};
+use std::process::Command;
 
 #[derive(Debug)]
 pub struct Chroot {
@@ -37,52 +38,49 @@ impl Chroot {
     }
 
     pub fn create<S: AsRef<OsStr>>(&self, config: &Config, pkgs: &[S]) -> Result<()> {
-        let args = &[
-            OsStr::new("install"),
-            OsStr::new("-dm755"),
-            self.path.as_os_str(),
-        ];
-        exec::command(&config.sudo_bin, ".", args)?;
+        let mut cmd = Command::new(&config.sudo_bin);
+        cmd.arg("install").arg("-dm755").arg(&self.path);
+        exec::command(&mut cmd)?;
 
         let tmp = pacman_conf(&self.pacman_conf)?;
         let dir = self.path.join("root");
 
-        let mut args = vec![
-            OsStr::new("-C"),
-            tmp.path().as_os_str(),
-            OsStr::new("-M"),
-            OsStr::new(&self.makepkg_conf),
-            dir.as_os_str(),
-        ];
-        args.extend(pkgs.iter().map(|p| p.as_ref()));
-        exec::command("mkarchroot", ".", &args)?;
+        let mut cmd = Command::new("makechrootpkg");
+        cmd.arg("-C")
+            .arg(tmp.path())
+            .arg("-M")
+            .arg(&self.makepkg_conf)
+            .arg(dir)
+            .args(pkgs);
+
+        exec::command(&mut cmd)?;
         Ok(())
     }
 
     pub fn run<S: AsRef<OsStr>>(&self, args: &[S]) -> Result<()> {
         let dir = self.path.join("root");
         let tmp = pacman_conf(&self.pacman_conf)?;
 
-        let mut a = vec![
-            OsStr::new("-C"),
-            tmp.path().as_os_str(),
-            OsStr::new("-M"),
-            OsStr::new(&self.makepkg_conf),
-            dir.as_os_str(),
-        ];
+        let mut cmd = Command::new("arch-nspawn");
+        cmd.arg("-C")
+            .arg(tmp.path())
+            .arg("-M")
+            .arg(&self.makepkg_conf)
+            .arg(dir);
 
         for file in &self.ro {
-            a.push(OsStr::new("--bind-ro"));
-            a.push(OsStr::new(file));
+            cmd.arg("--bind-ro");
+            cmd.arg(file);
         }
 
         for file in &self.rw {
-            a.push(OsStr::new("--bind"));
-            a.push(OsStr::new(file));
+            cmd.arg("--bind");
+            cmd.arg(file);
         }
 
-        a.extend(args.iter().map(|p| p.as_ref()));
-        exec::command("arch-nspawn", ".", &a)?;
+        cmd.args(args);
+
+        exec::command(&mut cmd)?;
         Ok(())
     }
 
@@ -91,72 +89,24 @@ impl Chroot {
     }
 
     pub fn build(&self, pkgbuild: &Path, chroot_flags: &[&str], flags: &[&str]) -> Result<()> {
-        let mut args = chroot_flags.iter().map(OsStr::new).collect::<Vec<_>>();
-        args.push(OsStr::new("-r"));
-        args.push(OsStr::new(self.path.as_os_str()));
+        let mut cmd = Command::new("makechrootpkg");
+
+        cmd.current_dir(pkgbuild)
+            .args(chroot_flags)
+            .arg("-r")
+            .arg(&self.path);
 
         for file in &self.ro {
-            args.push(OsStr::new("-D"));
-            args.push(OsStr::new(file));
+            cmd.arg("-D").arg(file);
         }
 
         for file in &self.rw {
-            args.push(OsStr::new("-d"));
-            args.push(OsStr::new(file));
+            cmd.arg("-d").arg(file);
         }
 
-        args.push(OsStr::new("--"));
-
-        for flag in flags {
-            args.push(OsStr::new(flag));
-        }
-        for flag in &self.mflags {
-            args.push(OsStr::new(flag));
-        }
+        cmd.arg("--").args(flags).args(&self.mflags);
 
-        exec::command("makechrootpkg", pkgbuild, &args)?;
+        exec::command(&mut cmd)?;
         Ok(())
     }
 }
-
-/*pub fn chroot(globals: clap::Globals, ch: clap::Chroot) -> Result<()> {
-    let pacman = pacmanconf::Config::from_file(&globals.pacman_conf)?;
-
-    let chroot = Chroot {
-        path: ch.chroot,
-        pacman_conf: globals.pacman_conf.clone(),
-        makepkg_conf: globals.makepkg_conf.clone(),
-        ro: repo::all_files(&pacman)
-            .iter()
-            .map(|s| s.to_string())
-            .collect(),
-        rw: pacman.cache_dir.clone(),
-    };
-
-    println!("{:?}", chroot);
-
-    if !chroot.path.exists() && !ch.create {
-        bail!("chroot does not exist: use 'chroot -c' to create it");
-    }
-
-    if ch.create {
-        if ch.targets.is_empty() {
-            chroot.create(&["base-devel"])?;
-        } else {
-            chroot.create(&ch.targets)?;
-        }
-    } else if ch.upgrade {
-        chroot.update()?;
-    } else if ch.interactive {
-        let targs: &[&str] = &[];
-        chroot.run(targs)?;
-    } else if ch.sync {
-        let mut cmd = vec!["pacman", "-S", "--ask=255"];
-        cmd.extend(ch.targets.iter().map(|s| s.as_str()));
-        chroot.run(&cmd)?;
-    } else {
-        chroot.run(&ch.targets)?;
-    }
-
-    Ok(())
-}*/

src/chroot.rs

@@ -410,10 +410,9 @@ pub async fn show_comments(config: &mut Config) -> Result<i32> {
     let c = config.color;
 
     for base in &bases.bases {
-        let url = config.aur_url.join(&format!(
-            "packages/{}/comments?&PP=1000000",
-            base.package_base()
-        ))?;
+        let url = config
+            .aur_url
+            .join(&format!("packages/{}", base.package_base()))?;
 
         let response = client
             .get(url.clone())
@@ -441,7 +440,7 @@ pub async fn show_comments(config: &mut Config) -> Result<i32> {
 
         if config.sort_mode == SortMode::TopDown {
             for (title, comment) in iter.into_iter() {
-                println!("{}", c.bold.paint(title.trim()));
+                print_indent(c.bold, 0, 0, config.cols, " ", title.split_whitespace());
 
                 for line in comment.trim().split('\n') {
                     let line = line.split_whitespace();
@@ -452,7 +451,7 @@ pub async fn show_comments(config: &mut Config) -> Result<i32> {
             }
         } else {
             for (title, comment) in iter.into_iter().rev() {
-                println!("{}", c.bold.paint(title.trim()));
+                print_indent(c.bold, 0, 0, config.cols, " ", title.split_whitespace());
 
                 for line in comment.trim().split('\n') {
                     let line = line.split_whitespace();