Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

`third-party-licenses` fails on Windows when the parent process is PowerShell

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#4,745 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
76/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
powershell, rust
Bereich
build-system, tooling

Rechercherichtung

Beginne in tools/third-party-licenses/src/cargo.rs, insbesondere bei run(), und verfolge, wie cargo about aufgerufen und die Ausgabe geparst wird. Führe den Reproduktionsbefehl in PowerShell aus; fertig ist die Aufgabe, wenn das Lizenz-JSON erfolgreich gelesen wird und das Tool erfolgreich ist, wenn PowerShell ein übergeordneter Prozess ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Summary

tools/third-party-licenses runs cargo about generate --format json --locked and parses stdout. On Windows, if PowerShell appears anywhere in the parent process chain, cargo-about refuses to write JSON to stdout and exits with an error telling you to use -o / --output-file.

That breaks cargo run desktop (and any CI/agent that launches the build from PowerShell), even though cargo-about itself is installed and works when given an output file.

Environment

  • OS: Windows 10/11
  • Shell: PowerShell 5.1 / PowerShell 7 (also fails when an ancestor is PowerShell, e.g. Cursor/VS Code terminal, cargo-run spawned from pwsh)
  • cargo-about 0.9.x (behavior from EmbarkStudios/cargo-about#198 / PR that added is_powershell_parent())

Steps to reproduce

  1. On Windows, open PowerShell in a Graphite checkout.
  2. Ensure cargo-about is on PATH.
  3. Run:
cargo run -p third-party-licenses --features desktop

(or the same step via cargo run desktop)

Expected

License JSON is generated and the tool succeeds.

Actual

cargo about generate fails roughly with:

cargo-about should not redirect its output in powershell

(or similar wording pointing at -o, --output-file). Redirecting/piping from the Graphite side does not help: the check walks ancestor processes, so Command::output() capturing stdout is enough to trigger it whenever PowerShell is an ancestor.

Root cause

Current code in tools/third-party-licenses/src/cargo.rs (run()):

Command::new("cargo")
    .args(["about", "generate", "--format", "json", "--locked"])
    // … reads output.stdout

cargo-about intentionally blocks stdout under PowerShell (UTF-16 / encoding footgun). The supported API is writing to a file with -o.

Suggested fix

Write to a temp file with -o, then read and parse that file (delete afterward). Minimal patch:

fn run() -> Result<Output, Error> {
	let out_path = std::env::temp_dir().join(format!("graphite-about-{}.json", std::process::id()));
	let output = Command::new("cargo")
		.args([
			"about",
			"generate",
			"--format",
			"json",
			"--locked",
			"-o",
			out_path.to_str().ok_or_else(|| Error::Command("about output path is not UTF-8".into()))?,
		])
		.current_dir(env!("CARGO_WORKSPACE_DIR"))
		.output()
		.map_err(|e| Error::Io(e, "Failed to run cargo about generate".into()))?;

	if !output.status.success() {
		let _ = std::fs::remove_file(&out_path);
		return Err(Error::Command(format!(
			"cargo about generate failed:\n{}",
			String::from_utf8_lossy(&output.stderr)
		)));
	}

	let json = std::fs::read_to_string(&out_path).map_err(|e| Error::Io(e, format!("reading {}", out_path.display())))?;
	let _ = std::fs::remove_file(&out_path);

	serde_json::from_str(&json).map_err(|e| Error::Json(e, "Failed to parse cargo about generate JSON".into()))
}

This is also slightly more robust on non-Windows hosts (no reliance on capturing a large JSON blob from a pipe).

Workaround (today)

# Skip the failing step and continue the desktop bundle, or invoke about with -o manually:
cargo about generate --format json --locked -o $env:TEMP\about.json

References

  • EmbarkStudios/cargo-about#198 (PowerShell / stdout encoding; led to rejecting stdout under PowerShell)
  • Graphite call site: tools/third-party-licenses/src/cargo.rs → run()
  • Invoked from desktop flow: cargo run desktop → third-party-licenses with --features desktop
Vorherrschende Sprache
Rust
Sterne
27.7k
Forks
1.3k
Ø Merge
1 T. 3 Std.
Gemergte PRs (30 T.)
145

Entwicklungsumgebung

In Codespaces öffnen

Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.

  • Kein Dockerfile und keine Docker-Compose-Datei
  • Hat eine Pull-Request-Vorlage
  • Kein Beitragsleitfaden

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus GraphiteEditor/Graphite

Alle Issues in GraphiteEditor/Graphite

Ähnliche Issues

Weitere Issues zu Rust

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.