Always show example description before compiling it (#3622)

examples/src/bin/lp_core_basic.rs

@@ -3,7 +3,7 @@
 //! Code on LP core increments a counter and continuously toggles LED. The
 //! current value is printed by the HP core.
 //!
-//! Make sure to first compile the `esp-lp-hal/examples/blinky.rs` example
+//! ⚠️ Make sure to first compile the `esp-lp-hal/examples/blinky.rs` example ⚠️
 //!
 //! The following wiring is assumed:
 //! - LED => GPIO1

xtask/src/commands/mod.rs

@@ -85,7 +85,6 @@ pub fn examples(workspace: &Path, mut args: ExamplesArgs, action: CargoAction) -
     // Execute the specified action:
     match action {
         CargoAction::Build(out_path) => build_examples(args, examples, &package_path, &out_path),
-        CargoAction::Run if args.example.is_some() => run_example(args, examples, &package_path),
         CargoAction::Run => run_examples(args, examples, &package_path),
     }
 }

xtask/src/commands/run.rs

@@ -4,7 +4,7 @@ use std::{
     process::Command,
 };
 
-use anyhow::{bail, ensure, Context as _, Result};
+use anyhow::{Context as _, Result, bail, ensure};
 use clap::{Args, Subcommand};
 use esp_metadata::Chip;
 
@@ -117,49 +117,33 @@ pub fn run_elfs(args: RunElfsArgs) -> Result<()> {
     Ok(())
 }
 
-pub fn run_example(args: ExamplesArgs, examples: Vec<Metadata>, package_path: &Path) -> Result<()> {
-    // Determine the appropriate build target for the given package and chip:
-    let target = args.package.target_triple(&args.chip)?;
-
-    // Filter the examples down to only the binary we're interested in, assuming it
-    // actually supports the specified chip:
-    let mut found_one = false;
-    for example in examples.iter().filter(|ex| ex.matches(&args.example)) {
-        found_one = true;
-        crate::execute_app(
-            package_path,
-            args.chip,
-            target,
-            example,
-            CargoAction::Run,
-            1,
-            args.debug,
-        )?;
-    }
-
-    ensure!(
-        found_one,
-        "Example not found or unsupported for {}",
-        args.chip
-    );
-
-    Ok(())
-}
-
 pub fn run_examples(
     args: ExamplesArgs,
     examples: Vec<Metadata>,
     package_path: &Path,
 ) -> Result<()> {
+    let mut examples = examples;
+
     // Determine the appropriate build target for the given package and chip:
     let target = args.package.target_triple(&args.chip)?;
 
-    // Filter the examples down to only the binaries we're interested in
+    let single_example = args.example.is_some();
-    let mut examples: Vec<Metadata> = examples
+
-        .iter()
+    // Filter the examples down to only the binaries supported by the given chip
-        .filter(|ex| ex.supports_chip(args.chip))
+    examples.retain(|ex| ex.supports_chip(args.chip));
-        .cloned()
+
-        .collect();
+    // User requested to run exactly one example
+    if single_example {
+        // Filter the examples down to only the binary we're interested in
+        examples.retain(|ex| ex.matches(&args.example));
+
+        ensure!(
+            examples.len() == 1,
+            "Example not found or unsupported for {}",
+            args.chip
+        );
+    }
+
     examples.sort_by_key(|ex| ex.tag());
 
     let console = console::Term::stdout();