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

Replace DNF's deprecated 'dnf update' alias with 'dnf upgrade' #1300

Merged
merged 1 commit into from
Jun 22, 2023

Conversation

JWShaw
Copy link
Contributor

@JWShaw JWShaw commented Jun 20, 2023

Addresses #1298. According to the DNF manpages, update is a deprecated alias.

@ldennington
Copy link
Contributor

Thank you for your contribution! To better understand the scope of this change, can you provide the version of RHEL/Fedora in which the update command was initially deprecated?

@JWShaw
Copy link
Contributor Author

JWShaw commented Jun 20, 2023

I've not been successful in finding the exact version of RHEL/Fedora in which update was initially deprecated, but I found by perusing the dnf repository that dnf update was deprecated by a commit on November 12, 2012.

@ldennington
Copy link
Contributor

I've not been successful in finding the exact version of RHEL/Fedora in which update was initially deprecated, but I found by perusing the dnf repository that dnf update was deprecated by a commit on November 12, 2012.

Thanks for doing this digging! I think we're good since the command was deprecated over 10 years ago, and none of the versions of the applicable distros supported by .NET were released before that time. I've also done a test run of our validation workflow against this branch, and it's all green! I'll go ahead and get this merged.

@ldennington ldennington merged commit d045abe into git-ecosystem:main Jun 22, 2023
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.

2 participants