[Migrated] spirv-builder could show warnings from building shaders.
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Rust
- Sterne
- 3.4k
- Forks
- 126
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Issue automatically imported from old repo: https://github.com/EmbarkStudios/rust-gpu/issues/880
Old labels: t: enhancement
Originally creatd by eddyb on 2022-07-04T06:48:36Z
Cargo build scripts have the ability to produce output through Cargo via cargo:warning=MESSAGE even when successful.
We should probably make spirv-builder transform stderr lines from its nested cargo build (of the shaders) into cargo:warning, but we may need to use the JSON output mode to be able to tell apart Cargo output from rustc output (as we presumably don't want to dump Compiling ... messages as "warnings").
We already pass --message-format=json-render-diagnostics to the nested cargo build though (to parse out "artifact notifications"), so it should be pretty easy to further rely on the JSON output.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der Verarbeitung des verschachtelten cargo build durch spirv-builder, die bereits --message-format=json-render-diagnostics verwendet, um Artefaktbenachrichtigungen zu analysieren. Ermittle, wie Cargo-Warnmeldungen in dieser Ausgabe dargestellt werden, und stelle dann sicher, dass relevante Warnungen aus dem Shader-Build den äußeren Build als cargo:warning-Zeilen erreichen, ohne gewöhnliche Cargo- oder rustc-Ausgaben in Warnungen umzuwandeln.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust
- Bereich
- build-system
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 48/100