rpm-software-management / dnf

Package manager based on libdnf and libsolv. Replaces YUM.
GNU General Public License v2.0
1.22k stars 405 forks source link

spec: Move libreport configuration to a separate package #2092

Closed ppisar closed 1 month ago

ppisar commented 2 months ago

dnf-data packaged a libreport configuration file for collecting DNF logs for ABRT. It has to depend on libreport-filesystem. That file and thus the dependency is useless if a user does not have ABRT/libreport installed. The script also requires dependencies, like systemd which were never declared.

This patch moves the configuration file into a separate dnf-libreport subpackage, declares all dependencies, and makes it conditionally required only if libreport is installed. This way the script is installed only when it is helpful. The new subpackage requires dnf-data to get uninstalled if a user uninstalls DNF.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2120960 Replaces: #1986

j-mracek commented 1 month ago

According to comment https://issues.redhat.com/browse/RHEL-40382?focusedId=24883517&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-24883517, it should be safe to drop the file.

ppisar commented 1 month ago

Obsoleted by issue #2101.