On Thu, Jan 21, 2021 at 20:34:23 +0100, Pavel Hrdina wrote:
> Same as virStorageFileBackend, it doesn't belong into util directory.
Arguably, after you remove the file access, it kinda becomes just an
utility function.
On the other hand, it's use is strictly limited to the storage source
code, so it makes sense to stop linking it with the general util module.
>
> Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> ---
> po/POTFILES.in | 2 +-
> src/libvirt_private.syms | 10 +++++-----
> src/qemu/qemu_driver.c | 2 +-
> src/storage/storage_backend_gluster.c | 2 +-
> src/storage/storage_util.c | 2 +-
> src/storage_file/meson.build | 1 +
> .../storage_file_probe.c} | 6 +++---
> .../storage_file_probe.h} | 2 +-
> src/storage_file/storage_source.c | 2 +-
> src/util/meson.build | 1 -
> 10 files changed, 15 insertions(+), 15 deletions(-)
> rename src/{util/virstoragefileprobe.c => storage_file/storage_file_probe.c} (99%)
> rename src/{util/virstoragefileprobe.h => storage_file/storage_file_probe.h} (95%)
>
> diff --git a/po/POTFILES.in b/po/POTFILES.in
> index 4da22ede8f..eeff3f9f2f 100644
> --- a/po/POTFILES.in
> +++ b/po/POTFILES.in
> @@ -226,6 +226,7 @@
> @SRCDIR@src/storage_file/storage_file_backend.c
> @SRCDIR@src/storage_file/storage_file_fs.c
> @SRCDIR@src/storage_file/storage_file_gluster.c
> +@SRCDIR@src/storage_file/storage_file_probe.c
Hmm, I think we should add a prefix for the _fs and _gluster backend
drivers so that it doesn't get confused with the probing code.
Also same comment as previous patch regarding naming.
Reviewed-by: Peter Krempa <pkrempa@xxxxxxxxxx>
