Re: [PATCH V2] qemu: pre-create the dbus directory in qemuStateInitialize

[Pino Toscano <ptoscano@xxxxxxxxxx>]

On Wednesday, 22 July 2020 04:09:24 CEST Bihong Yu wrote:
> +    if (virDirCreate(cfg->dbusStateDir, 0770, cfg->user, cfg->group,
> +                     VIR_DIR_CREATE_ALLOW_EXIST) < 0) {
> +        virReportSystemError(errno, _("Failed to create dbus state dir %s"),
> +                             cfg->dbusStateDir);

Minor notes on the message:
- spell "D-Bus" correctly
- no need to abbreviate "directory"
- quote the path placeholder
so I suggest something like:
  "Failed to create the D-Bus state directory '%s'"

(Can't comment on the rest of the changes, sorry.)

-- 
Pino Toscano

Attachment: signature.asc
Description: This is a digitally signed message part.