[PATCH v3 29/30] scripts: apibuild: add parsing variable's comments

Andrea Bolognani abologna at redhat.com
Fri Apr 22 17:05:41 UTC 2022


On Fri, Apr 22, 2022 at 06:54:25PM +0200, Victor Toso wrote:
> The itching to change the code all around is still here. I
> actually created another (local) branch to put some fixes and
> removing dead code... but I knew if I started reworking things it
> would consume more time to reach the end goal,
> libvirt-go-module's MR.

I completely agree that you should focus on getting the feature
working, even if it requires changing the script in less than ideal
ways. It's not like you'd be ruining a flawless piece of art ;)

> I'm finishing the last touches of v4 and I plan to send it
> without this addressed. Later, perhaps Today, I'll add a patch on
> top that can either be an +1 or squashed in to address this.

Please don't post patches that have known issues in them. It's very
unlikely that anyone will look at v4 until Monday anyway, so just
take the time you need to fix this and post v4 once you've done so.

-- 
Andrea Bolognani / Red Hat / Virtualization



More information about the libvir-list mailing list