<div dir="ltr"><div class="gmail_extra"><br><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 id=":i0">-        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>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.<br>

-- <br>Christopher Larson</div></div>