[Spice-devel] [PATCH spice-protocol] build-sys: simplify autogen
Uri Lublin
uril at redhat.com
Mon Dec 8 04:34:09 PST 2014
On 12/06/2014 04:04 AM, Fabiano Fidêncio wrote:
> On Fri, Dec 5, 2014 at 11:08 PM, Marc-André Lureau wrote:
>> On Fri, Dec 5, 2014 at 10:38 PM, Jonathon Jongsma wrote:
>>> For what it's worth, I basically agree with Christophe and Jeremy.
> I agree with Marc-André here.
I too agree with Marc-Andre about _trivial_ patches (such as typo fixing).
For any non-trivial patch, a review should be performed.
> Just to finish, IMHO, this patch, specifically, was not a trivial one.
The problem here is that "trivial" is subjective.
I view this specific patch as not trivial, as it practically replaces
the whole file.
I understand why Marc-Andre thinks it is trivial as the result of it is
a common autogen.sh.
Thanks,
Uri.
More information about the Spice-devel
mailing list