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

Makefile: Suppress package already installed error for tools #2008

Merged
merged 1 commit into from
Aug 16, 2019

Conversation

matevz
Copy link
Member

@matevz matevz commented Aug 16, 2019

The new rust version prints error on cargo install, if the package is already installed despite the --quiet switch. This hack just redirects error output to /dev/null to get rid of the message. See rust-lang/cargo#7116 for details.

@codecov
Copy link

codecov bot commented Aug 16, 2019

Codecov Report

Merging #2008 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2008   +/-   ##
=======================================
  Coverage   53.91%   53.91%           
=======================================
  Files         242      242           
  Lines       23637    23637           
=======================================
  Hits        12745    12745           
  Misses       9523     9523           
  Partials     1369     1369

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 3b1128a...b807c82. Read the comment docs.

@codecov
Copy link

codecov bot commented Aug 16, 2019

Codecov Report

Merging #2008 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2008   +/-   ##
=======================================
  Coverage   53.91%   53.91%           
=======================================
  Files         242      242           
  Lines       23637    23637           
=======================================
  Hits        12745    12745           
  Misses       9523     9523           
  Partials     1369     1369

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 3b1128a...d4d6695. Read the comment docs.

Makefile Outdated Show resolved Hide resolved
@matevz matevz changed the title Makefile: Supress package already installed error for tools Makefile: Suppress package already installed error for tools Aug 16, 2019
@matevz matevz force-pushed the matevz/feature/supress-cargo-install-error branch from 71b7fa1 to d4d6695 Compare August 16, 2019 11:17
@matevz matevz merged commit 5931711 into master Aug 16, 2019
@matevz matevz deleted the matevz/feature/supress-cargo-install-error branch August 16, 2019 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants