|
From: | Kewei Yu |
Subject: | Re: [Qemu-devel] [PULL 10/42] qtest: Fix the bug about disable vnc causes "make check" fail |
Date: | Sun, 26 Jan 2014 16:04:36 +0800 |
Am 18.01.2014 12:54, schrieb Kewei Yu:Thank you for your patient explains, I will. ^ ^
>
> 2014/1/17 Andreas Färber <address@hidden <mailto:address@hidden>>
>> > From: Kewei Yu <address@hidden <mailto:address@hidden>>
> Am 15.01.2014 11:22, schrieb Kevin Wolf:
> >> > Signed-off-by: Kewei Yu <address@hidden <mailto:address@hidden>>
> > When we disable vnc from "./configure", QEMU can't use the vnc option.
> > So qtest can't use the "vnc -none ", otherwise "make check" fails.
> > If QEMU uses "-display none", "-vnc none" is excrescent, So we
> just need to drop it.
> >
> > Reviewed-by: Paolo Bonzini <address@hidden
> <mailto:address@hidden>>
> > Signed-off-by: Kevin Wolf <address@hidden <mailto:address@hidden>>
>Sorry for the late answer. Since it was already in the maintainer's
> If the pull does get respun, 'Fix "make check" failing for
> --disable-vnc' would be better English. ;)
>
> Yes, It is more accurate. So.... I should re-submit it?
queue and just about the commit message, it's not necessary for you to
resubmit, my comment was addressed to Kevin. When a "PULL" has been
sent, only major reasons (like build breakages, missing/wrong Sob, etc.)
lead to a resubmission by the maintainer.
Apart from the grammar issues ("causes ... to fail" or "... failure",
"disabling ... causes"), please keep in mind that many commits fix bugs,
so it will be more useful to the reader of patch/commit to read
prominently what is changing (and why) to determine whether he/she is
affected. Hope that explains.
Regards,
Andreas
>
>
> Andreas
>
> --
> SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
> GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
>
> Kewei
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
[Prev in Thread] | Current Thread | [Next in Thread] |