https://bugzilla.redhat.com/show_bug.cgi?id=848388 Milan Bartos <mbartos@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mbartos@xxxxxxxxxx --- Comment #1 from Milan Bartos <mbartos@xxxxxxxxxx> --- Hi, this is just an informational review. 1. liblognorm requires libestr, it should be in BuildRequires 2. %clean tag is not necessary 3. use INSTALL='install -p' Regards, Milan -- You are receiving this mail because: You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review