<div class="gmail_quote">On Fri, Oct 29, 2010 at 2:27 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 22:51, David Hendricks wrote:<br>
> When running the script, it seems that the added "$FLASHROM_PARAM" can get<br>
> expanded incorrectly by the shell thus causing flashrom to barf when<br>
> $FLASHROM_PARAM is empty. I suspect it quietly passes in "".<br>
><br>
> The attached patch is a simple s/\"$FLASHROM_PARAM\"/${FLASHROM_PARAM}/g.<br>
> This seems to work.<br>
><br>
> My shell-fu is only moderate, perhaps someone with more expertise can verify<br>
> that this is the -correct- way to do it.<br>
><br>
<br>
</div>If you reply to this with a Signed-off-by line, this is<br>
<div class="im">Acked-by: Carl-Daniel Hailfinger <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>><br></div></blockquote><div><br></div><div>D'oh! Thanks again for catching my silly mistakes. The fix-parameter-passing-shell-stupidity.patch is:</div>

<div>Signed-off-by: David Hendricks <<a href="mailto:dhendrix@google.com">dhendrix@google.com</a>></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="im">
<br>
</div><div><div></div><div class="h5">Regards,<br>
Carl-Daniel<br>
<br>
--<br>
<a href="http://www.hailfinger.org/" target="_blank">http://www.hailfinger.org/</a><br>
<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>