Diagram editor fails to build on Windows: build.rs can't find pnpm (pnpm.cmd)

Aperta Adatta ai principianti
  • Qualcosa non va?
#235 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

@diyagarg09 ci sta già lavorando.

Dal 12/11/2026.

  • #238 di @diyagarg09 — aperta

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
76/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
rust
Ambito
build-system, devops

Direzione di ricerca

Inizia da diagram-editor/build.rs e segui le due chiamate al comando pnpm utilizzate durante la build del frontend. Verifica come lo script di build esegue comandi esterni su Windows, quindi compila l’esempio di calcolatrice con la funzionalità frontend abilitata. Il lavoro è completato quando la build riesce se pnpm è installato come pnpm.cmd e continua a funzionare su Linux e macOS.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Building the diagram editor from the repo fails on Windows because diagram-editor/build.rs can't find pnpm.

Environment: Windows 11, Rust 1.98.0, Node 22.23.3, pnpm 10.30.3 (installed with npm), crossflow main at 36d0f06.

Steps:

cd diagram-editor
pnpm install
cargo build --manifest-path ../examples/diagram/calculator/Cargo.toml

(pnpm dev:backend hits the same build, and also fails earlier in cmd because of the BUILD_FRONTEND=1 prefix.)

Error:

error: failed to run custom build command for `crossflow_diagram_editor v0.0.7`
  thread 'main' panicked at diagram-editor\build.rs:22:14:
  Failed to execute pnpm install: Error { kind: NotFound, message: "program not found" }

Cause: on Windows, npm and corepack install pnpm as a pnpm.cmd shim. Command::new("pnpm") only resolves pnpm.exe, so it never finds it. Since build_frontend() always calls build_uncompressed_pnpm() to compute the hash, any build with the frontend feature (which basic_executor enables) hits this. Published crates aren't affected, because build.rs is excluded from the package. The Windows CI job only builds --package crossflow, so it doesn't catch it.

Proposed fix: route both calls through a small helper that tries pnpm.cmd first on Windows and falls back to pnpm (for a standalone pnpm.exe):

fn pnpm(args: &[&str]) -> std::io::Result<std::process::ExitStatus> {
    let run = |program: &str| Command::new(program).args(args).status();
    if cfg!(windows) {
        run("pnpm.cmd").or_else(|_| run("pnpm"))
    } else {
        run("pnpm")
    }
}

let status =
    pnpm(&["install", "--frozen-lockfile"]).expect("Failed to execute pnpm install");
// ...
let status = pnpm(&["build"]).expect("Failed to execute pnpm build");

I tested this locally on Windows. With BUILD_FRONTEND=1 and no standalone pnpm.exe on PATH, the calculator example builds and runs with the change, and fails with the error above without it. cargo fmt --check and cargo clippy -p crossflow_diagram_editor are clean. Behaviour on Linux and macOS is unchanged.

Optionally, adding a cargo build -p crossflow_diagram_editor step to ci_windows.yaml would keep this covered.

Happy to open a PR with this if it looks good.

Lingua principale
Rust
Stelle
75
Fork
18
Merge medio
12g 12h
PR unite (30g)
3

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di open-rmf/crossflow

Tutte le issue di open-rmf/crossflow

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.