Mention somewhere that the latest commit should be used
Nessuno ha ancora preso questa issue.
- Lingua principale
- Rust
- Stelle
- 3.4k
- Fork
- 126
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
In the initial crate setup as mentioned in the book, it tells you to setup spirv_builder by adding the 0.9 version to your Cargo.toml: https://rust-gpu.github.io/rust-gpu/book/writing-shader-crates.html#using-spirv-builder
This of course does not work because the last spirv_builder release is ancient. In fact, the rust-toochain.toml mentioned immediately before this does not match the one from spirv_builder so upon doing this the crate won't even compile. This is easy enough to fix:
[build-dependencies]
# maybe use a commit if you need some semblance of stability
spirv-builder = { git = "https://github.com/Rust-GPU/rust-gpu", revision = "master" }
Figuring this out however was challenging. I feel like this should be mentioned somewhere in this section of the book.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia dalla sezione “Using spirv-builder” del libro in writing-shader-crates.html e confronta la configurazione di spirv_builder 0.9 con quella attuale del repository. Aggiorna la sezione per spiegare che deve essere usato il commit più recente o una git revision appropriata, e assicurati che le istruzioni corrispondano al rust-toochain.toml menzionato. Il lavoro è completato quando un nuovo arrivato viene avvisato che la vecchia release non funziona.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- rust
- Ambito
- documentation
- Tipo di issue
- Documentazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 55/100