fedora-review denied: [Bug 226411] Merge Review: setserial
bugzilla at redhat.com
bugzilla at redhat.com
Tue Feb 6 20:32:59 UTC 2007
Bug 226411: Merge Review: setserial
Product: Fedora Extras
Version: devel
Component: Package Review
manuel wolfshant <wolfy at nobugconsulting.ro> has denied Tim Waugh
<twaugh at redhat.com>'s request for fedora-review:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226411
------- Additional Comments from manuel wolfshant <wolfy at nobugconsulting.ro>
Review for Release: 19.2.2
MUSTFIX:
there are a couple of problems with the spec and the patches
* the fhs patch has two errors
+mandir = @bindir@ <-- this should be @mandir@
+ $(STRIP) $(DESTDIR)$(bindir)/setserial -< should not be at all, leads
to
empty debuginfo
* the readme patch should include references to Fedora, not Red Hat
* the spec does not include the preferred BUILDROOT, does not honor SMP
flags,uses %makeinstall instead of make install
Warning from rpmlint: Summary ends with dot
I will attach the fixes for all of the above. Please use whatever you find
useful and once corrected I will do the formal full review.
More information about the Fedora-package-review
mailing list