Erik,

Thanks for the update. It certainly was a strange problem to debug.


Earl



________________________________
 From: Erik de Castro Lopo <mle...@mega-nerd.com>
To: flac-dev@xiph.org 
Cc: Earl Chew <earl_c...@yahoo.com> 
Sent: Sunday, March 18, 2012 1:36:42 PM
Subject: Re: [flac-dev] PATCH: Add test for metaflac --add-replay-gain
 
Earl Chew wrote:

> I cannot reproduce the problem you see with either dash(1) or bash(1).

Just found something really werid. I opened the patch in a text editor
that doesn't support unicode and found this:

    +    awk -- '
    +    BEGIN {
    +            samplerate = '$1';

Everywhere there was supposed to be a space (hex 0a) there was a pair
(hex c2 0a). In my regular unicode aware editor, the hex c2 0a pair
just showed up as a space. I just checked and your original patch also
has these extra bytes. This may be something added by YahooMail.

With the cruft removed, the patch applied and the tests ran correctly.

I'll commit and push after a little testing.

Thanks.

Cheers,
Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/
_______________________________________________
flac-dev mailing list
flac-dev@xiph.org
http://lists.xiph.org/mailman/listinfo/flac-dev

Reply via email to