Re: [Qemu-devel] Re: [PATCH 05/17] monitor: Don't change VNC server when disabled

[Anthony Liguori <anthony@codemonkey.ws>]

Jan Kiszka wrote:
> Anthony Liguori wrote:
>   
>> Jan Kiszka wrote:
>>     
>>> Avoid a segfault when the user issues 'change vnc' without having vnc
>>> enabled on startup.
>>>
>>> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
>>>   
>>>       
>> It'd be easier on me if you split out these sort of fixes from the rest
>> of the series (that clearly has nothing to do with improving the monitor).
>>     
>
> You are free to apply this whenever you want, it has no dependency. I'll
> drop it from the series when it's no longer needed.
>
> [ Small fixes unfortunately tend to fall through the cracks, so I chose
> this path. ]
>   

 From a purely pragmatic sense, my patch series scripts verify that all 
patches in the series are present as some folks have issues with mailers 
or SMTP delay.  That means for a patch like this, I'd have to apply it 
by hand if I wanted to apply it in the absence of the rest of the 
series.  That means I'll simply wait until the rest of the series is 
ready to go.

BTW, I've now got something locally tracking patches on the list so 
hopefully we're past the days of stuff slipping through the cracks.  I 
have some old patches in my queue still that I've been waiting for other 
maintainers to ack/commit but rest assured, they haven't been forgotten :-)

Regards,

Anthony Liguori

> Jan
>
>