[PATCH v1 2/4] scripts: apibuild: parse 'Since' for typedefs

Victor Toso victortoso at redhat.com
Tue Apr 5 15:36:51 UTC 2022


Hi,

On Tue, Apr 05, 2022 at 04:52:09PM +0200, Michal Prívozník wrote:
> On 4/5/22 13:43, Victor Toso wrote:
> > This patch adds 'version' parameter to the generated XML API for
> > typedefs
> > 
> > It'll require, for new additions, to add a comment with the version
> > that the typedef value was added. An example bellow of code diff and
> > the change in the generated XML.
> > 
> > Note that the Since tag is removed from the comment as there is a
> > proper field for it in the XML.
> > 
> > ```diff
> > --- a/include/libvirt/libvirt-host.h
> > +++ b/include/libvirt/libvirt-host.h
> > @@ -41,6 +41,8 @@ typedef struct _virConnect virConnect;
> >   *
> >   * a virConnectPtr is pointer to a virConnect private structure, this is the
> >   * type used to reference a connection to the Hypervisor in the API.
> > + *
> > + * Since 0.0.1
> >   */
> >  typedef virConnect *virConnectPtr;
> > ```
> > 
> > ```xml
> >     <typedef name='virConnectPtr'
> >              file='libvirt-host'
> >              type='virConnect *'
> >              version='0.0.1'>
> >       <info><![CDATA[a virConnectPtr is pointer to a virConnect private
> >  structure, this is the type used to reference a connection to the
> >  Hypervisor in the API.]]></info>
> >     </typedef>
> > ```
> > 
> 
> I'm not exactly sure why, but this diff in commit message causes my git
> to think this is a broken patch. Maybe it's trying to parse it as an
> actual patch? Funnily enough, there's no problem with 1/4.
> 
> Michal

When I send a v2, I'll make both examples above quoted, that is,
prefixed with: " > ". Not sure why I actually used ```diff and
```xml, gitlab does nothing with it x)

Cheers,
Victor
-------------- 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/20220405/681cf4bb/attachment.sig>


More information about the libvir-list mailing list