feat(xtask): add log

Signed-off-by: Woshiluo Luo <woshiluo.luo@outlook.com>

Cargo.lock

@@ -8,6 +8,15 @@ version = "0.0.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "8a01ba40421eca6c4f1afcedd8465fba6d9e5ef8e0e13060d0141e4cded4ab4a"
 
+[[package]]
+name = "aho-corasick"
+version = "1.1.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "8e60d3430d3a69478ad0993f19238d2df97c507009a52b3c10addcd7f6bcb916"
+dependencies = [
+ "memchr",
+]
+
 [[package]]
 name = "anstream"
 version = "0.6.18"
@@ -57,6 +66,17 @@ dependencies = [
  "windows-sys",
 ]
 
+[[package]]
+name = "atty"
+version = "0.2.14"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "d9b39be18770d11421cdb1b9947a45dd3f37e93092cbf377614828a319d5fee8"
+dependencies = [
+ "hermit-abi",
+ "libc",
+ "winapi",
+]
+
 [[package]]
 name = "autocfg"
 version = "1.4.0"
@@ -145,6 +165,19 @@ version = "1.0.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "361a90feb7004eca4019fb28352a9465666b24f840f5c3cddf0ff13920590b89"
 
+[[package]]
+name = "env_logger"
+version = "0.7.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "44533bbbb3bb3c1fa17d9f2e4e38bbbaf8396ba82193c4cb1b6445d711445d36"
+dependencies = [
+ "atty",
+ "humantime",
+ "log",
+ "regex",
+ "termcolor",
+]
+
 [[package]]
 name = "fast-trap"
 version = "0.0.1"
@@ -157,12 +190,36 @@ version = "0.5.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "2304e00983f87ffb38b55b444b5e3b60a884b5d30c0fca7d82fe33449bbe55ea"
 
+[[package]]
+name = "hermit-abi"
+version = "0.1.19"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "62b467343b94ba476dcb2500d242dadbb39557df889310ac77c5d99100aaac33"
+dependencies = [
+ "libc",
+]
+
+[[package]]
+name = "humantime"
+version = "1.3.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "df004cfca50ef23c36850aaaa59ad52cc70d0e90243c3c7737a4dd32dc7a3c4f"
+dependencies = [
+ "quick-error",
+]
+
 [[package]]
 name = "is_terminal_polyfill"
 version = "1.70.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "7943c866cc5cd64cbc25b2e01621d07fa8eb2a1a23160ee81ce38704e97b8ecf"
 
+[[package]]
+name = "libc"
+version = "0.2.169"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "b5aba8db14291edd000dfcc4d620c7ebfb122c613afb886ca8803fa4e128a20a"
+
 [[package]]
 name = "lock_api"
 version = "0.4.12"
@@ -179,6 +236,12 @@ version = "0.4.22"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "a7a70ba024b9dc04c27ea2f0c0548feb474ec5c54bba33a7f72f873a39d07b24"
 
+[[package]]
+name = "memchr"
+version = "2.7.4"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "78ca9ab1a0babb1e7d5695e3530886289c18cf2f87ec19a575a0abdce112e3a3"
+
 [[package]]
 name = "naked-function"
 version = "0.1.5"
@@ -218,6 +281,16 @@ version = "0.0.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "2ad606bf31d67b0e10a161b7df7d6a97dda7be22ce4bebcff889476e867c9b7a"
 
+[[package]]
+name = "pretty_env_logger"
+version = "0.4.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "926d36b9553851b8b0005f1275891b392ee4d2d833852c417ed025477350fb9d"
+dependencies = [
+ "env_logger",
+ "log",
+]
+
 [[package]]
 name = "proc-macro2"
 version = "1.0.92"
@@ -227,6 +300,12 @@ dependencies = [
  "unicode-ident",
 ]
 
+[[package]]
+name = "quick-error"
+version = "1.2.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "a1d01941d82fa2ab50be1e79e6714289dd7cde78eba4c074bc5a4374f650dfe0"
+
 [[package]]
 name = "quote"
 version = "1.0.37"
@@ -246,6 +325,35 @@ dependencies = [
  "spin",
 ]
 
