hob: only add new layer's to the UI

If a user added the same layer more than once the underlying bblayers.conf
file would not have multiple entries but the UI would show an entry for
the layer for each time it was added. This patch changes the Configurator
to return None values if the layer wasn't added to bblayers.conf and the
LayerEditor to only update the UI if non-None values are returned.

Fixes [YOCTO #1275]

(Bitbake rev: 9c59baa4da3c863ebc412a56ffd8dbd7a8ffeb60)

Signed-off-by: Joshua Lock <josh@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Joshua Lock 2011-07-22 11:32:27 -07:00 committed by Richard Purdie
parent 52e0be44e0
commit 9cf5190679
2 changed files with 6 additions and 3 deletions

View File

@ -139,7 +139,9 @@ class Configurator(gobject.GObject):
name = self._getLayerName(layerpath)
if name not in self.enabled_layers:
self.addLayer(name, layerpath)
return name, layerpath
return name, layerpath
else:
return None, None
def addLayer(self, name, path):
self.enabled_layers[name] = path

View File

@ -132,5 +132,6 @@ class LayerEditor(gtk.Dialog):
# FIXME: verify we've actually got a layer conf?
if path.endswith(".conf"):
name, layerpath = self.configurator.addLayerConf(path)
self.newly_added[name] = layerpath
self.layer_store.append([name, layerpath, True])
if name:
self.newly_added[name] = layerpath
self.layer_store.append([name, layerpath, True])