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

install from source: use native feeds for Ubuntu 22.04 #839

Merged
merged 2 commits into from
Aug 22, 2022

Conversation

ldennington
Copy link
Contributor

@ldennington ldennington commented Aug 15, 2022

The dotnet team recently added support for installing natively from Jammy feeds on
Ubuntu:

dotnet/core#7699

This caused our current installation to fail, as it caused conflicts with
the packages.microsoft.com feed we use in the install from source script
for Debian/Ubuntu.

This change modifies the Debian/Ubuntu dotnet install process to install from
Jammy feeds for users on Ubuntu 22.04 and greater while continuing to use
the packages.microsoft.com feed for Debian and older Ubuntu versions.

Here is a successful run of the validation workflow with these changes: https://github.com/ldennington/git-credential-manager/actions/runs/2884331734

Fixes #841

@ldennington ldennington self-assigned this Aug 15, 2022
@ldennington ldennington changed the title install from source: use Jammy feed for Ubuntu install from source: use Jammy feed for Ubuntu 22.04 Aug 16, 2022
@ldennington ldennington changed the title install from source: use Jammy feed for Ubuntu 22.04 install from source: use Jammy feeds for Ubuntu 22.04 Aug 16, 2022
Copy link
Contributor

@derrickstolee derrickstolee left a comment

Choose a reason for hiding this comment

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

This makes sense to me. Ignoring whitespace made the diff look really nice.

@ldennington ldennington force-pushed the ubuntu-jammy branch 6 times, most recently from 75bcbea to 0e43350 Compare August 18, 2022 17:48
The dotnet team recently added support for installing natively from Jammy feeds on
Ubuntu 22.04:

dotnet/core#7699

This unfortunately created problems with our current install from source
script, as it caused conflicts with
the packages.microsoft.com feed we use for Debian/Ubuntu.

This change modifies the Debian/Ubuntu dotnet install process to install from
Jammy feeds for users on Ubuntu 22.04 and greater while continuing to use
the packages.microsoft.com feed for Debian and older Ubuntu versions.
Update comments in the install from source script so that:

1. Sentences begin with capital letters.
2. Sentences end with punctuation.
3. Comments are within 80 characters per line.
@ldennington ldennington changed the title install from source: use Jammy feeds for Ubuntu 22.04 install from source: use native feeds for Ubuntu 22.04 Aug 18, 2022
@ldennington ldennington merged commit c2c0cdc into git-ecosystem:main Aug 22, 2022
@ldennington ldennington mentioned this pull request Nov 3, 2022
ldennington added a commit that referenced this pull request Nov 3, 2022
Changes:

- Check for broken links in documentation (#700)
- Support macOS `arm64` installs via Homebrew (#798) 
- Validate installers before publishing (#813)
- Auto-generate maintainer away notification issues (#842)
- Install dotnet via Jammy feeds on Ubuntu 22.04 and greater (#839)
- Access Azure storage account using service principle credentials
(#851)
- Update documentation to use reference-style links (#680)
- Unify documentation line length (#862)
- Add generic username/password UI (#871)
- Bitbucket DC OAuth support (#607)
- Distribute GCM as a dotnet tool (#886)
- Drop `-core` suffix from entry executable #551 
- Speed up build graph (#924)
@ldennington ldennington deleted the ubuntu-jammy branch July 12, 2023 18:27
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.

Install from source script failing (Ubuntu 22.04)
3 participants