https://bugzilla.redhat.com/show_bug.cgi?id=1466961 Dave Love <dave.love@xxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dave.love@xxxxxxxxxxxxxxxx --- Comment #3 from Dave Love <dave.love@xxxxxxxxxxxxxxxx> --- Some comments, as I intended to get round to re-submitting this: It needs Requires(preun): info The licence isn't installed, and there's an empty %doc. It should run make check (and BR perl(Digest::MD5) perl(Digest::SHA) perl(Data::Dumper)). I don't think the completion file should be %config. [The packaging guide could use instructions for completions.] Can I interest you in changes for completion installation on EPEL? -- 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 To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx