Re: [libvirt PATCH 2/2] include: Explicitly reserve values for overlapping flag types

2022-05-05 Thread Victor Toso
Hi, On Thu, May 05, 2022 at 08:57:04AM +0100, Daniel P. Berrangé wrote: > On Thu, May 05, 2022 at 09:48:54AM +0200, Victor Toso wrote: > > Hi, > > > > On Wed, May 04, 2022 at 07:02:48PM +0100, Daniel P. Berrangé wrote: > > > On Wed, May 04, 2022 at 07:30:08PM

Re: [libvirt PATCH 2/2] include: Explicitly reserve values for overlapping flag types

2022-05-05 Thread Victor Toso
Hi, On Wed, May 04, 2022 at 07:02:48PM +0100, Daniel P. Berrangé wrote: > On Wed, May 04, 2022 at 07:30:08PM +0200, Andrea Bolognani wrote: > > Due to hystorical reasons, it needs to be possible to pass values hystorical -> historical ? > > from the virTypedParameterFlags and

Re: [PATCH v4 00/19] Add 'version' to other exported types

2022-05-04 Thread Victor Toso
Hi, On Wed, Apr 27, 2022 at 02:17:01AM -0700, Andrea Bolognani wrote: > On Tue, Apr 26, 2022 at 06:26:26PM +0200, Victor Toso wrote: > > On Tue, Apr 26, 2022 at 04:06:08PM +, Andrea Bolognani wrote: > > > Ideas for follow-up work: > > > > > > * impr

[PATCH 2/2] apibuild: remove 'v' from pattern matching

2022-05-04 Thread Victor Toso
This makes madatory to *not* add 'v' to version number. Signed-off-by: Victor Toso --- scripts/apibuild.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index ddc611ec58..2a343208c5 100755 --- a/scripts/apibuild.py +++ b

Re: [PATCH v4 00/19] Add 'version' to other exported types

2022-04-26 Thread Victor Toso
Hi, On Tue, Apr 26, 2022 at 04:06:08PM +, Andrea Bolognani wrote: > On Fri, Apr 22, 2022 at 09:23:25PM +0200, Victor Toso wrote: > > - For exported version of functions, the syms file are the source of > > truth apart from 4 functions (Peter) > > https://listman.redha

[PATCH v4 10/19] docstring: typedef: admin:: Add 'Since version' metadata

2022-04-22 Thread Victor Toso
Either create or append to existing docstring, the version (git tag) that a given typedef was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- include/libvirt/libvirt-admin.h | 17 + 1 file changed, 17 insertions(+) diff --git a/include/libvirt

[PATCH v4 19/19] scripts: apibuild: add parsing variable's comments

2022-04-22 Thread Victor Toso
to highlight this. Signed-off-by: Victor Toso --- scripts/apibuild.py | 31 --- 1 file changed, 24 insertions(+), 7 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index 40ee0a3c63..c2920d811c 100755 --- a/scripts/apibuild.py +++ b/scripts/apibuild.py

[PATCH v4 18/19] scripts: apibuild: factor out comment cleaning

2022-04-22 Thread Victor Toso
So we can use for comments that are being hold in helper variables. Signed-off-by: Victor Toso --- scripts/apibuild.py | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index b86d20e364..40ee0a3c63 100755 --- a/scripts

[PATCH v4 16/19] scripts: apibuild: parse 'Since' for macros

2022-04-22 Thread Victor Toso
as there is a proper field for it in the XML. Signed-off-by: Victor Toso --- scripts/apibuild.py | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index 24ca98bd1c..b77eea0624 100755 --- a/scripts/apibuild.py +++ b/scripts/apibuild.py

[PATCH v4 17/19] scripts: apibuild: parse 'Since' for functions

2022-04-22 Thread Victor Toso
for keeping both. For now, I've added a warning in case there is a mismatch between the version provided by .syms and docstring. Signed-off-by: Victor Toso --- scripts/apibuild.py | 70 - 1 file changed, 57 insertions(+), 13 deletions(-) diff --git

[PATCH v4 14/19] scripts: apibuild: fix parsing block comments from typedef enum

2022-04-22 Thread Victor Toso
Removed the TODO as we can rely to the serialize_typedef() the job to report missing comments. Signed-off-by: Victor Toso --- scripts/apibuild.py | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index 19e897b1ba..ddc2fff7af

[PATCH v4 15/19] scripts: apibuild: parse 'Since' for typedefs

2022-04-22 Thread Victor Toso
as there is a proper field for it in the XML. Signed-off-by: Victor Toso --- scripts/apibuild.py | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index ddc2fff7af..24ca98bd1c 100755 --- a/scripts/apibuild.py +++ b

[PATCH v4 12/19] docstring: function: admin: Add 'Since version' metadata

2022-04-22 Thread Victor Toso
Either create or append to existing docstring, the version (git tag) that a given function was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- include/libvirt/libvirt-admin.h | 2 ++ src/admin/libvirt-admin.c | 62 + 2 files

[PATCH v4 13/19] scripts: apibuild: parse 'Since' version for enums

2022-04-22 Thread Victor Toso
This patch adds 'version' parameter to the generated XML API for enums. It'll require, for new additions, to add a comment with the version that the enum value was added. Note that the Since tag is removed from the comment as there is a proper field for it in the XML. Signed-off-by: Victor Toso

[PATCH v4 09/19] docstring: macros: admin: Add 'Since version' metadata

2022-04-22 Thread Victor Toso
Either create or append to existing docstring, the version (git tag) that a given macro was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- include/libvirt/libvirt-admin.h | 40 + 1 file changed, 40 insertions(+) diff --git a/include

[PATCH v4 08/19] docstring: enums: admin: Add 'Since version' metadata

2022-04-22 Thread Victor Toso
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 --- include/libvirt/libvirt-admin.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/include

[PATCH v4 06/19] docstring: avoid sc_prohibit_nonreentrant in a comment

2022-04-22 Thread Victor Toso
Follow up patch will add Since metadata to enum values of virStorageVolWipeAlgorithm. This patch is necessary to avoid breaking the build with syntax-check. Signed-off-by: Victor Toso --- include/libvirt/libvirt-storage.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH v4 03/19] docstring: typedef: fix version of virDomainMemoryModFlags

2022-04-22 Thread Victor Toso
This was a mistake on my side in the v3 iteration of the series. I've introduced the wrong version number in v3 10/30 and fixed later in v3 21/30. Apologies. Fixes: 034432e47b Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v4 05/19] docstring: remove comments between enum values

2022-04-22 Thread Victor Toso
scripts/apibuild.py does not handle well enum's with comments between values. The outcome depends on whether the enum value has its own docstring or not and if there is spaces between the enum values or not. This commit removes some generic comments that I have spotted. Signed-off-by: Victor

[PATCH v4 00/19] Add 'version' to other exported types

2022-04-22 Thread Victor Toso
st/2022-April/230352.html - Removed an extra line in the block of comments (Andrea) https://listman.redhat.com/archives/libvir-list/2022-April/230291.html The CI run for this series: https://gitlab.com/victortoso/libvirt/-/pipelines/522757286 Have a great weekend, Victor Victor Toso (19):

[PATCH v4 02/19] docstring: typedef: fix version of virDomainDeviceModifyFlags

2022-04-22 Thread Victor Toso
This was a mistake on my side in the v3 iteration of the series. I've introduced the wrong version number in v3 10/30 and fixed later in v3 20/30. Apologies. Fixes: 034432e47b Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v4 04/19] docstring: typedef: fix version of virDomainVcpuFlags

2022-04-22 Thread Victor Toso
This was a mistake on my side in the v3 iteration of the series. I've introduced the wrong version number in v3 10/30 and fixed later in v3 22/30. Apologies. Fixes: 034432e47b Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v4 01/19] docstring: typedef: fix version of virConnectListAllStoragePoolsFlags

2022-04-22 Thread Victor Toso
This was a mistake on my side in the v3 iteration of the series. I've introduced the wrong version number in v3 10/30 and fixed later in v3 19/30. Apologies. Fixes: 034432e47b Signed-off-by: Victor Toso --- include/libvirt/libvirt-storage.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion

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

2022-04-22 Thread Victor Toso
On Fri, Apr 22, 2022 at 10:05:41AM -0700, Andrea Bolognani wrote: > 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 de

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

2022-04-22 Thread Victor Toso
Hi, On Fri, Apr 22, 2022 at 09:36:30AM -0700, Andrea Bolognani wrote: > On Wed, Apr 20, 2022 at 09:08:18PM +0200, Victor Toso wrote: > > def serialize_variable(self, output, name): > > id = self.idx.variables[name] > > -if id.info is not None: > >

Re: [PATCH v3 00/30] Add 'version' to other exported types

2022-04-22 Thread Victor Toso
On Fri, Apr 22, 2022 at 07:21:11AM -0700, Andrea Bolognani wrote: > On Fri, Apr 22, 2022 at 10:20:11AM +0200, Peter Krempa wrote: > > Based on discussions on individual patches I've applied the feedback I > > had including Andrea's request to remove the extra empty line on the > > following

Re: [PATCH v3 18/30] docstring: Fix generated documentation of virStorageVolInfoFlags

2022-04-22 Thread Victor Toso
Hi, On Fri, Apr 22, 2022 at 02:13:50AM -0700, Andrea Bolognani wrote: > On Thu, Apr 21, 2022 at 08:24:39PM +0200, Victor Toso wrote: > > On Thu, Apr 21, 2022 at 03:48:51PM +0200, Peter Krempa wrote: > > > Actually, this paragraph is inaccurate since the commits were rename

Re: [PATCH v3 13/30] docstring: function: libvirt: Add 'Since version' metadata

2022-04-22 Thread Victor Toso
Hi, On Fri, Apr 22, 2022 at 01:29:07AM -0700, Andrea Bolognani wrote: > On Thu, Apr 21, 2022 at 09:12:10PM +0200, Victor Toso wrote: > > On Thu, Apr 21, 2022 at 11:45:05AM -0700, Andrea Bolognani wrote: > > > On Thu, Apr 21, 2022 at 08:34:10PM +0200, Victor Toso wrote: >

Re: [PATCH v3 00/30] Add 'version' to other exported types

2022-04-22 Thread Victor Toso
Hi, On Fri, Apr 22, 2022 at 10:20:11AM +0200, Peter Krempa wrote: > On Wed, Apr 20, 2022 at 21:07:49 +0200, Victor Toso wrote: > > Hi, > > > > Based on discussions on individual patches I've applied the feedback I > had including Andrea's request to remove the extra empty

Re: [PATCH v3 08/30] docstring: macros: libvirt: Add 'Since version' metadata

2022-04-21 Thread Victor Toso
Hi, On Thu, Apr 21, 2022 at 03:13:05PM +0200, Peter Krempa wrote: > On Wed, Apr 20, 2022 at 21:07:57 +0200, Victor Toso wrote: > > Either create or append to existing docstring, the version (git tag) > > that a given macro was introduced in the format: > > > > Si

Re: [PATCH v3 13/30] docstring: function: libvirt: Add 'Since version' metadata

2022-04-21 Thread Victor Toso
Hi, On Thu, Apr 21, 2022 at 11:45:05AM -0700, Andrea Bolognani wrote: > On Thu, Apr 21, 2022 at 08:34:10PM +0200, Victor Toso wrote: > > Not sure I understood what is preferable. For all functions and > > typdefs and macros, it should be: > > > > Line > > 1

Re: [PATCH v3 20/30] docstring: Fix and append Since to virDomainDeviceModifyFlags

2022-04-21 Thread Victor Toso
Hi, On Thu, Apr 21, 2022 at 03:52:58PM +0200, Peter Krempa wrote: > On Thu, Apr 21, 2022 at 15:51:25 +0200, Peter Krempa wrote: > > On Wed, Apr 20, 2022 at 21:08:09 +0200, Victor Toso wrote: > > > This commit is similar to "docs: Fix

Re: [PATCH v3 13/30] docstring: function: libvirt: Add 'Since version' metadata

2022-04-21 Thread Victor Toso
Hi, On Thu, Apr 21, 2022 at 06:21:06AM -0700, Andrea Bolognani wrote: > On Wed, Apr 20, 2022 at 09:08:02PM +0200, Victor Toso wrote: > > @@ -4815,6 +4821,9 @@ typedef void > > (*virConnectDomainEventGenericCallback)(virConnectPtr conn, > > * > > * The cal

Re: [PATCH v3 18/30] docstring: Fix generated documentation of virStorageVolInfoFlags

2022-04-21 Thread Victor Toso
Hi, On Thu, Apr 21, 2022 at 03:48:51PM +0200, Peter Krempa wrote: > On Thu, Apr 21, 2022 at 15:43:55 +0200, Peter Krempa wrote: > > On Wed, Apr 20, 2022 at 21:08:07 +0200, Victor Toso wrote: > > > This commit is similar to "docs: Fix

Re: [PATCH v3 09/30] docstring: macros: admin: Add 'Since version' metadata

2022-04-21 Thread Victor Toso
Hi, On Thu, Apr 21, 2022 at 03:14:05PM +0200, Peter Krempa wrote: > On Wed, Apr 20, 2022 at 21:07:58 +0200, Victor Toso wrote: > > Either create or append to existing docstring, the version (git tag) > > that a given macro was introduced in the format: > > > > Si

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

2022-04-21 Thread Victor Toso
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) > > &

Re: [PATCH v3 02/30] syntax-check: prohibit_virXXXFree: skip strings

2022-04-21 Thread Victor Toso
Hi, On Thu, Apr 21, 2022 at 02:02:25PM +0200, Peter Krempa wrote: > On Wed, Apr 20, 2022 at 21:07:51 +0200, Victor Toso wrote: > > We should allow strings (not the functions) in the format of > > "virDomainFree" and friends. > > > > This fixes an error in la

Re: [PATCH v3 01/30] syntax-check: sc_prohibit_nonreentrant: skip comments

