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.

Sign up for GitHub

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

Jump to bottom

Scientfic Linux support #228

Closed
opoplawski opened this issue Nov 13, 2018 · 2 comments
Closed

Scientfic Linux support #228

opoplawski opened this issue Nov 13, 2018 · 2 comments

Comments

@opoplawski
Copy link
Contributor

Short description of the problem

Scientific Linux is a RHEL rebuild, but isn't treated that way.

There are currently two inconsistent listings of "RHEL" clones in the code:

mockbuild/package_manager.py:
if distribution in ['redhat', 'rhel', 'centos', 'ol']:
mockbuild/util.py:
epel = ('centos', 'deskos', 'ol', 'rhel')

Couple items:

  • We probably want to have a single place in the code to list these.
  • We should add 'scientific' to support Scientific Linux
  • I'm not sure 'redhat' is valid - my RHEL7.6 machine reports 'rhel'

Output of rpm -q mock

mock-1.4.13-1.el7.noarch
@xsuchy
Copy link
Member

xsuchy commented Nov 13, 2018

fixed in 1fed4c7

@xsuchy xsuchy closed this as completed Nov 13, 2018
@opoplawski
Copy link
Contributor Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xsuchy @opoplawski