<div class="gmail_quote">On Fri, Oct 29, 2010 at 1:19 PM, Carl-Daniel Hailfinger <span dir="ltr"><<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="im">On 29.10.2010 21:53, David Hendricks wrote:<br>
> Before you commit, add the last range to the unaligned layout and maybe add<br>
> a note to the comments above. Maybe something like:<br>
</div>> [...]<br>
><br>
<br>
Done. Thanks for the torture test script.<br>
<br>
Acked-by: Carl-Daniel Hailfinger <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>><br>
and committed in r1218.<br></blockquote><div><br></div><div>Naturally, I already hit a snag.</div><div><br></div><div>When running the script, it seems that the added "$FLASHROM_PARAM" can get expanded incorrectly by the shell thus causing flashrom to barf when $FLASHROM_PARAM is empty. I suspect it quietly passes in "".</div>

<div><br></div><div>The attached patch is a simple s/\"$FLASHROM_PARAM\"/${FLASHROM_PARAM}/g. This seems to work.</div><div><br></div><div>My shell-fu is only moderate, perhaps someone with more expertise can verify that this is the -correct- way to do it.</div>

<div><br></div></div>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>