Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Declarative build_tarballs.jl parsing! #201

Merged
merged 2 commits into from
Nov 17, 2019
Merged

Conversation

staticfloat
Copy link
Member

This enables us to level up our CI game, namely:

  • Download sources once, to avoid clobbering when multiple jobs start at once
  • Run a separate job PER TARGET PLATFORM; not only does this increase parallelism, it also makes debugging easier.

@staticfloat staticfloat force-pushed the sf/ci_double_expansion branch 2 times, most recently from 96c8d2e to 293a33b Compare November 15, 2019 21:58
@giordano giordano added the meta 🌳 Issues and pull requested related to Yggdrasil itself label Nov 15, 2019
azure-pipelines.yml Outdated Show resolved Hide resolved
@staticfloat
Copy link
Member Author

image

image

This enables us to level up our CI game, namely:

* Download sources once, to avoid clobbering when multiple jobs start at once
* Run a separate job PER TARGET PLATFORM; not only does this increase parallelism, it also makes debugging easier.
Currently contains:

* C
* C++
* Fortran
* Rust
* Go

The `Rust` test suite is quite restricted in platform, as we have a bit
of breakage there.
@staticfloat staticfloat merged commit 74c30ca into master Nov 17, 2019
@giordano giordano deleted the sf/ci_double_expansion branch November 17, 2019 22:55
@giordano
Copy link
Member

giordano commented Nov 17, 2019

Apart from making it easier to inspect the log of the build for a specific platform, this allows better parallelisation of the jobs:

Great job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta 🌳 Issues and pull requested related to Yggdrasil itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants