<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 31, 2013 at 12:50 PM, Chris Larson <span dir="ltr">&lt;<a href="mailto:clarson@kergoth.com" target="_blank">clarson@kergoth.com</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="im"><div class="gmail_quote">On Thu, Jan 31, 2013 at 12:11 PM, Saul Wold <span dir="ltr">&lt;<a href="mailto:sgw@linux.intel.com" target="_blank">sgw@linux.intel.com</a>&gt;</span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>-        rdepends = d.getVar(&#39;RDEPENDS_%s&#39; % pkg, True)<br>
-        rdepends = rdepends + &#39; &#39; + d.getVar(&#39;MLPREFIX&#39;) + &quot;hicolor-icon-theme&quot;<br>
-        d.setVar(&#39;RDEPENDS_%s&#39; % pkg, rdepends)<br>
+        rdepends = &#39; &#39; + d.getVar(&#39;MLPREFIX&#39;) + &quot;hicolor-icon-theme&quot;<br>
+        d.appendVar(&#39;RDEPENDS_%s&#39; % pkg, rdepends)</div></blockquote></div><br></div>The new version isn&#39;t using a getVar of the old rdepends at all, so it&#39;s just replacing it every time. Is that really what you intended here? Wouldn&#39;t a getVar() or &#39;&#39; be a better approach? Further, the &#39;&#39; + d.getVar(&#39;MLPREFIX&#39;) is pointless if you aren&#39;t including the original value, as it adds a separator to the beginning when there&#39;s nothing to separate.</blockquote>

</div><br>Erm, ignore this, I&#39;m apparently blind as a bat if I can&#39;t see &#39;appendVar&#39; :)<br>-- <br>Christopher Larson</div></div>