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

Force RPM processing for libsolv #683

Closed

Conversation

@directhex
Copy link
Contributor

@directhex directhex commented Feb 14, 2019

libsolv will by default assume if it is built for Debian, Arch or Haiku systems, then the architecture name for platform-independent packages is not noarch (all on Debian, any on the others). This causes dnf to fail to run on those OSes, as it considers package redhat-rpm-config-115-1.el8.noarch does not have a compatible architecture etc.

Since libdnf is purely for RPM use, force it to always initialize libsolv in RPM mode, with noarch as the noarch name.

libsolv will by default assume if it is built for Debian, Arch or Haiku systems, then the architecture name for platform-independent packages is not `noarch` (`all` on Debian, `any` on the others). This causes dnf to fail to run on those OSes, as it considers `package redhat-rpm-config-115-1.el8.noarch does not have a compatible architecture` etc.

Since libdnf is purely for RPM use, force it to always initialize libsolv in RPM mode, with `noarch` as the noarch name.
@Conan-Kudo
Copy link
Member

@Conan-Kudo Conan-Kudo commented Feb 19, 2019

@j-mracek @jrohel @dmach Can we get this patch in? It's pretty trivial...

@j-mracek
Copy link
Contributor

@j-mracek j-mracek commented Feb 19, 2019

@rh-atomic-bot
Copy link

@rh-atomic-bot rh-atomic-bot commented Feb 19, 2019

📌 Commit 47fdfb5 has been approved by j-mracek

@rh-atomic-bot
Copy link

@rh-atomic-bot rh-atomic-bot commented Feb 19, 2019

Testing commit 47fdfb5 with merge 6eba696...

@rh-atomic-bot
Copy link

@rh-atomic-bot rh-atomic-bot commented Feb 19, 2019

☀️ Test successful - status-papr
Approved by: j-mracek
Pushing 6eba696 to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants