https://bugzilla.redhat.com/show_bug.cgi?id=971836 Ville Skyttä <ville.skytta@xxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |ville.skytta@xxxxxx Assignee|nobody@xxxxxxxxxxxxxxxxx |ville.skytta@xxxxxx Flags| |fedora-review? --- Comment #2 from Ville Skyttä <ville.skytta@xxxxxx> --- The binutils and glibc-common build deps are redundant and should be removed. Renaming the docs seems unusual and quite pointless deviation from upstream to me, I'd just refer to their names like in %doc debian/... (non-blocker as far as the review goes). I suspect that the examples in parenthesis in %description are not quite accurate and are also subject to bitrot, I'd just remove them and while at it, remove some unnecessary bits off it and remove some extraneous hyphens, fix capitalization etc: ---- %{name} is a tool to check whether an already compiled ELF file was built using hardening flags. It checks, using readelf, for these hardening characteristics: * Position Independent Executable * Stack protected * Fortify source functions * Read-only relocations * Immediate binding -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=evRm1miur9&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review