Re: [PATCH spice-protocol] build-sys: simplify autogen

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

 



On Fri, Dec 5, 2014 at 10:38 PM, Jonathon Jongsma <jjongsma@xxxxxxxxxx> wrote:
> For what it's worth, I basically agree with Christophe and Jeremy.
> (Although I think that describing it as "mandatory code review" is
> over-stating the case a bit -- there is nothing but peer pressure and
> polite requests preventing contributors from pushing unreviewed code).

Peer pressure is precisely what one should try to avoid. If you think
your change does not require second look, because it is trivial, then
why would you do it? Now you will have to bother others repeatedly for
the most basic thing that they might not even care about. I would
rather work differently as I explained before, and as we did until
now.

-- 
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]