2022-04-21 Thread Victor Toso
On Thu, Apr 21, 2022 at 02:11:03PM +0200, Peter Krempa wrote: > On Wed, Apr 20, 2022 at 21:07:50 +0200, Victor Toso wrote: > > It doesn't need to fail the test if a reentrant function is found in > > a comment. > > That is true, but your patch doesn't do exactly that. >

Re: [PATCH v3 27/30] scripts: apibuild: parse 'Since' for functions

2022-04-21 Thread Victor Toso
On Thu, Apr 21, 2022 at 01:56:19PM +0200, Peter Krempa wrote: > On Wed, Apr 20, 2022 at 21:08:16 +0200, Victor Toso wrote: > > This patch adds 'version' parameter to generated XML API for functions > > and functypes. > > > > The 'version' metadata has been added with

Re: [PATCH v3 00/30] Add 'version' to other exported types

2022-04-20 Thread Victor Toso
Hi, On Wed, Apr 20, 2022 at 09:07:49PM +0200, Victor Toso wrote: > Gitlab run of this series (fails with codestyle of do-not-commit > script). Actually, I fixed before sending. > https://gitlab.com/victortoso/libvirt/-/pipelines/520901763 Cheers, Victor signature.asc Descrip

[PATCH v3 24/30] scripts: apibuild: fix parsing block comments from typedef enum

2022-04-20 Thread Victor Toso
Removed the TODO as we can rely to the serialize_typedef() the job to report missing comments. Signed-off-by: Victor Toso --- scripts/apibuild.py | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index 19e897b1ba..ddc2fff7af

[PATCH v3 10/30] docstring: typedef: libvirt:: Add 'Since version' metadata

2022-04-20 Thread Victor Toso
Either create or append to existing docstring, the version (git tag) that a given typedef was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain-checkpoint.h | 34 +- include/libvirt/libvirt-domain-snapshot.h | 40 +- include/libvirt

[PATCH v3 20/30] docstring: Fix and append Since to virDomainDeviceModifyFlags

2022-04-20 Thread Victor Toso
This commit is similar to "docs: Fix generated documentation of virConnectListAllNodeDeviceFlags", check it out for more info. Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain.h | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/inclu

[PATCH v3 30/30] do-not-commit: script to validate output

2022-04-20 Thread Victor Toso
./build/docs/ | # libvirt has 2101 symbols | # libvirt-qemu has 19 symbols | # libvirt-lxc has 4 symbols | # libvirt-admin has 63 symbols | # Total of 2187 symbols checked Signed-off-by: Victor Toso --- scripts/version-quest.py | 190 +++ 1 file changed

[PATCH v3 27/30] scripts: apibuild: parse 'Since' for functions

2022-04-20 Thread Victor Toso
for keeping both. For now, I've added a warning in case there is a mismatch between the version provided by .syms and docstring. Signed-off-by: Victor Toso --- scripts/apibuild.py | 126 +++- 1 file changed, 113 insertions(+), 13 deletions(-) diff --git

[PATCH v3 16/30] docstring: function: admin: Add 'Since version' metadata

2022-04-20 Thread Victor Toso
Either create or append to existing docstring, the version (git tag) that a given function was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- include/libvirt/libvirt-admin.h | 3 ++ src/admin/libvirt-admin.c | 93 + 2 files

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

2022-04-20 Thread Victor Toso
to highlight this. Signed-off-by: Victor Toso --- scripts/apibuild.py | 22 -- 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index 151eba37e6..9bdea648d4 100755 --- a/scripts/apibuild.py +++ b/scripts/apibuild.py

[PATCH v3 28/30] scripts: apibuild: factor out comment cleaning

2022-04-20 Thread Victor Toso
So we can use for comments that are being hold in helper variables. Signed-off-by: Victor Toso --- scripts/apibuild.py | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index ec10931151..151eba37e6 100755 --- a/scripts

[PATCH v3 26/30] scripts: apibuild: parse 'Since' for macros

2022-04-20 Thread Victor Toso
as there is a proper field for it in the XML. Signed-off-by: Victor Toso --- scripts/apibuild.py | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index 24ca98bd1c..b77eea0624 100755 --- a/scripts/apibuild.py +++ b/scripts/apibuild.py

[PATCH v3 25/30] scripts: apibuild: parse 'Since' for typedefs

2022-04-20 Thread Victor Toso
as there is a proper field for it in the XML. Signed-off-by: Victor Toso --- scripts/apibuild.py | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index ddc2fff7af..24ca98bd1c 100755 --- a/scripts/apibuild.py +++ b

[PATCH v3 23/30] scripts: apibuild: parse 'Since' version for enums

2022-04-20 Thread Victor Toso
This patch adds 'version' parameter to the generated XML API for enums. It'll require, for new additions, to add a comment with the version that the enum value was added. Note that the Since tag is removed from the comment as there is a proper field for it in the XML. Signed-off-by: Victor Toso

