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

Install cargo via rustup in alpine image #316

Merged
merged 3 commits into from
Jul 19, 2021

Conversation

mirgee
Copy link
Contributor

@mirgee mirgee commented Jul 12, 2021

Cargo has been installed via apk until now, limiting us with regards to the toolchain version we used to compile our images. Via rustup, we have more freedom to choose the toolchain version.

Signed-off-by: Miroslav Kovar miroslavkovar@protonmail.com

Signed-off-by: Miroslav Kovar <miroslavkovar@protonmail.com>
@mirgee mirgee added hotfix High priority fixes of unexpected issues ci labels Jul 12, 2021
@mirgee mirgee requested a review from a team as a code owner July 12, 2021 10:39
@Patrik-Stas
Copy link
Contributor

Patrik-Stas commented Jul 12, 2021

Additional background info
#180

Patrik-Stas
Patrik-Stas previously approved these changes Jul 12, 2021
@codecov-commenter
Copy link

codecov-commenter commented Jul 12, 2021

Codecov Report

Merging #316 (541cb92) into master (3c2165d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #316   +/-   ##
=======================================
  Coverage   49.72%   49.72%           
=======================================
  Files         156      156           
  Lines       19634    19634           
  Branches     6205     6205           
=======================================
  Hits         9763     9763           
  Misses       5070     5070           
  Partials     4801     4801           
Flag Coverage Δ
integration 22.13% <ø> (ø)
unittests 45.54% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3c2165d...541cb92. Read the comment docs.

@Patrik-Stas
Copy link
Contributor

 error: failed to run custom build command for `openssl-sys v0.9.65`

I don't know why the changes trigger this error, but the problem might disappear with these changes #317

@mirgee mirgee force-pushed the hotfix/rustc-version-in-docker-image branch 2 times, most recently from f3268d0 to 0b71ea8 Compare July 12, 2021 19:18
Signed-off-by: Miroslav Kovar <miroslavkovar@protonmail.com>
@mirgee mirgee force-pushed the hotfix/rustc-version-in-docker-image branch from 0b71ea8 to b0c879e Compare July 12, 2021 20:23
@mirgee mirgee requested a review from Patrik-Stas July 13, 2021 10:17
ci/libvcx.dockerfile Outdated Show resolved Hide resolved
Signed-off-by: Miroslav Kovar <miroslavkovar@protonmail.com>
@Patrik-Stas Patrik-Stas merged commit d15e558 into master Jul 19, 2021
@Patrik-Stas Patrik-Stas deleted the hotfix/rustc-version-in-docker-image branch July 19, 2021 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci hotfix High priority fixes of unexpected issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants