Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225638 --- Comment #5 from Matthias Saou <matthias@xxxxxxxxxxxx> 2009-12-08 12:26:33 EDT --- >From the spec, something got done at some point, all hope is not lost! (thanks, ajax ;-)) : * Tue Feb 10 2009 Adam Jackson <ajax@xxxxxxxxxx> - Merge review cleanups (not finished, #225638) Nevertheless, I've taken the liberty to at least partially continue the changes by including all of the obvious and unavoidable ones. Here is the detail : - Don't prefix summaries with "A" nor suffix them with a dot. - Move .so symlink to the devel sub-package (#203620). - Add highest known version to the cdparanoia-III obsoletes. - Remove incorrect buildroot removal from %%build. - Use acceptable %%clean section. - Provide cdparanoia-static in the devel sub-package since the *.a is there. - Use single-command scriplet syntax for /sbin/ldconfig calls. - Escape all macros in changelog. - Include license file since it is present with the sources. I'll rebase my other changes (cosmetic cleanups and install patch) against this new spec file. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review