[PATCH v3 22/30] docstring: Fix and append Since to virDomainVcpuFlags

2022-04-20 Thread Victor Toso
This commit is similar to "docs: Fix generated documentation of virConnectListAllNodeDeviceFlags", check it out for more info. Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain.h | 19 +-- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/inclu

[PATCH v3 15/30] docstring: function: lxc: Add 'Since version' metadata

2022-04-20 Thread Victor Toso
Either create or append to existing docstring, the version (git tag) that a given function was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- src/libvirt-lxc.c | 12 1 file changed, 12 insertions(+) diff --git a/src/libvirt-lxc.c b/src/libvirt-lxc.c

[PATCH v3 21/30] docstring: Fix and append Since to virDomainMemoryModFlags

2022-04-20 Thread Victor Toso
This commit is similar to "docs: Fix generated documentation of virConnectListAllNodeDeviceFlags", check it out for more info. Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain.h | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/inclu

[PATCH v3 12/30] docstring: typedef: admin:: Add 'Since version' metadata

2022-04-20 Thread Victor Toso
Either create or append to existing docstring, the version (git tag) that a given typedef was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- include/libvirt/libvirt-admin.h | 24 1 file changed, 24 insertions(+) diff --git a/include/libvirt

[PATCH v3 17/30] docstring: libvirt-common: Add 'Since version' metadata

2022-04-20 Thread Victor Toso
Similarly to previous commits but with the targeting libvirt-common in a batch. Either create or append to existing docstring, the version (git tag) that a given typedef, enum value, macro or function was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- include

[PATCH v3 19/30] docstring: Fix and append Since to virConnectListAllStoragePoolsFlags

2022-04-20 Thread Victor Toso
This commit is similar to "docs: Fix generated documentation of virConnectListAllNodeDeviceFlags", check it out for more info. Signed-off-by: Victor Toso --- include/libvirt/libvirt-storage.h | 49 +++ 1 file changed, 24 insertions(+), 25 deletions(-)

[PATCH v3 18/30] docstring: Fix generated documentation of virStorageVolInfoFlags

2022-04-20 Thread Victor Toso
ical size in allocation'/>+} ... -] {+version='3.0.0'/>+} Signed-off-by: Victor Toso --- include/libvirt/libvirt-storage.h | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/include/libvirt/libvirt-storage.h b/include/libvirt/libvirt-storage.h index 434454f455..8

[PATCH v3 11/30] docstring: typedef: qemu:: Add 'Since version' metadata

2022-04-20 Thread Victor Toso
Either create or append to existing docstring, the version (git tag) that a given typedef was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- include/libvirt/libvirt-qemu.h | 18 ++ 1 file changed, 18 insertions(+) diff --git a/include/libvirt

[PATCH v3 14/30] docstring: function: qemu: Add 'Since version' metadata

2022-04-20 Thread Victor Toso
Either create or append to existing docstring, the version (git tag) that a given function was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- include/libvirt/libvirt-qemu.h | 3 +++ src/libvirt-qemu.c | 18 ++ 2 files changed, 21

[PATCH v3 09/30] docstring: macros: admin: Add 'Since version' metadata

2022-04-20 Thread Victor Toso
Either create or append to existing docstring, the version (git tag) that a given macro was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- include/libvirt/libvirt-admin.h | 60 + 1 file changed, 60 insertions(+) diff --git a/include

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

2022-04-20 Thread Victor Toso
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 --- include/libvirt/libvirt-admin.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/include

[PATCH v3 08/30] docstring: macros: libvirt: Add 'Since version' metadata

2022-04-20 Thread Victor Toso
Either create or append to existing docstring, the version (git tag) that a given macro was introduced in the format: Since: v1.2.3 Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain.h | 663 +- include/libvirt/libvirt-host.h| 123 ++ include

[PATCH v3 06/30] docstring: enums: qemu: Add 'Since version' metadata

2022-04-20 Thread Victor Toso
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 --- include/libvirt/libvirt-qemu.h | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git

[PATCH v3 04/30] docstring: variable: Move docstring from source to header file

2022-04-20 Thread Victor Toso
reasonable to move the docstring to the header file instead. Signed-off-by: Victor Toso --- include/libvirt/libvirt-host.h | 13 + src/libvirt.c | 11 +-- 2 files changed, 14 insertions(+), 10 deletions(-) diff --git a/include/libvirt/libvirt-host.h b/include

[PATCH v3 03/30] docstring: Fix generated documentation of virConnectListAllNodeDeviceFlags

