<div dir="ltr">Just now saw this email - sorry for the delay,<div><br></div><div>I used USER_CLASSES just because this is how it&#39;s enabled in the default local.conf ...</div><div><br></div><div>Alex</div></div><div class="gmail_extra">

<br><br><div class="gmail_quote">On Thu, Dec 19, 2013 at 9:21 PM, Richard Purdie <span dir="ltr">&lt;<a href="mailto:richard.purdie@linuxfoundation.org" target="_blank">richard.purdie@linuxfoundation.org</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, 2013-12-18 at 19:26 +0000, Alex DAMIAN wrote:<br>
&gt; From: Alexandru DAMIAN &lt;<a href="mailto:alexandru.damian@intel.com">alexandru.damian@intel.com</a>&gt;<br>
&gt;<br>
&gt; Toaster needs buildstats to be enabled in order to<br>
&gt; be able to collect task running stats. This patch<br>
&gt; adds buildstats to the running configuration in toaster mode.<br>
&gt;<br>
&gt; Signed-off-by: Alexandru DAMIAN &lt;<a href="mailto:alexandru.damian@intel.com">alexandru.damian@intel.com</a>&gt;<br>
&gt; ---<br>
&gt;  bin/toaster | 7 ++++++-<br>
&gt;  1 file changed, 6 insertions(+), 1 deletion(-)<br>
&gt;<br>
&gt; diff --git a/bin/toaster b/bin/toaster<br>
&gt; index 38e71ac..d6c3175 100755<br>
&gt; --- a/bin/toaster<br>
&gt; +++ b/bin/toaster<br>
&gt; @@ -61,10 +61,13 @@ function webserverStartAll()<br>
&gt;  }<br>
&gt;<br>
&gt;  # Helper functions to add a special configuration file<br>
&gt; +function initConfiguration()<br>
&gt; +{<br>
&gt; +        echo &quot;#Created by toaster start script&quot; &gt; ${BUILDDIR}/conf/$1<br>
&gt; +}<br>
&gt;<br>
&gt;  function addtoConfiguration()<br>
&gt;  {<br>
&gt; -        echo &quot;#Created by toaster start script&quot; &gt; ${BUILDDIR}/conf/$2<br>
&gt;          echo $1 &gt;&gt; ${BUILDDIR}/conf/$2<br>
&gt;  }<br>
&gt;<br>
&gt; @@ -167,7 +170,9 @@ fi<br>
&gt;  case $CMD in<br>
&gt;      start )<br>
&gt;          start_success=1<br>
&gt; +        initConfiguration toaster.conf<br>
&gt;          addtoConfiguration &quot;INHERIT+=\&quot;toaster buildhistory\&quot;&quot; toaster.conf<br>
&gt; +        addtoConfiguration &quot;USER_CLASSES+=\&quot;buildstats\&quot;&quot; toaster.conf<br>
&gt;          if ! webserverStartAll; then<br>
&gt;              echo &quot;Failed ${CMD}.&quot;<br>
&gt;              return 4<br>
<br>
</div></div>Why not just add this to the INHERIT? I&#39;m trying to figure out if<br>
USER_CLASSES is special in some way and when we should use one and when<br>
we should use the other...<br>
<br>
Cheers,<br>
<br>
Richard<br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Alex Damian<div>Yocto Project<br></div><div>SSG / OTC </div></div>
</div>