Skip to content

Conversation

@yooakim
Copy link
Contributor

@yooakim yooakim commented Dec 12, 2017

I belive that wget is more commonly installed on later distributions so it makes sense to use wget in the commands to install.

I belive that wget is more commonly installed on later distributions so it makes sense to use wget in the commands to install.
@PRMerger3
Copy link
Contributor

@yooakim : Thanks for your contribution to the SQL documentation! The author, @rothja, has been notified to review your proposed change.

@rothja
Copy link
Collaborator

rothja commented Dec 13, 2017

@yooakim Thanks for the feedback. When you went through the steps, did you have to install curl? I usually use Azure Ubuntu machines for testing, and curl seems to already be on those images. Thanks!

@craigg-msft
Copy link
Contributor

@yooakim thanks again for your time. Did you have feedback to @rothja question or should I close this PR? Thank you.

@yooakim
Copy link
Contributor Author

yooakim commented Jan 3, 2018

Sorry for sluggish response, Yes @rothja - I had to install CURL on the new versions of the Desktop versions

@rothja
Copy link
Collaborator

rothja commented Jan 3, 2018

@yooakim Thanks. And no problem on the delayed response. I've been out of the office for over a week. Thanks for confirming. Let me run a test and, if wget requires no additional installation, I'll change the instructions. Appreciate the tip.

@rothja
Copy link
Collaborator

rothja commented Jan 5, 2018

@yooakim I verified your commands worked. I also ran it by engineering. They looked at curl and found that it is installed by default by server installations but not by standard ones. Since wget is installed with both, your recommendation is a good one. I'm approving your change here. Thanks! #sign-off

@rothja
Copy link
Collaborator

rothja commented Jan 5, 2018

#sign-off

@craigg-msft craigg-msft merged commit 0eb3d6d into MicrosoftDocs:live Jan 8, 2018
@craigg-msft
Copy link
Contributor

thank you @yooakim and @rothja merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants