Remove cargo fetch step

We no longer need this step because bootloader is now a normal dependency.

Author: phil-opp

src/subcommand/build.rs

@@ -1,5 +1,5 @@
 use crate::{args::Args, builder::Builder, config, ErrorMessage};
-use std::{path::PathBuf, process};
+use std::path::PathBuf;
 
 pub(crate) fn build(mut args: Args) -> Result<(), ErrorMessage> {
     let builder = Builder::new(args.manifest_path().clone())?;
@@ -15,8 +15,6 @@ pub(crate) fn build_impl(
     args: &Args,
     quiet: bool,
 ) -> Result<Vec<PathBuf>, ErrorMessage> {
-    run_cargo_fetch(&args).map_err(|()| "cargo fetch failed")?;
-
     let executables = builder.build_kernel(&args.cargo_args, quiet)?;
     if executables.len() == 0 {
         Err("no executables built")?;
@@ -40,16 +38,3 @@ pub(crate) fn build_impl(
     Ok(bootimages)
 }
 
-fn run_cargo_fetch(args: &Args) -> Result<(), ()> {
-    let mut command = process::Command::new("cargo");
-    command.arg("fetch");
-    if let Some(manifest_path) = args.manifest_path() {
-        command.arg("--manifest-path");
-        command.arg(manifest_path);
-    }
-    if command.status().map(|s| s.success()).unwrap_or(false) {
-        Ok(())
-    } else {
-        Err(())
-    }
-}