[Bug 226428] Merge Review: speex

[bugzilla at redhat.com]

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 at rpmforge.net  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 at redhat.com> - 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 at fedoraproject.org> -
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.