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

Broken yum config-manager command on CentOS 7 #448

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

singhsegv
Copy link
Contributor

@singhsegv singhsegv commented Sep 24, 2024

- What I did
"yum config-manager" to "yum-config-manager"

- How I did it

- How to verify it

- Description for the changelog

The install script seems to be broken on CentOS 7. It results in "--add-repo not found" error. The actual issue seems from be about config-manager command not being there even after installation of yum-utils

No such command: config-manager. Please use /usr/bin/yum --help

Installing yum-utils should enable yum config-manager command in later versions of the OS (haven't tested) but it doesn't do it on the CentOS 7. This PR is to point this issue out and maybe someone can test on the newers OS versions if yum-config-manager is good to go or not. Thanks.

- A picture of a cute animal (not mandatory but encouraged)

@thaJeztah
Copy link
Member

Doh! Looks like I broke this in b982474, which was part of;

I should warn though that we no longer support CentOS 7 (which was the last distro using yum) as it reached EOL, so no updated packages will be published, and the portions of the script that use yum may eventually be removed.

So if you're still running CentOS 7 (or 8), make sure you update to a supported distro.

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@thaJeztah thaJeztah merged commit 6d51e2c into docker:master Sep 24, 2024
9 checks passed
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