https://bugzilla.redhat.com/show_bug.cgi?id=2120418 --- Comment #6 from Nicola Sella <nsella@xxxxxxxxxx> --- Hello, I asked other reviewers an opinion, thank you for your patience. > As for the /usr/bin/appstream-util and /usr/bin/desktop-file-validate buildrequires, I am still of the mind that it's best to spell out what executable we need as I've done here, but if you consider this a review blocker, I can of course change them. Not a blocker. Final note. Consider that using the explicit file path, you will have a longer build time because dnf will have to go through the whole database of paths and download additional data to do so. Just be aware that builds will last longer. If you are fine with it, just ping back, and I would pass the review. Thanks for the submission of this package and for your kind patience. -- 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=2120418 _______________________________________________ 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
