Re: [PATCH v3 5/5] secret: Move virStorageSecretType to secret_util and rename

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 03, 2016 at 06:52:53 -0400, John Ferlan wrote:
> Move the enum into secret_util, rename it to be just virSecretLookupType.
> This includes quite a bit of collateral damage, but the goal is to remove
> the "virStorage*" and replace with the virSecretLookupType so that it's
> easier to to add new lookups that aren't necessarily storage pool related.
> 
> Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>
> ---
>  cfg.mk                              |  2 +-
>  src/conf/secret_conf.h              |  2 +-
>  src/libxl/libxl_conf.c              |  2 +-
>  src/qemu/qemu_domain.c              |  4 ++--
>  src/secret/secret_util.c            | 18 +++++++++---------
>  src/secret/secret_util.h            | 22 ++++++++++++++++++++--
>  src/storage/storage_backend_iscsi.c |  7 ++++---
>  src/storage/storage_backend_rbd.c   |  3 ++-
>  src/util/virstoragefile.c           | 33 +++++++++++++++++----------------
>  src/util/virstoragefile.h           | 17 +++--------------
>  tests/qemuargv2xmltest.c            |  4 ++--
>  11 files changed, 62 insertions(+), 52 deletions(-)
> 
> diff --git a/cfg.mk b/cfg.mk
> index a7b7266..0529a4e 100644
> --- a/cfg.mk
> +++ b/cfg.mk
> @@ -780,7 +780,7 @@ mid_dirs=access|conf|cpu|locking|logging|network|node_device|rpc|security|storag
>  sc_prohibit_cross_inclusion:
>  	@for dir in $(cross_dirs); do					\
>  	  case $$dir in							\
> -	    util/) safe="util";;					\
> +	    util/) safe="(util|secret)";;				\

I don't think this is a good idea. utils are used in many places that
don't link to the secret driver. While this is now used just to pull in
one data type, the check is meant to prevent problems with linking the
file if certain modules are disabled.

Peter

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]