apease clippy

Author: Skgland

build.rs

@@ -8,7 +8,7 @@ fn main() -> Result<(), Box<dyn Error>> {
     let target = std::env::var("TARGET")?;
 
     let output = std::env::var("OUT_DIR")?;
-    ::std::fs::write(format!("{}/target", output), target.as_bytes())?;
+    ::std::fs::write(format!("{output}/target"), target.as_bytes())?;
 
     println!("cargo:rerun-if-env-changed=DOCS_RS");
     if std::env::var_os("DOCS_RS").as_deref() == Some(std::ffi::OsStr::new("1")) {

src/cmd/mod.rs

@@ -501,14 +501,14 @@ impl<'w, 'pl> Command<'w, 'pl> {
                 cmd.env(k, v);
             }
 
-            let cmdstr = format!("{:?}", cmd);
+            let cmdstr = format!("{cmd:?}");
 
             if let Some(ref cd) = self.cd {
                 cmd.current_dir(cd);
             }
 
             if self.log_command {
-                info!("running `{}`", cmdstr);
+                info!("running `{cmdstr}`");
             }
 
             let out = RUNTIME
@@ -521,7 +521,7 @@ impl<'w, 'pl> Command<'w, 'pl> {
                     self.log_output,
                 ))
                 .map_err(|e| {
-                    error!("error running command: {}", e);
+                    error!("error running command: {e}");
                     e
                 })?;
 

src/cmd/sandbox.rs

@@ -29,7 +29,7 @@ impl SandboxImage {
     /// error will be returned instead.
     pub fn remote(name: &str) -> Result<Self, CommandError> {
         let mut image = SandboxImage { name: name.into() };
-        info!("pulling image {} from Docker Hub", name);
+        info!("pulling image {name} from Docker Hub");
         Command::new_workspaceless("docker")
             .args(&["pull", name])
             .run()
@@ -219,7 +219,7 @@ impl SandboxBuilder {
     }
 
     pub(super) fn user(mut self, user: u32, group: u32) -> Self {
-        self.user = Some(format!("{}:{}", user, group));
+        self.user = Some(format!("{user}:{group}"));
         self
     }
 
@@ -242,7 +242,7 @@ impl SandboxBuilder {
 
         for (var, value) in &self.env {
             args.push("-e".into());
-            args.push(format! {"{}={}", var, value})
+            args.push(format! {"{var}={value}"})
         }
 
         if let Some(workdir) = self.workdir {
@@ -308,10 +308,10 @@ impl SandboxBuilder {
         scopeguard::defer! {{
             if let Err(err) = container.delete() {
                 error!("failed to delete container {}", container.id);
-                error!("caused by: {}", err);
+                error!("caused by: {err}");
                 let mut err: &dyn Error = &err;
                 while let Some(cause) = err.source() {
-                    error!("caused by: {}", cause);
+                    error!("caused by: {cause}");
                     err = cause;
                 }
             }

src/crates/registry.rs

@@ -122,7 +122,7 @@ impl RegistryCrate {
                     git2::build::RepoBuilder::new()
                         .fetch_options(fo)
                         .clone(url, &index_path)
-                        .with_context(|| format!("unable to update_index at {}", url))?;
+                        .with_context(|| format!("unable to update_index at {url}"))?;
                     info!("cloned registry index");
                 }
                 let config = std::fs::read_to_string(index_path.join("config.json"))?;

src/inside_docker.rs

@@ -62,7 +62,7 @@ pub(crate) fn probe_container_id(workspace: &Workspace) -> anyhow::Result<Option
         .log_command(false)
         .run_capture()?;
     for id in out.stdout_lines() {
-        info!("probing container id {}", id);
+        info!("probing container id {id}");
 
         let res = Command::new(workspace, "docker")
             .args(&["exec", id, "cat", probe_path_str])
@@ -71,7 +71,7 @@ pub(crate) fn probe_container_id(workspace: &Workspace) -> anyhow::Result<Option
             .run_capture();
         if let Ok([probed]) = res.as_ref().map(|out| out.stdout_lines()) {
             if *probed == probe_content {
-                info!("probe successful, this is container ID {}", id);
+                info!("probe successful, this is container ID {id}");
                 return Ok(Some(id.clone()));
             }
         }

src/prepare.rs

@@ -268,7 +268,7 @@ impl<'a> TomlTweaker<'a> {
         // Eliminate parent workspaces
         if let Some(&mut Value::Table(ref mut package)) = self.table.get_mut("package") {
             if package.remove("workspace").is_some() {
-                info!("removed parent workspace from {}", krate);
+                info!("removed parent workspace from {krate}");
             }
         }
     }
@@ -295,15 +295,15 @@ impl<'a> TomlTweaker<'a> {
 
         // Strip the 'publish-lockfile' key from [package]
         if has_publish_lockfile {
-            info!("disabled cargo feature 'publish-lockfile' from {}", krate);
+            info!("disabled cargo feature 'publish-lockfile' from {krate}");
             if let Some(&mut Value::Table(ref mut package)) = self.table.get_mut("package") {
                 package.remove("publish-lockfile");
             }
         }
 
         // Strip the 'default-run' key from [package]
         if has_default_run {
-            info!("disabled cargo feature 'default-run' from {}", krate);
+            info!("disabled cargo feature 'default-run' from {krate}");
             if let Some(&mut Value::Table(ref mut package)) = self.table.get_mut("package") {
                 package.remove("default-run");
             }
@@ -390,6 +390,7 @@ pub enum PrepareError {
     /// Some of the dependencies do not exist anymore.
     #[error("the crate depends on missing dependencies: \n\n{0}")]
     MissingDependencies(String),
+    /// Some not further detaild error
     #[error("prepare failed without further details")]
     Unknown,
 }

src/toolchain.rs

@@ -407,8 +407,7 @@ impl Toolchain {
                 .collect()),
             Err(_) if not_installed => Err(ToolchainError::NotInstalled.into()),
             Err(err) => Err(anyhow!(err).context(format!(
-                "failed to read the list of installed {}s for {} with rustup",
-                thing, name
+                "failed to read the list of installed {thing}s for {name} with rustup"
             ))),
         }
     }
@@ -419,7 +418,7 @@ impl Toolchain {
         Command::new(workspace, &RUSTUP)
             .args(&["toolchain", "uninstall", &name])
             .run()
-            .with_context(|| format!("unable to uninstall toolchain {} via rustup", name))?;
+            .with_context(|| format!("unable to uninstall toolchain {name} via rustup"))?;
         Ok(())
     }
 

src/tools/rustup.rs

@@ -49,7 +49,7 @@ impl Tool for Rustup {
             .error_for_status()?;
 
         let tempdir = tempdir()?;
-        let installer = &tempdir.path().join(format!("rustup-init{}", EXE_SUFFIX));
+        let installer = &tempdir.path().join(format!("rustup-init{EXE_SUFFIX}"));
         {
             let mut file = File::create(installer)?;
             io::copy(&mut resp, &mut file)?;
@@ -81,7 +81,7 @@ impl Tool for Rustup {
         Command::new(workspace, &RUSTUP)
             .args(&["update", MAIN_TOOLCHAIN_NAME])
             .run()
-            .with_context(|| format!("failed to update main toolchain {}", MAIN_TOOLCHAIN_NAME))?;
+            .with_context(|| format!("failed to update main toolchain {MAIN_TOOLCHAIN_NAME}"))?;
         Ok(())
     }
 }

src/utils.rs

@@ -35,7 +35,7 @@ pub(crate) fn file_lock<T>(
     let mut message_displayed = false;
     while let Err(err) = file.try_lock_exclusive() {
         if !message_displayed && err.kind() == fs2::lock_contended_error().kind() {
-            warn!("blocking on other processes finishing to {}", msg);
+            warn!("blocking on other processes finishing to {msg}");
             message_displayed = true;
         }
         file.lock_exclusive()?;