Skip to content
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.

Check if tput command exists before using it #8314

Merged
merged 2 commits into from Jan 2, 2018
Merged

Check if tput command exists before using it #8314

merged 2 commits into from Jan 2, 2018

Conversation

ghost
Copy link

@ghost ghost commented Jan 2, 2018

Fix #8311

@eerhardt

@eerhardt
Copy link
Member

eerhardt commented Jan 2, 2018

Thanks for the contribution, @kasper3.

Do you think you can remove the ncurses entry from here as well?

https://github.com/dotnet/cli/blob/master/scripts/docker/alpine.3.6/Dockerfile#L9

It will ensure this change is working correctly.

@ghost
Copy link
Author

ghost commented Jan 2, 2018

👍 Removed

@johnbeisner johnbeisner self-requested a review January 2, 2018 21:40
Copy link
Member

@eerhardt eerhardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @kasper3!

@johnbeisner johnbeisner merged commit ecdaa66 into dotnet:master Jan 2, 2018
@ghost ghost deleted the patch-1 branch January 2, 2018 22:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants