[virt-tools-list] [virt-manager PATCH v2] [RFC] Make deleting storage files default with added prompt
Daniel P. Berrange
berrange at redhat.com
Tue Dec 4 15:31:53 UTC 2012
On Mon, Dec 03, 2012 at 05:12:59PM +0100, Martin Kletzander wrote:
> This patch changes the default checkbox-state of "Delete all
> associated storage" to be checked, but adds a prompt with a warning
> for users to be sure they notice this change and they know what they
> are doing (hopefully).
> ---
> I tried playing with this and it works, but unfortunately after the
> configuration gets changed few times. I haven't tried on a clean
> machine, so it maybe a problem of some old schemas being stuck in
> there. Feel free to correct my thoughts and patch.
>
> v2:
> - there is no configuration for what the default state should be,
> it's changed to 'checked' by default, but there is a prompt added
>
> src/virt-manager.schemas.in | 13 +++++++++++++
> src/virtManager/config.py | 8 +++++++-
> src/virtManager/delete.py | 12 ++++++++++--
> src/virtManager/preferences.py | 11 ++++++++++-
> src/vmm-preferences.ui | 30 ++++++++++++++++++++++++++++++
> 5 files changed, 70 insertions(+), 4 deletions(-)
ACK
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the virt-tools-list
mailing list