https://bugzilla.redhat.com/show_bug.cgi?id=2120418
--- Comment #6 from Nicola Sella <[email protected]> --- 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 -- [email protected] To unsubscribe send an email to [email protected] 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/[email protected] Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
