https://bugzilla.redhat.com/show_bug.cgi?id=1585565
--- Comment #6 from Mohammed Isam <[email protected]> --- (In reply to Robert-André Mauchin from comment #5) > - Please use your Fedora People space instead of Google to host your files, Done. New links are added below. > - Not needed: > %defattr(-,root,root,-) Removed. > - Install the doc but without COPYING: > %doc AUTHORS NEWS THANKS README ChangeLog > %license COPYING Fixed. > - %{_datadir}/metainfo → %{_metainfodir} (F28+ only) Fixed. > - Remove: > update-desktop-database &> /dev/null ||: Removed. > - This shouldn't be necessary as you already installing the man page with > 644 perm: > chmod -x %{buildroot}%{_mandir}/man1/shaman.1 Removed. > - Not needed: > Requires(post): info > Requires(post): desktop-file-utils > Requires(preun): info > Requires(post): desktop-file-utils Removed. -- 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 _______________________________________________ package-review mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/[email protected]/message/7AOCYMZGD2RDD54ARQXL5A5XGKGSIS3K/