+[[package]]
+name = "regex"
+version = "1.11.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "b544ef1b4eac5dc2db33ea63606ae9ffcfac26c1416a2806ae0bf5f56b201191"
+dependencies = [
+ "aho-corasick",
+ "memchr",
+ "regex-automata",
+ "regex-syntax",
+]
+
+[[package]]
+name = "regex-automata"
+version = "0.4.9"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "809e8dc61f6de73b46c85f4c96486310fe304c434cfa43669d7b40f711150908"
+dependencies = [
+ "aho-corasick",
+ "memchr",
+ "regex-syntax",
+]
+
+[[package]]
+name = "regex-syntax"
+version = "0.8.5"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "2b15c43186be67a4fd63bee50d0303afffcef381492ebe2c5d87f324e1b8815c"
+
 [[package]]
 name = "riscv"
 version = "0.11.1"
@@ -479,6 +587,15 @@ dependencies = [
  "unicode-ident",
 ]
 
+[[package]]
+name = "termcolor"
+version = "1.4.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "06794f8f6c5c898b3275aebefa6b8a1cb24cd2c6c79397ab15774837a0bc5755"
+dependencies = [
+ "winapi-util",
+]
+
 [[package]]
 name = "uart16550"
 version = "0.0.1"
@@ -521,6 +638,37 @@ dependencies = [
  "vcell",
 ]
 
+[[package]]
+name = "winapi"
+version = "0.3.9"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "5c839a674fcd7a98952e593242ea400abe93992746761e38641405d28b00f419"
+dependencies = [
+ "winapi-i686-pc-windows-gnu",
+ "winapi-x86_64-pc-windows-gnu",
+]
+
+[[package]]
+name = "winapi-i686-pc-windows-gnu"
+version = "0.4.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "ac3b87c63620426dd9b991e5ce0329eff545bccbbb34f3be09ff6fb6ab51b7b6"
+
+[[package]]
+name = "winapi-util"
+version = "0.1.9"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "cf221c93e13a30d793f7645a0e7762c55d169dbb0a49671918a2319d289b10bb"
+dependencies = [
+ "windows-sys",
+]
+
+[[package]]
+name = "winapi-x86_64-pc-windows-gnu"
+version = "0.4.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "712e227841d057c1ee1cd2fb22fa7e5a5461ae8e48fa2ca79ec42cfc1931183f"
+
 [[package]]
 name = "windows-sys"
 version = "0.59.0"
@@ -599,6 +747,8 @@ name = "xtask"
 version = "0.1.0"
 dependencies = [
  "clap",
+ "log",
+ "pretty_env_logger",
 ]
 
 [[package]]

xtask/Cargo.toml

@@ -7,3 +7,5 @@ repository.workspace = true
 
 [dependencies]
 clap = { version = "4.5.4", features = ["derive", "env", "suggestions"] }
+log = "0.4.21"
+pretty_env_logger = "0.4"

xtask/src/bench.rs

