https://bugzilla.redhat.com/show_bug.cgi?id=1422555 Randy Barlow <randy@xxxxxxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #12 from Randy Barlow <randy@xxxxxxxxxxxxxxxxxxxxx> --- Thanks for fixing the gnu_regex bit. I agree that the header files aren't usually used for executables - I was just thrown off by the wording from fedora-review. I've read over the packaging guidelines and I do not believe the headers are required by them. Thus, the package is approved! One thing I noticed that I recommend fixing: There's a /usr/lib/.build-id folder being created by the RPM. I recommend removing it after the make install step. Apparently a few other RPMs do this too, so fedora-review sees it as a conflicting owned directory. -- 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