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@rpmforge.net 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@redhat.com - 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.