<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 26, 2015 at 7:20 AM, Ed Maste <span dir="ltr"><<a href="mailto:emaste@freebsd.org" target="_blank">emaste@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
><br>
</span><span class="">> +    # Update scripts<br>
> +    getShellCommandStep(f, name='update scripts',<br>
> +                        command=['updateScripts' + scriptExt])<br>
<br>
</span>Should this one be [pathSep + 'updateScripts' + ...?</blockquote><div> </div><div>updateScripts should be included in the system path for initial checkout of the scripts, while the rest of scripts will be updated by updateScript step, and put into scripts directory.</div><div>Since it is not guaranteed scripts directory and this file exists before calling "update scritps" step. For example, in the initial run or if we need to clean the build directory after every build and etc.</div><div>The thought is put updateScrpts to system path as pre-condition of builder setup.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Also, is getLLDBScriptCommandsFactory intended to be a general purpose<br>
LLDB builder or Android-specific?<br>
</blockquote></div><div class="gmail_extra"><br></div>It is intended to be general purpose of LLDB builder or for "any builder".</div><div class="gmail_extra"><br></div><div class="gmail_extra">This factory only defines the prototype of build steps, but not the actual commands.</div><div class="gmail_extra">You could provide customized implementation to each step.</div><div class="gmail_extra">For example, you could write your own code for build steps and upload them to some repository, then have updateScripts to get your script from the repository.</div><div class="gmail_extra"><br></div><div class="gmail_extra">The major purpose of adding this factory is to avoid the rebooting of buildbot master for build command changes.</div><div class="gmail_extra">If we would like to change any commands, like switching from debug build to release build, adding extra test flags, we only need to make change to scripts and the change will be reflected in the next build.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Please let me know if you have further questions.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks,</div><div class="gmail_extra">Ying</div><div class="gmail_extra"><br></div><div class="gmail_extra"><br></div></div>