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

Prioritize official Microsoft Linux Repo for azure-cli #3192

Closed
wants to merge 2 commits into from

Conversation

jiasli
Copy link
Member

@jiasli jiasli commented Jun 15, 2022

Copy the prioritization logic from automation script (Azure/azure-cli#20557) to manual steps.

This prevents installing unofficial azure-cli on Debian sid (Azure/azure-cli#19640).

@opbld33
Copy link

opbld33 commented Jun 15, 2022

Docs Build status updates of commit ea68c7c:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/includes/cli-install-linux-apt.md ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@jiasli jiasli changed the title Prioritize Microsoft Linux Repo for azure-cli Prioritize official Microsoft Linux Repo for azure-cli Jun 15, 2022
Comment on lines 113 to 121
To do this, set the value of `AZ_REPO` manually when [adding the repository](#set-release). For Ubuntu distributions use the `jammy` repository, and for Debian distributions use `bullseye`:

```bash
# Ubuntu
AZ_REPO=jammy

# Debian
AZ_REPO=bullseye
```
Copy link
Member Author

Choose a reason for hiding this comment

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

Update the fallback version by the way since we now support jammy and bullseye.

Distributions released before Ubuntu Trusty and Debian Wheezy are not supported.

This sentence is removed because we now have this info on top of this page:

| Distribution | Version |
|:-------------|:--------|
| Ubuntu | 18.04 LTS (Bionic Beaver), 20.04 LTS (Focal Fossa), 21.10 (Impish Indri), 22.04 (Jammy Jellyfish) |
| Debian | 10 (Buster), 11 (Bullseye) |

@opbld30
Copy link

opbld30 commented Sep 13, 2022

Docs Build status updates of commit ed59eb0:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/includes/cli-install-linux-apt.md ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@dbradish-microsoft
Copy link
Collaborator

@bebound / @jiasli Should this PR be merged (with fixed conflict) or closed?

@dbradish-microsoft
Copy link
Collaborator

@jiasli, This PR is almost 2 years old. Please move it back to Draft so it doesn't get closed in June 2024 as part of my PR cleanup process. Thank you.

@dbradish-microsoft
Copy link
Collaborator

I am closing this PR as it is outdated. Please reopen and fix merge conflicts if this update is still needed.

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.

7 participants