Michal has pointed out to me that this needs to work on systems with a
libbe which does not have beVerifyBEName. I have updated the webrev,
http://cr.opensolaris.org/~padraig/ips-4698-v3/,
to disable editing of BE name if libbe does not contain beVerifyBEName.
Padraig
On 02/12/09 20:05, Joanmarie Diggs wrote:
On Thu, 2009-02-12 at 10:38 +0000, Padraig O'Briain wrote:
Joanie,
If I understand you correctly, the BE name field for the currently
active boot environment should not be editable.
Correct. Sorry about not being clearer.
I have respun the webrev to achieve this.
See http://cr.opensolaris.org/~padraig/ips-4698-v2/
Awesome. Just tried it. I think that's much better. Thank you!
Take care.
--Joanie
Padraig
On 02/11/09 20:10, Joanmarie Diggs wrote:
Hi Padraig.
With respect to this:
name_renderer = gtk.CellRendererText()
+ name_renderer.set_property("editable", True)
+ name_renderer.connect('edited', self.__be_name_edited, model)
column = gtk.TreeViewColumn(_("Boot Environment"),
name_renderer, text = BE_NAME)
column.set_cell_data_func(name_renderer,
self.__cell_data_function, None)
Instead of setting editable here, could you create a method similar to
__cell_data_default_function and set the editable property based on
whether or not the row is marked/currently active?
Take care.
--Joanie
On Wed, 2009-02-11 at 11:18 +0000, Padraig O'Briain wrote:
The webrev http://cr.opensolaris.org/~padraig/ips-4698-v1/ fixes
4698 Boot environment manager doesn't support rename of BE
The cells in the GtkTreeView containing the BE names are made editable.
When the user edits a name, we call beVerifyBEName from libbe to check
that the proposed name is valid and then call beRename from libbe to
change the name.
If beVerifyBEName fails we display an error message that the BE name is
invalid in an error dialog.
If beRename fails we display the error text corresponding to the return
code in an error dialog.
Padraig
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
