[libvirt] [PATCH 06/11] util: use glib base64 encoding/decoding APIs

Andrea Bolognani abologna at redhat.com
Tue Oct 1 15:18:27 UTC 2019


On Tue, 2019-10-01 at 10:12 -0400, Laine Stump wrote:
> On 9/30/19 1:35 PM, Andrea Bolognani wrote:
> > Writing a check that compares the situation before a commit and
> > after it is not as easy as a point-in-time check.
> 
> Not all that bad though - just examine the lines that start with +

Fair enough. That wouldn't work outside of git, but I guess it would
be an acceptable compromise as all our CI jobs are executed from
inside a git checkout anyway.

> >   Instead of spending
> > a non-trival amount of time implementing something like that, I'd
> > rather spend my time dealing with the fallout of a one-time
> > conversion.
> 
> Without seeing concrete examples of what actually is "dealing with the 
> fallout" in both cases, I don't want to speculate too much on which 
> would cause more difficulty. I would say that even if we do the 
> conversion all at once, it should be in multiple patches so that if 
> there is some regression caused by the conversion, a git bisect will 
> lead to a multi-hundred line commit instead of a multi-thousand line commit.

Of course when I talk about "big-bang conversion" I mean done in a
single series, not a single patch!

-- 
Andrea Bolognani / Red Hat / Virtualization




More information about the libvir-list mailing list