On Fri, Feb 22, 2019 at 04:32:27PM +0100, Erik Skultety wrote: > Unfortunately, in order to support the new flags, the last patch introduces an > API breakage as the convention we use for the bindings is to also enforce types > for flags. Yes, unfortunately this need to break ABI is fallout resulting from my decision to make the Go bindings do stricter enum validation that we have had at the C level. On balance I think that is still the right tradeoff to have stronger type checking as it catches real errors in app code. Due to the widespread use of "vendoring" in the Go community where apps fixate on specific git commit hashes of their dependandancies, the fallout is more limited and will mostly only impact devs at the time they decide to explicitly sync to newer git. So for all three Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx> Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