2022-04-20 Thread Victor Toso
+} -]{+Target'/>+} -]{+info='System capability'/>+} -]{+interface'/>+} -]{+info='Device with VPD'/>+} -]{+info='Capable of vport'/>+} -]{+info='Inactive devices'/>+} Signed-off-by: Victor Toso --- include/libvirt/libvirt-nodedev.h | 5 ++--- 1 file changed, 2 insert

[PATCH v3 01/30] syntax-check: sc_prohibit_nonreentrant: skip comments

2022-04-20 Thread Victor Toso
) */ Signed-off-by: Victor Toso --- build-aux/syntax-check.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build-aux/syntax-check.mk b/build-aux/syntax-check.mk index a6e3ada300..eb403f3d3f 100644 --- a/build-aux/syntax-check.mk +++ b/build-aux/syntax-check.mk @@ -300,7 +300,7

[PATCH v3 02/30] syntax-check: prohibit_virXXXFree: skip strings

2022-04-20 Thread Victor Toso
virXXXFree] Error 1 Signed-off-by: Victor Toso --- build-aux/syntax-check.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build-aux/syntax-check.mk b/build-aux/syntax-check.mk index eb403f3d3f..07e0295995 100644 --- a/build-aux/syntax-check.mk +++ b/build-aux/syntax-check

[PATCH v3 00/30] Add 'version' to other exported types

2022-04-20 Thread Victor Toso
irt/libvirt-go-module/-/merge_requests/7 Gitlab run of this series (fails with codestyle of do-not-commit script). https://gitlab.com/victortoso/libvirt/-/pipelines/520901763 Have a wonderful day, Victor Victor Toso (30): syntax-check: sc_prohibit_nonreentrant: skip comments syntax-ch

Re: [PATCH v2 03/34] docs: generated: enums: libvirt: append 'Since version' metadata

2022-04-19 Thread Victor Toso
Hi, On Tue, Apr 19, 2022 at 06:37:19AM -0700, Andrea Bolognani wrote: > On Tue, Apr 19, 2022 at 03:30:12PM +0200, Peter Krempa wrote: > > On Tue, Apr 19, 2022 at 15:17:07 +0200, Victor Toso wrote: > > > On Tue, Apr 19, 2022 at 02:40:05PM +0200, Peter Krempa wrote: > >

Re: [PATCH v2 00/34] Add 'version' to other exported types

2022-04-19 Thread Victor Toso
Hi, On Tue, Apr 19, 2022 at 02:31:01PM +0200, Peter Krempa wrote: > On Thu, Apr 14, 2022 at 22:47:11 +0200, Victor Toso wrote: > > Hi, > > Couple of comments which I summarize on the top level rather than doing > specifics since they might need a reorg of the series: &g

Re: [PATCH v2 03/34] docs: generated: enums: libvirt: append 'Since version' metadata

2022-04-19 Thread Victor Toso
Hi, On Tue, Apr 19, 2022 at 02:40:05PM +0200, Peter Krempa wrote: > On Thu, Apr 14, 2022 at 22:47:14 +0200, Victor Toso wrote: > > Minor hand adjustments around syntax, not the metadata per se. > > > > Signed-off-by: Victor Toso > > --- > > include/libvirt/

Re: [PATCH v2 00/34] Add 'version' to other exported types

2022-04-14 Thread Victor Toso
Hi, On Thu, Apr 14, 2022 at 10:47:11PM +0200, Victor Toso wrote: > Hi, > > The goal of this patch series is to provide 'since' version to all > exported types. Forgot to add before/after diff... For 24h or so: https://paste.centos.org/view/be5b5c78 Cheers, > This is the n

[PATCH v2 30/34] scripts: apibuild: factor out comment cleaning

2022-04-14 Thread Victor Toso
So we can use for comments that are being hold in helper variables. Signed-off-by: Victor Toso --- scripts/apibuild.py | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index d16beaa4ae..97f448f6bf 100755 --- a/scripts

[PATCH v2 31/34] scripts: apibuild: add parsing variable's comments

2022-04-14 Thread Victor Toso
to highlight this. Signed-off-by: Victor Toso --- scripts/apibuild.py | 22 -- 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index 97f448f6bf..d1edfce3a6 100755 --- a/scripts/apibuild.py +++ b/scripts/apibuild.py

[PATCH v2 33/34] syms: libvirt: move virDomainSetBlockThreshold to 3.2.0

2022-04-14 Thread Victor Toso
irt-domain.h:int virDomainSetBlockThreshold(virDomainPtr domain, Signed-off-by: Victor Toso --- src/libvirt_public.syms | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/libvirt_public.syms b/src/libvirt_public.syms index f93692c427..84e608c7b0 100644 --- a/src/libvirt_publi

[PATCH v2 34/34] syntax-check: sc_prohibit_nonreentrant: skip comments

2022-04-14 Thread Victor Toso
) */ Signed-off-by: Victor Toso --- build-aux/syntax-check.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build-aux/syntax-check.mk b/build-aux/syntax-check.mk index a6e3ada300..eb403f3d3f 100644 --- a/build-aux/syntax-check.mk +++ b/build-aux/syntax-check.mk @@ -300,7 +300,7

