Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: speex https://bugzilla.redhat.com/show_bug.cgi?id=226428 ------- Additional Comments From matthias@xxxxxxxxxxxx 2008-05-13 07:36 EST ------- I still see the "%{_defaultdocdir}/speex" line, which is probably including an empty directory or the same files as the %doc of the main package, given that the manual.pdf is removed in %install. And the macro in the %changelog still isn't escaped. These are the changes I'm taking about : --- speex.spec 25 Apr 2008 10:39:11 -0000 1.25 +++ speex.spec 13 May 2008 11:35:37 -0000 @@ -79,7 +79,6 @@ %files tools %defattr(-,root,root,-) -%{_defaultdocdir}/speex %doc doc/manual.pdf %{_bindir}/speexenc %{_bindir}/speexdec @@ -96,7 +95,7 @@ (CVE-2008-1686, #441239, https://trac.xiph.org/changeset/14701) * Mon Mar 31 2008 Marcela Maslanova <mmaslano@xxxxxxxxxx> - 1.2-0.6.beta3 -- 439284 add owner to %{_defaultdocdir}/speex +- 439284 add owner to %%{_defaultdocdir}/speex * Tue Feb 19 2008 Fedora Release Engineering <rel-eng@xxxxxxxxxxxxxxxxx> - 1.2-0.5.beta3 - Autorebuild for GCC 4.3 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review