Skip to content
New issue

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

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

Already on GitHub? # to your account

Update cargo install instructions to declare 'trunk' branch #27

Merged
merged 3 commits into from
Dec 18, 2020

Conversation

lopopolo
Copy link
Member

@lopopolo lopopolo commented Dec 18, 2020

This PR updates the Dockerfiles to use ARTICHOKE_NIGHTLY_VER if set to anything other than latest to install a specific git revision. This matches the behavior of artichoke/nightly where the git revision is resolved at workflow-time and propagated through the installation workflow. This change ensures that the images tagged with a git SHA match the built artichoke they contain.

See also rust-lang/cargo#3517.

Breakage caused by artichoke/artichoke#961.

@lopopolo lopopolo added A-build Area: CI build infrastructure. A-dockerfile Area: Dockerfile builds. labels Dec 18, 2020
Default to trunk if ARTICHOKE_NIGHTLY_VER is not set via the docker command line.
@lopopolo lopopolo merged commit 432d90f into trunk Dec 18, 2020
@lopopolo lopopolo deleted the cargo-install-branch-trunk branch December 18, 2020 14:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-build Area: CI build infrastructure. A-dockerfile Area: Dockerfile builds.
Development

Successfully merging this pull request may close these issues.

1 participant