[PATCH v2 29/34] scripts: apibuild: parse 'Since' for functions

2022-04-14 Thread Victor Toso
for keeping both. For now, I've added a warning in case there is a mismatch between the version provided by .syms and docstring. Signed-off-by: Victor Toso --- scripts/apibuild.py | 50 ++--- 1 file changed, 38 insertions(+), 12 deletions(-) diff --git

[PATCH v2 27/34] scripts: apibuild: parse 'Since' for typedefs

2022-04-14 Thread Victor Toso
as there is a proper field for it in the XML. Signed-off-by: Victor Toso --- scripts/apibuild.py | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index 5335670c1c..da70d67fa6 100755 --- a/scripts/apibuild.py +++ b

[PATCH v2 28/34] scripts: apibuild: parse 'Since' for macros

2022-04-14 Thread Victor Toso
as there is a proper field for it in the XML. Signed-off-by: Victor Toso --- scripts/apibuild.py | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index da70d67fa6..c4ff733550 100755 --- a/scripts/apibuild.py +++ b/scripts/apibuild.py

[PATCH v2 25/34] scripts: apibuild: parse 'Since' version for enums

2022-04-14 Thread Victor Toso
This patch adds 'version' parameter to the generated XML API for enums. It'll require, for new additions, to add a comment with the version that the enum value was added. Note that the Since tag is removed from the comment as there is a proper field for it in the XML. Signed-off-by: Victor Toso

[PATCH v2 32/34] syms: admin: Add sections to match when API was introduced

2022-04-14 Thread Victor Toso
Discover this mismatch with the docstring work. scripts/apibuild.py complained. Signed-off-by: Victor Toso --- src/admin/libvirt_admin_public.syms | 58 ++--- 1 file changed, 45 insertions(+), 13 deletions(-) diff --git a/src/admin/libvirt_admin_public.syms b/src/admin

[PATCH v2 24/34] docs: Fix and append Since to virDomainVcpuFlags

2022-04-14 Thread Victor Toso
Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain.h | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/include/libvirt/libvirt-domain.h b/include/libvirt/libvirt-domain.h index 015bc2faa4..451a398c42 100644 --- a/include/libvirt/libvirt-domain.h +++ b

[PATCH v2 26/34] scripts: apibuild: fix parsing block comments from typedef enum

2022-04-14 Thread Victor Toso
Removed the TODO as we can rely to the serialize_typedef() the job to report missing comments. Signed-off-by: Victor Toso --- scripts/apibuild.py | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/scripts/apibuild.py b/scripts/apibuild.py index 4e9f802bde..5335670c1c

[PATCH v2 23/34] docs: Fix and append Since to virDomainMemoryModFlags

2022-04-14 Thread Victor Toso
Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain.h | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/include/libvirt/libvirt-domain.h b/include/libvirt/libvirt-domain.h index 8f9a3775f2..015bc2faa4 100644 --- a/include/libvirt/libvirt-domain.h +++ b

[PATCH v2 12/34] docs: generated: functions: qemu: append 'Since version' metadata

2022-04-14 Thread Victor Toso
Signed-off-by: Victor Toso --- include/libvirt/libvirt-qemu.h | 3 +++ src/libvirt-qemu.c | 18 ++ 2 files changed, 21 insertions(+) diff --git a/include/libvirt/libvirt-qemu.h b/include/libvirt/libvirt-qemu.h index 838b6e9caf..b85e77f5b1 100644 --- a/include

[PATCH v2 22/34] docs: Fix and append Since to virDomainDeviceModifyFlags

2022-04-14 Thread Victor Toso
Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/include/libvirt/libvirt-domain.h b/include/libvirt/libvirt-domain.h index 20a476412f..8f9a3775f2 100644 --- a/include/libvirt/libvirt-domain.h +++ b

[PATCH v2 19/34] docs: manual: libvirt-common: add docstring and Since metadata

2022-04-14 Thread Victor Toso
This commit adds Since metadata to the libvirt-common's macros, enums and functions. Signed-off-by: Victor Toso --- include/libvirt/libvirt-common.h.in | 54 --- src/util/virtypedparam-public.c | 57 + 2 files changed, 98 insertions(+), 13

[PATCH v2 21/34] docs: Fix and append Since to virConnectListAllStoragePoolsFlags

2022-04-14 Thread Victor Toso
This commit is similar to "docs: Fix generated documentation of virConnectListAllNodeDeviceFlags", check it out for more info. Signed-off-by: Victor Toso --- include/libvirt/libvirt-storage.h | 47 +++ 1 file changed, 23 insertions(+), 24 deletions(-)

