[PATCH v3 07/30] docstring: enums: admin: Add 'Since version' metadata

Victor Toso victortoso at redhat.com
Thu Apr 21 18:18:16 UTC 2022


Hi,

On Thu, Apr 21, 2022 at 06:43:56PM +0200, Peter Krempa wrote:
> On Thu, Apr 21, 2022 at 15:05:06 +0200, Peter Krempa wrote:
> > On Wed, Apr 20, 2022 at 21:07:56 +0200, Victor Toso wrote:
> > > Either create or append to existing docstring, the version (git tag)
> > > that a given enum value was introduced in the format:
> > > 
> > >     Since: v1.2.3
> > > 
> > > Signed-off-by: Victor Toso <victortoso at redhat.com>
> > > ---
> > >  include/libvirt/libvirt-admin.h | 8 ++++----
> > >  1 file changed, 4 insertions(+), 4 deletions(-)
> > > 
> > > diff --git a/include/libvirt/libvirt-admin.h b/include/libvirt/libvirt-admin.h
> > > index e414f776e4..f65886f20d 100644
> > > --- a/include/libvirt/libvirt-admin.h
> > > +++ b/include/libvirt/libvirt-admin.h
> > 
> > This looks good on the first look, although there's a possibility that
> > for some reason these might not be usable e.g. by preventing inclusion
> > of the header. Unfortunately I don't remember the history and don't feel
> > like digging in such old code.
> 
> Okay, I dug through the history and we deliberately removed the header from
> the installation until the v2.0.0 release. So all virAdm* symbols must
> have v2.0.0 as the minimum version as none of them was usable ableit
> from internall use before that.

Ah, nice catch.

Minimum version is v2.0.0 for all virAdm*.

I'll fix it.

Thanks!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20220421/8c8221df/attachment.sig>


More information about the libvir-list mailing list