
3 Feb
2022
3 Feb
'22
12:56 p.m.
Hi Michal, On 03/02/2022 18:09, Michal Prívozník wrote:
Apart from patch 2/3 this looks good. So here's what we are going to do. I've uploaded fixup commits onto my gitlab:
https://gitlab.com/MichalPrivoznik/libvirt/-/commits/virpidfile/
Please take a look and if you agree I will squash those fixup commits and merge.
Thank you for the detailed explanation in 2/3. Makes sense. Probably I need to dive a bit deeper into the code the next time. And yeah, I missed that redundant include initially and noticed it only after sending the patches. I agree on squashing the fixups and merging. -- Vasily Ulyanov <vulyanov@suse.de> Software Engineer, SUSE Labs Core