[PATCH v2 20/34] docs: Fix generated documentation of virStorageVolInfoFlags

2022-04-14 Thread Victor Toso
ical size in allocation'/>+} ... -] {+version='3.0.0'/>+} Signed-off-by: Victor Toso --- include/libvirt/libvirt-storage.h | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/include/libvirt/libvirt-storage.h b/include/libvirt/libvirt-storage.h index 9a0439f348..9

[PATCH v2 18/34] docs: manual: macros: add docstring and Since metadata

2022-04-14 Thread Victor Toso
This commit finishes the script work and adds docstring to types that are not documented. >From this commit onwards, all exported macros are versioned. Further commits will enforce that it stays like that with new additions. Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain.h |

[PATCH v2 17/34] docs: manual: enums: add docstring and Since metadata

2022-04-14 Thread Victor Toso
This commit finishes the script work and adds docstring to types that are not documented. >From this commit onwards, all exported typedefs are versioned. Further commits will enforce that it stays like that with new additions. Signed-off-by: Victor Toso --- include/libvirt/libvirt-admi

[PATCH v2 16/34] docs: manual: functions: add Since metadata

2022-04-14 Thread Victor Toso
This commit finishes the script work and adds docstring to types that are not documented. >From this commit onwards, all exported functions are versioned. Further commits will enforce that it stays like that with new additions. Signed-off-by: Victor Toso --- src/libvirt-domain.c |

[PATCH v2 15/34] docs: manual: typedef: add docstring and Since metadata

2022-04-14 Thread Victor Toso
This commit finishes the script work and adds docstring to types that are not documented. >From this commit onwards, all exported typedefs are versioned. Further commits will enforce that it stays like that with new additions. Signed-off-by: Victor Toso --- include/libvirt/libvirt-dom

[PATCH v2 14/34] docs: generated: functions: admin: append 'Since version' metadata

2022-04-14 Thread Victor Toso
Signed-off-by: Victor Toso --- include/libvirt/libvirt-admin.h | 3 ++ src/admin/libvirt-admin.c | 93 + 2 files changed, 96 insertions(+) diff --git a/include/libvirt/libvirt-admin.h b/include/libvirt/libvirt-admin.h index 2bfb45145d..f2cb528524 100644

[PATCH v2 13/34] docs: generated: functions: lxc: append 'Since version' metadata

2022-04-14 Thread Victor Toso
Signed-off-by: Victor Toso --- src/libvirt-lxc.c | 12 1 file changed, 12 insertions(+) diff --git a/src/libvirt-lxc.c b/src/libvirt-lxc.c index 2a271b74f0..14a2f2a6ec 100644 --- a/src/libvirt-lxc.c +++ b/src/libvirt-lxc.c @@ -59,6 +59,9 @@ VIR_LOG_INIT("libvir

[PATCH v2 05/34] docs: generated: enums: admin: append 'Since version' metadata

2022-04-14 Thread Victor Toso
Signed-off-by: Victor Toso --- include/libvirt/libvirt-admin.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/include/libvirt/libvirt-admin.h b/include/libvirt/libvirt-admin.h index e414f776e4..0f90ec8cdb 100644 --- a/include/libvirt/libvirt-admin.h +++ b/include

[PATCH v2 10/34] docs: generated: typedefs: admin: append 'Since version' metadata

2022-04-14 Thread Victor Toso
With style fixes. Signed-off-by: Victor Toso --- include/libvirt/libvirt-admin.h | 24 1 file changed, 24 insertions(+) diff --git a/include/libvirt/libvirt-admin.h b/include/libvirt/libvirt-admin.h index f54aeb0c1d..2bfb45145d 100644 --- a/include/libvirt/libvirt

[PATCH v2 09/34] docs: generated: typedefs: qemu: append 'Since version' metadata

2022-04-14 Thread Victor Toso
With style fixes. Signed-off-by: Victor Toso --- include/libvirt/libvirt-qemu.h | 12 1 file changed, 12 insertions(+) diff --git a/include/libvirt/libvirt-qemu.h b/include/libvirt/libvirt-qemu.h index 24bf67998b..838b6e9caf 100644 --- a/include/libvirt/libvirt-qemu.h +++ b

[PATCH v2 08/34] docs: generated: typedefs: libvirt: append 'Since version' metadata

2022-04-14 Thread Victor Toso
typedef fixes that are related to appending *Since* metadata. Signed-off-by: Victor Toso --- include/libvirt/libvirt-domain-checkpoint.h | 18 +- include/libvirt/libvirt-domain-snapshot.h | 24 +- include/libvirt/libvirt-domain.h| 583 +++- include/libvirt/libvirt

  1   2   >