<html><body><div style="color:#000; background-color:#fff; font-family:times new roman, new york, times, serif;font-size:12pt"><div>Erik<br></div><div><br></div><div>&nbsp; &nbsp; Testing FLAC file with unknown metadata... OK<br>&nbsp; &nbsp; ./test_metaflac.sh: line 450: syntax error near unexpected token `then'<br>&nbsp; &nbsp; ./test_metaflac.sh: line 450: `&nbsp; then'<br></div><div><br></div><div>Do both /bin/bash and /bin/dash give exactly the same error message ?</div><div><br></div><div>Earl<br></div><div><br></div>  <div style="font-family: times new roman, new york, times, serif; font-size: 12pt;"> <div style="font-family: times new roman, new york, times, serif; font-size: 12pt;"> <div dir="ltr"> <font face="Arial" size="2"> <hr size="1">  <b><span style="font-weight:bold;">From:</span></b> Erik de Castro Lopo &lt;mle+la@mega-nerd.com&gt;<br> <b><span style="font-weight: bold;">To:</span></b> flac-dev@xiph.org <br><b><span style="font-weight:
 bold;">Cc:</span></b> Earl Chew &lt;earl_chew@yahoo.com&gt; <br> <b><span style="font-weight: bold;">Sent:</span></b> Sunday, March 11, 2012 3:16:11 AM<br> <b><span style="font-weight: bold;">Subject:</span></b> Re: [flac-dev] PATCH: Add test for metaflac --add-replay-gain<br> </font> </div> <br>Earl Chew wrote:<br><br>&gt; There is only one test to verify handling of harmonics of the filter table<br>&gt; sampling frequencies.<br><br>Earl,<br><br>I've had a couple of attempts to get this patch working, without luck.<br><br>With this patch applied, when i run the test_metaflac.sh script, I get:<br><br>&nbsp; &nbsp; Testing FLAC file with unknown metadata... OK<br>&nbsp; &nbsp; ./test_metaflac.sh: line 450: syntax error near unexpected token `then'<br>&nbsp; &nbsp; ./test_metaflac.sh: line 450: `&nbsp; then'<br><br>It's basically barfing at this line:<br><br>&nbsp; if [ -n "${ACTION##*:*}" ] ; then<br><br>Not sure where to go with this. You offerred Perl
 which I rejected mainly<br>because I read Perl :-). Are you comfortable with Python?<br><br>Erik<br>-- <br>----------------------------------------------------------------------<br>Erik de Castro Lopo<br><a href="http://www.mega-nerd.com/" target="_blank">http://www.mega-nerd.com/</a><br><br><br> </div> </div>  </div></body></html>