@@ -25,6 +25,7 @@ pub fn run(arg: &BenchArg) -> Option<ExitStatus> {
         .join(arch)
         .join("release");
 
+    info!("Building bench kernel");
     cargo::Cargo::new("build")
         .package("rustsbi-bench-kernel")
         .target(arch)
@@ -32,6 +33,7 @@ pub fn run(arg: &BenchArg) -> Option<ExitStatus> {
         .status()
         .ok()?;
 
+    info!("Copy to binary");
     let exit_status = Command::new("rust-objcopy")
         .args(["-O", "binary"])
         .arg("--binary-architecture=riscv64")
@@ -41,6 +43,7 @@ pub fn run(arg: &BenchArg) -> Option<ExitStatus> {
         .ok()?;
 
     if arg.pack {
+        info!("Pack to image");
         match fs::exists(target_dir.join("rustsbi-prototyper.bin")) {
             Ok(true) => {}
             Ok(false) => {

xtask/src/main.rs

@@ -7,6 +7,10 @@ mod bench;
 mod prototyper;
 mod test;
 
+extern crate pretty_env_logger;
+#[macro_use]
+extern crate log;
+
 use crate::bench::BenchArg;
 use crate::prototyper::PrototyperArg;
 use crate::test::TestArg;
@@ -30,14 +34,17 @@ enum Cmd {
 }
 
 fn main() -> ExitCode {
+    pretty_env_logger::init();
     if let Some(code) = match Cli::parse().cmd {
         Cmd::Prototyper(ref arg) => prototyper::run(arg),
         Cmd::Test(ref arg) => test::run(arg),
         Cmd::Bench(ref arg) => bench::run(arg),
     } {
         if code.success() {
+            info!("Finished");
             return ExitCode::SUCCESS;
         }
     }
+    error!("Failed to run task!");
     ExitCode::FAILURE
 }

xtask/src/prototyper.rs

@@ -34,7 +34,8 @@ pub fn run(arg: &PrototyperArg) -> Option<ExitStatus> {
         .join(arch)
         .join("release");
 
-    let status = cargo::Cargo::new("build")
+    info!("Building Protoyper");
+    cargo::Cargo::new("build")
         .package("rustsbi-prototyper")
         .target(arch)
         .unstable("build-std", ["core"])
@@ -52,40 +53,44 @@ pub fn run(arg: &PrototyperArg) -> Option<ExitStatus> {
         .status()
         .ok()?;
 
-    if status.success() {
-        let exit_status = Command::new("rust-objcopy")
-            .args(["-O", "binary"])
-            .arg("--binary-architecture=riscv64")
-            .arg(target_dir.join("rustsbi-prototyper"))
-            .arg(target_dir.join("rustsbi-prototyper.bin"))
-            .status()
-            .ok()?;
-
-        if arg.payload.is_some() {
-            fs::copy(
-                target_dir.join("rustsbi-prototyper"),
-                target_dir.join("rustsbi-prototyper-payload.elf"),
-            )
-            .ok()?;
-            fs::copy(
-                target_dir.join("rustsbi-prototyper.bin"),
-                target_dir.join("rustsbi-prototyper-payload.bin"),
-            )
-            .ok()?;
-        } else {
-            fs::copy(
-                target_dir.join("rustsbi-prototyper"),
-                target_dir.join("rustsbi-prototyper-dynamic.elf"),
-            )
-            .ok()?;
-            fs::copy(
-                target_dir.join("rustsbi-prototyper.bin"),
-                target_dir.join("rustsbi-prototyper-dynamic.bin"),
-            ).ok()?;
-        }
+    info!("Copy to binary");
+    let exit_status = Command::new("rust-objcopy")
+        .args(["-O", "binary"])
+        .arg("--binary-architecture=riscv64")
+        .arg(target_dir.join("rustsbi-prototyper"))
+        .arg(target_dir.join("rustsbi-prototyper.bin"))
+        .status()
+        .ok()?;
+    if !exit_status.success() {
+        error!("Failed to exec rust-objcopy, please check if cargo-binutils has been installed?");
         return Some(exit_status);
+    }
+
+    if arg.payload.is_some() {
+        info!("Copy for payload mode");
+        fs::copy(
+            target_dir.join("rustsbi-prototyper"),
+            target_dir.join("rustsbi-prototyper-payload.elf"),
+        )
+        .ok()?;
+        fs::copy(
+            target_dir.join("rustsbi-prototyper.bin"),
+            target_dir.join("rustsbi-prototyper-payload.bin"),
+        )
+        .ok()?;
     } else {
-        eprintln!("Build failed with status: {:?}", status);
-        return Some(status);
+        info!("Copy for dynamic mode");
+        fs::copy(
+            target_dir.join("rustsbi-prototyper"),
+            target_dir.join("rustsbi-prototyper-dynamic.elf"),
+        )
+        .ok()?;
+        fs::copy(
+            target_dir.join("rustsbi-prototyper.bin"),
+            target_dir.join("rustsbi-prototyper-dynamic.bin"),
+        )
+        .ok()?;
     }
+
+    Some(exit_status)
 }

xtask/src/test.rs

@@ -25,6 +25,7 @@ pub fn run(arg: &TestArg) -> Option<ExitStatus> {
         .join(arch)
         .join("release");
 
+    info!("Building test kernel");
     cargo::Cargo::new("build")
         .package("rustsbi-test-kernel")
         .target(arch)
@@ -32,6 +33,7 @@ pub fn run(arg: &TestArg) -> Option<ExitStatus> {
         .status()
         .ok()?;
 
+    info!("Copy to binary");
     let exit_status = Command::new("rust-objcopy")
         .args(["-O", "binary"])
         .arg("--binary-architecture=riscv64")
@@ -41,6 +43,7 @@ pub fn run(arg: &TestArg) -> Option<ExitStatus> {
         .ok()?;
 
     if arg.pack {
+        info!("Pack to image");
         match fs::exists(target_dir.join("rustsbi-prototyper.bin")) {
             Ok(true) => {}
             Ok(false) => {