[Spice-devel] [vdagent-win PATCH v6 2/5] Initial rewrite of image conversion code

Frediano Ziglio fziglio at redhat.com
Thu Jul 27 13:07:16 UTC 2017


> > On 27 Jul 2017, at 12:39, Christophe Fergeau < cfergeau at redhat.com > wrote:
> 

> > On Thu, Jul 27, 2017 at 11:54:08AM +0200, Victor Toso wrote:
> 

> > > On Mon, Jul 24, 2017 at 10:47:34AM -0400, Frediano Ziglio wrote:
> > 
> 

> > > > Not really familiar with GitLab merge requests but on GitHub they
> > > 
> > 
> 
> > > > remain open till closed so this would help with old ones.
> > > 
> > 
> 
> > > > The big change on moving to full PR is the way of commenting patches.
> > > 
> > 
> 
> > > > Unless PR are just used for tracking and are replicated to ML but
> > > 
> > 
> 
> > > > maybe is hard to keep them consistent. I think is possible to configure
> > > 
> > 
> 
> > > > PRs to send changes to ML. This would make the history persistent on
> > > 
> > 
> 
> > > > the ML.
> > > 
> > 
> 

> > > > Could we try for a period and see how does it go?
> > > 
> > 
> 

> > > +1, but how should we approach that? I think we need to move from
> > 
> 
> > > freedesktop to either gitlab or github first otherwise we can get
> > 
> 
> > > confused on what's going on.
> > 
> 

> > Just to be sure, you are both suggesting switching to pull requests, and
> 
> > doing the reviews in gitlab web UI?
> 

> Not me. Reviewing by mail is fine with me.

> Actually, Im concerned that if we switch to PR/MR, then some comments
> will end up being made only there, which is why I asked if there was a
> way to forward these comments to the mailing list. I believe this
> is possible from the group administration (its certainly possible
> for individual accounts on gitlab).

> > The initial problem is that some reviews are not done in a timely
> 
> > manner. Being able to easily get a list of pending reviews was brought
> 
> > forward as a potential solution to this problem, and apparently, you
> 
> > both think that switching from email based reviews to a web based review
> 
> > system would help in getting more reviews faster? (iow, it would make
> 
> > you more efficient at reviewing code, and you vastly prefer that over
> 
> > email).
> 

> No, that’s not correct (at least for me). The review itself can happen over
> mail,
> what I find inefficient is:

> a) to get the list of things to review, and
> b) to get a working version of the code after patching

> For a), email is good to notify you of recent reviews posted. But searching
> through e-mail for unreviewed stuff is not easy. Do you have a good trick for
> that?

> For b), Im not talking about git am. You may not realize that just
> figuring out which of our 17 repositories (not including personal
> ones) some particular patch applies to is not always obvious.

> Im pretty sure this is not so much of a problem when you have worked
> longer on the project, so I am bringing that up precisely because
> Im still fresh enough to remember this being an issue.

> > I'm not necessarily opposed to trying things out, I'm just trying to get
> 
> > a clear view of what we are expecting to get out of the change.
> 

> First things first, if we want to try MR/PR, we should switch to gitlab
> as the primary. I understand there was a desire to do that also because
> freedesktop user creation was slow.

> Would this be a valid first step?

> > Christophe
> 

There's already https://gitlab.com/spice/spice 

Frediano 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20170727/2badb171/attachment.html>


More information about the Spice-devel mailing list