[Migrated] spirv-builder should declare dependencies on Cargo.toml files.
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/404
Old labels: t: enhancement,s: needs update
Originally creatd by eddyb on 2021-02-01T12:18:03Z
For #403 I changed the workspace Cargo.toml, but that didn't result in a rebuild of sky-shader, whereas something like this does cause a rebuild (of example-runner-wgpu, in this example):
[profile.dev.package."example-runner-wgpu"]
opt-level = 3
So I think we need spirv-builder to get the workspace Cargo.toml (and Cargo.lock, I suppose) paths from Cargo somehow, and also list those out as "rebuild if changed".
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 damit, spirv-builder nachzuverfolgen und zu untersuchen, wie es Cargo-Metadaten für sky-shader abruft. Vergleiche anschließend das Verhalten von Cargo.toml und Cargo.lock des Workspace mit dem im Issue beschriebenen Fall example-runner-wgpu. Bestätige, wie Cargo Dateien für einen Rebuild markiert, und füge gezielte Tests hinzu oder aktualisiere sie, sodass Änderungen am Workspace-Manifest den erwarteten Rebuild auslösen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust
- Bereich
- build-system
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100