[Bug 519512] Review Request: cmusphinx3 - Large vocabulary speech recognition in C

[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.


https://bugzilla.redhat.com/show_bug.cgi?id=519512





--- Comment #6 from Martin Gieseking <martin.gieseking at uos.de>  2009-09-17 16:57:17 EDT ---
(In reply to comment #5)
> D'oh!  I saw 0775 and my brain translated it into 0755 before handing it to my
> logic unit.  It's all my brain's fault.  The logic unit worked fine. :-)

Hope you've been able to reset your brain successfully. At least I'm relieved
to hear I'm not the only one who got a buggy version. :)


Here are a couple of further things I noticed: 

- you can drop BR: perl, tcsh as they are not necessary

- BR: python-setuptools-devel should also be removed but the -python subpackage
needs BR: python-devel

- I suggest to preserve the timestamp of python-setuptools-devel by adding
touch -r, e.g.:

  pushd doc/s3-2_files/
  iconv -f WINDOWS-1252 -t UTF-8 master03_stylesheet.css > x.css
  touch -r master03_stylesheet.css x.css
  mv -f x.css master03_stylesheet.css
  popd

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.