Skip to content

Update install_linux.md - #1736

Merged
vilmibm merged 2 commits into
cli:trunkfrom
dbruns:patch-linux-docs
Sep 17, 2020
Merged

Update install_linux.md#1736
vilmibm merged 2 commits into
cli:trunkfrom
dbruns:patch-linux-docs

Conversation

@dbruns

@dbruns dbruns commented Sep 17, 2020

Copy link
Copy Markdown
Contributor

Remove non-existant option: -u

`$ sudo apt-add-repository -u https://cli.github.com/packages
Usage: apt-add-repository [options] repository

apt-add-repository: error: no such option: -u
`

This does not work on Mint and older versions of Ubuntu, so removing the -you and manually doing the apt update should cover all the bases.

Remove non-existant option: -u
@vilmibm

vilmibm commented Sep 17, 2020

Copy link
Copy Markdown
Contributor

Thanks; out of curiosity, what distro are you on? -u is supported for me in a recent ubuntu.

@vilmibm vilmibm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The purpose of the -u was to avoid a apt update; taking it out should then include an apt update call before the install.

Add apt update
@dbruns

dbruns commented Sep 17, 2020

Copy link
Copy Markdown
Contributor Author

The purpose of the -u was to avoid a apt update; taking it out should then include an apt update call before the install.

Updated

@dbruns
dbruns requested a review from vilmibm September 17, 2020 16:50
@vilmibm

vilmibm commented Sep 17, 2020

Copy link
Copy Markdown
Contributor

thanks!

@vilmibm
vilmibm merged commit 9a8deb0 into cli:trunk Sep 17, 2020
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