[Migrated] CI should check that we don't build `rustc_codegen_spirv` more than once.
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 3.4k
- Forks
- 125
- PR merge metrics
- No merged PRs in 30d
Description
Issue automatically imported from old repo: https://github.com/EmbarkStudios/rust-gpu/issues/549
Old labels: t: enhancement,c: meta,a: test
Originally creatd by eddyb on 2021-03-29T07:28:00Z
See #546 for some context and ideas - especially https://github.com/EmbarkStudios/rust-gpu/pull/546#issuecomment-809089160.
Sadly, this is likely blocked on moving CI to use --release to build examples (which depend on "build dependency release mode rustc_codegen_spirv via the spirv-builder-using build scripts), otherwise the problem can't be reproduced in the first place (AFAIK).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with issue #546 and the linked comment on pull request #546, then inspect the repository's CI configuration and how examples use spirv-builder. First establish the required --release CI change so duplicate builds are reproducible; done means CI detects and reports building rustc_codegen_spirv more than once.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- build-system, ci-cd
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100