[PATCH] Fix a few typos in error messages

Jiri Denemark posted 1 patch 4 days, 4 hours ago
src/esx/esx_vi.c                 | 2 +-
src/network/bridge_driver_conf.c | 2 +-
src/qemu/qemu_postparse.c        | 2 +-
src/security/security_apparmor.c | 2 +-
src/vz/vz_sdk.c                  | 2 +-
5 files changed, 5 insertions(+), 5 deletions(-)
[PATCH] Fix a few typos in error messages
Posted by Jiri Denemark 4 days, 4 hours ago
Fixes: https://gitlab.com/libvirt/libvirt/-/issues/722
Reported-by: Göran Uddeborg <goeran@uddeborg.se>
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
---
 src/esx/esx_vi.c                 | 2 +-
 src/network/bridge_driver_conf.c | 2 +-
 src/qemu/qemu_postparse.c        | 2 +-
 src/security/security_apparmor.c | 2 +-
 src/vz/vz_sdk.c                  | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/esx/esx_vi.c b/src/esx/esx_vi.c
index 3ecd406e1d..e2589aa69a 100644
--- a/src/esx/esx_vi.c
+++ b/src/esx/esx_vi.c
@@ -370,7 +370,7 @@ esxVI_CURL_Download(esxVI_CURL *curl, const char *url, char **content,
          */
         if (*length > INT32_MAX / 2) {
             virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
-                           _("Download length it too large"));
+                           _("Download length is too large"));
             return -1;
         }
 
diff --git a/src/network/bridge_driver_conf.c b/src/network/bridge_driver_conf.c
index 4e40286ee3..738652390f 100644
--- a/src/network/bridge_driver_conf.c
+++ b/src/network/bridge_driver_conf.c
@@ -97,7 +97,7 @@ virNetworkLoadDriverConfig(virNetworkDriverConfig *cfg G_GNUC_UNUSED,
 
             if (fwBackends[0] < 0) {
                 virReportError(VIR_ERR_INTERNAL_ERROR,
-                               _("unrecognized 'firewall_backend = '%1$s' set in network driver config file %2$s"),
+                               _("unrecognized firewall_backend = '%1$s' set in network driver config file %2$s"),
                                fwBackendStr, filename);
                 return -1;
             }
diff --git a/src/qemu/qemu_postparse.c b/src/qemu/qemu_postparse.c
index f48f172c37..4fbd849ebf 100644
--- a/src/qemu/qemu_postparse.c
+++ b/src/qemu/qemu_postparse.c
@@ -1643,7 +1643,7 @@ qemuDomainDefVcpusPostParse(virDomainDef *def)
             /* they can be ordered only at the beginning */
             if (prevvcpu->hotpluggable == VIR_TRISTATE_BOOL_YES) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
-                               _("online non-hotpluggable vcpus need to be ordered prior to hotplugable vcpus"));
+                               _("online non-hotpluggable vcpus need to be ordered prior to hotpluggable vcpus"));
                 return -1;
             }
 
diff --git a/src/security/security_apparmor.c b/src/security/security_apparmor.c
index 07e95ec81d..c8e77c6cd2 100644
--- a/src/security/security_apparmor.c
+++ b/src/security/security_apparmor.c
@@ -374,7 +374,7 @@ AppArmorGenSecurityLabel(virSecurityManager *mgr G_GNUC_UNUSED,
 
     if (secdef->baselabel) {
         virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
-                       "%s", _("Cannot set a base label with AppArmour"));
+                       "%s", _("Cannot set a base label with AppArmor"));
         return -1;
     }
 
diff --git a/src/vz/vz_sdk.c b/src/vz/vz_sdk.c
index ce4586a3f5..b20d454fb8 100644
--- a/src/vz/vz_sdk.c
+++ b/src/vz/vz_sdk.c
@@ -2703,7 +2703,7 @@ static int prlsdkCheckVideoUnsupportedParams(virDomainDef *def)
             return 0;
         } else {
             virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
-                           _("Video adapters are not supported int containers."));
+                           _("Video adapters are not supported in containers."));
             return -1;
         }
     } else {
-- 
2.47.1
Re: [PATCH] Fix a few typos in error messages
Posted by Daniel P. Berrangé 4 days, 4 hours ago
On Tue, Dec 17, 2024 at 12:06:45PM +0100, Jiri Denemark wrote:
> Fixes: https://gitlab.com/libvirt/libvirt/-/issues/722
> Reported-by: Göran Uddeborg <goeran@uddeborg.se>
> Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
> ---
>  src/esx/esx_vi.c                 | 2 +-
>  src/network/bridge_driver_conf.c | 2 +-
>  src/qemu/qemu_postparse.c        | 2 +-
>  src/security/security_apparmor.c | 2 +-
>  src/vz/vz_sdk.c                  | 2 +-
>  5 files changed, 5 insertions(+), 5 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|