<div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Apr 23, 2014 at 4:01 AM, Von Welch <span dir="ltr"><<a href="mailto:von@vwelch.com" target="_blank">von@vwelch.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
It'd be nice if you could develop consistent style for $variable vs ${variable}, but for the most part, things look okay now.</blockquote>

<div class="gmail_extra"><br></div></div><div class="gmail_extra">I tend to use the brackets if I think it makes for easier-to-read code, but it's fairly subjective and I don't feel strongly. Looks like you prefer the first form unless you are using a modifier?</div>
</div></blockquote><div><br></div><div>More or less, or when next to something odd in a quoted string.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="">

<div class="gmail_extra"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">One thing I'd like to see in the harness is generating new gpg keys on the fly, but perhaps this is unnecessary.</blockquote>


<div class="gmail_extra"><br></div></div><div class="gmail_extra">I don't see the value off-hand</div></div></blockquote><div><br></div><div>Okay.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class=""><div class="gmail_extra"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


Another thing is I'd like to keep setup.sh fairly lean, and move any of those functions that aren't needed globally into the specific tests themselves. But for now, again, things seem fine.</blockquote><div><br></div>


</div><div>Every function is used in multiple tests, or at least I envision it will be once we have a full suite. In any case, I suggest waiting to do that optimization until you have a complete suite.</div></div></blockquote>
<div><br></div><div>Agreed. </div></div></div></div>