On Thu, Jan 31, 2013 at 12:11 PM, Saul Wold <[email protected]> wrote:
> - rdepends = d.getVar('RDEPENDS_%s' % pkg, True)
> - rdepends = rdepends + ' ' + d.getVar('MLPREFIX') +
> "hicolor-icon-theme"
> - d.setVar('RDEPENDS_%s' % pkg, rdepends)
> + rdepends = ' ' + d.getVar('MLPREFIX') + "hicolor-icon-theme"
> + d.appendVar('RDEPENDS_%s' % pkg, rdepends)
>
The new version isn't using a getVar of the old rdepends at all, so it's
just replacing it every time. Is that really what you intended here?
Wouldn't a getVar() or '' be a better approach? Further, the '' +
d.getVar('MLPREFIX') is pointless if you aren't including the original
value, as it adds a separator to the beginning when there's nothing to
separate.
--
Christopher Larson
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
