diff options
| author |   Joshua Lock <josh@linux.intel.com> | 2011-09-02 16:53:32 -0700 |
|---|---|---|
| committer |   Richard Purdie <richard.purdie@linuxfoundation.org> | 2011-09-05 20:16:07 +0100 |
| commit | 73b6ff4654d10baae59d83e8568d58d989e99dd7 (patch) | |
| tree | c068a06e035e4305c7d020655d30c72cc737f526 /lib | |
| parent | b0cce5b52a20c0dad5ec0c4053f437cae89b6137 (diff) | |
| download | bitbake-73b6ff4654d10baae59d83e8568d58d989e99dd7.tar.gz | |
hob: disable removal of packages
It's felt that the stability of package deselection is not sufficient for
the upcoming release and thus package removal should be disabled.
I'd actually like to see this patch, or its effects, reverted as soon as
the release bits have been frozen so that this issue can continue to be
worked on.
Signed-off-by: Joshua Lock <josh@linux.intel.com>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/bb/ui/hob.py | 42 |
1 files changed, 24 insertions, 18 deletions
diff --git a/lib/bb/ui/hob.py b/lib/bb/ui/hob.py index 3b0cacca4..76b03cdac 100644 --- a/lib/bb/ui/hob.py +++ b/lib/bb/ui/hob.py @@ -544,14 +544,17 @@ class MainWindow (gtk.Window): """ # Whether the item is currently included inc = self.model[opath][self.model.COL_INC] + # FIXME: due to inpredictability of the removal of packages we are + # temporarily disabling this feature # If the item is already included, mark it for removal then # the sweep_up() method finds affected items and marks them # appropriately - if inc: - self.model.mark(opath) - self.model.sweep_up() - # If the item isn't included, mark it for inclusion - else: + # if inc: + # self.model.mark(opath) + # self.model.sweep_up() + # # If the item isn't included, mark it for inclusion + # else: + if not inc: self.model.include_item(item_path=opath, binb="User Selected", image_contents=image) @@ -563,19 +566,22 @@ class MainWindow (gtk.Window): inc = model[path][self.model.COL_INC] # Warn user before removing included packages if inc: - pn = model[path][self.model.COL_NAME] - revdeps = self.model.find_reverse_depends(pn) - if len(revdeps): - lbl = "<b>Remove %s?</b>\n\nThis action cannot be undone and all packages which depend on this will be removed\nPackages which depend on %s include %s." % (pn, pn, ", ".join(revdeps).rstrip(",")) - else: - lbl = "<b>Remove %s?</b>\n\nThis action cannot be undone." % pn - dialog = CrumbsDialog(self, lbl, gtk.STOCK_DIALOG_WARNING) - dialog.add_button(gtk.STOCK_CANCEL, gtk.RESPONSE_CANCEL) - dialog.add_button("Remove", gtk.RESPONSE_OK) - response = dialog.run() - dialog.destroy() - if response == gtk.RESPONSE_CANCEL: - return + # FIXME: due to inpredictability of the removal of packages we are + # temporarily disabling this feature + return + # pn = model[path][self.model.COL_NAME] + # revdeps = self.model.find_reverse_depends(pn) + # if len(revdeps): + # lbl = "<b>Remove %s?</b>\n\nThis action cannot be undone and all packages which depend on this will be removed\nPackages which depend on %s include %s." % (pn, pn, ", ".join(revdeps).rstrip(",")) + # else: + # lbl = "<b>Remove %s?</b>\n\nThis action cannot be undone." % pn + # dialog = CrumbsDialog(self, lbl, gtk.STOCK_DIALOG_WARNING) + # dialog.add_button(gtk.STOCK_CANCEL, gtk.RESPONSE_CANCEL) + # dialog.add_button("Remove", gtk.RESPONSE_OK) + # response = dialog.run() + # dialog.destroy() + # if response == gtk.RESPONSE_CANCEL: + # return self.set_busy_cursor() # Convert path to path in original model |