[Spice-devel] Proposal: review branches (was Re: [vdagent-win PATCH v6 2/5] Initial rewrite of image conversion code)

[Christophe Fergeau]

On Wed, Jul 26, 2017 at 09:18:37AM +0200, Christophe de Dinechin wrote:
> Now, any objection to
> 
> 1. Recommending that we use git URLs in patches?

We can emphasize this, but this has been done in the past, and anyone is
free to do it anyway if they want ;) For what it's worth, for small
series (just a few patches), it might be a bit overkill.

> 2. Having a shared location for branches under review?

I've seen various options being discussed (push requests, patchew,
shared location, ...), but no clear consensus yet. Shared location was
the option with the fewer "free bonuses" (CI, ...), and has the drawback
that everyone needs to have access to that location (which is not going
to be true for new contributors).

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20170726/2e065b74/attachment.sig>