[Bug 2120661] Review Request: dnf5 - package management library

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=2120661

Nicola Sella <nsella@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |needinfo?(msuchy@xxxxxxxxxx
                   |                            |)



--- Comment #15 from Nicola Sella <nsella@xxxxxxxxxx> ---
> FIX: This configures access permissions to the D-Bus service. I'm not familiar with D-Bus, but I believe the file should be marked with `%config(noreplace)'. It's clearly a configuration file which user may want to modify (e.g. add a permission to other users). And e.g. avahi package does the same.

Fixed

> FIX: Move dnf5daemon-server executable to /usr/sbin.

Fixed

Thank you for the feedback, I added the TODOs to future changes list while the
FIXes were addressed now to land in Fedora. I believe that everything discussed
as blocking has now been fixed and included.

SPEC:
https://download.copr.fedorainfracloud.org/results/rpmsoftwaremanagement/dnf5-src-fedora/fedora-rawhide-x86_64/04898744-dnf5/dnf5.spec
SRPM:
https://download.copr.fedorainfracloud.org/results/rpmsoftwaremanagement/dnf5-src-fedora/fedora-rawhide-x86_64/04898744-dnf5/dnf5-5.0.0-0~pre.fc38.src.rpm

msuchy what's your opinion?


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2120661
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux