https://bugzilla.redhat.com/show_bug.cgi?id=1128384 Miroslav Suchý <msuchy@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|nobody@xxxxxxxxxxxxxxxxx |msuchy@xxxxxxxxxx Flags| |fedora-review? --- Comment #3 from Miroslav Suchý <msuchy@xxxxxxxxxx> --- This: for f in `ls sig*.{5,8}`; do gzip -9 ${f} done is not needed as rpm will gzip or xz those man pages anyway. Before adding user/group you should first check if it exist. See: https://fedoraproject.org/wiki/Packaging:UsersAndGroups?rd=Packaging/UsersAndGroups#Dynamic_allocation Otherwise it looks good to me. -- 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 package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review