Re: [PATCH 2/2] virsh: Allow users to reedit rejected XML

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 22.05.2012 14:41, Eric Blake wrote:
> On 05/22/2012 03:49 AM, Michal Privoznik wrote:
>> On 18.05.2012 19:27, Eric Blake wrote:
>>> On 05/18/2012 06:48 AM, Michal Privoznik wrote:
>>>> If users *-edit but make a mistake in XML all changes are
>>>> permanently lost. However, if virsh is not running within
>>>> a script we can as user if he wants to re-edit the file
>>>
> 
>>>
>>> [1] ...you are blindly calling it from all platforms here.  You need to
>>> fix mingw compilation.
>>
>> Okay, on mingw I've make vshAskReedit function return always 0.
> 
> Makes sense - if we don't know how to ask the question, then it is the
> same as if we asked the question and the answer was successfully 'no'.
> 
> 
>>>   /* TRANSLATORS: For now, we aren't using LC_MESSAGES, and the user
>>>      choices really are limited to just 'y' and 'n'.  */
>>> vshPrintf(ctl, "\r%s", _("Failed. Try again..."));
>>
>> Well, if I take into account your last e-mail, how should this message
>> look like? I mean - how offer users 3 choices with intuitive names hence
>> shortcuts?
>>
>> Failed. [R]eedit/[S]tart over again/[Q]uit?
> 
> Eww. That does raise an interesting question.  Maybe it's better to make
> it a two part question:
> 
> 1. Simultaneous external edit detected.  Continue your edit [y/n]?
> 
> and if yes,
> 
> 2. Discard local edits by reloading external state [y/n]?
> 
> or something along those lines, where we can at a minimum reuse our
> yes/no parsing (and thus have only one place that needs to learn I18N in
> the future).
> 

I don't like being asked twice. I think users would prefer one question
with many answers, e.g. 'git add -p' produces:

	Stage this hunk [y,n,q,a,d,/,e,?]?

So maybe:

	Failed. Try again [y,n,f,?]?

with '?' printing out:
y - yes
n - no
f - force to continue with my change and drop changes made meanwhile
? - print this help

Michal

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]