Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: soundtouch - Audio Processing library for changing 
Tempo, Pitch and Playback

------- Additional Comments From [EMAIL PROTECTED]  2006-08-01 00:57 EST -------
(In reply to comment #7)
> Needs work/Niggle
> rm -rf $RPM_BUILD_ROOT%{_prefix}/doc/%{name} - why is this going to /usr/doc
> instead of %{_docdir}?

This is what upstreams Makefile does, thats why its removed and then later added
under files with %doc.

> Question
> sed -i 's|\r||' README.html COPYING.TXT - is this to remove the incorrect
> termination of the files? I tend to use 's/\r//'
Same thing you can use any special char after the s in sed not only / but also |
or ! or :, etc. This is handy when your search or replace string contains / for
example when search replacing file / dir paths using / after the s is a bad 

Configure bugmail:
------- 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

Reply via email to