External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-12-26 15:45:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

qemu: Alter qemuGetCompressionProgram warning message

Browse Source 
Add a new parameter 'styleFormat' to be used when printing the
warning message so that it's "clearer" what style of compression
call caused the error. Add that style to both messages as a paremter.

Also a VIR_WARN error message doesn't need to be translated
 (e.g. inside _()), so remove the need for the translation.
This commit is contained in:
John Ferlan 2016-09-23 07:09:03 -04:00
parent 882e360dac
commit 4052ac2726
1 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/qemu/qemu_driver.c
View File

@ -3270,6 +3270,7 @@ qemuCompressProgramAvailable(virQEMUSaveFormat compress)
/* qemuGetCompressionProgram:
* @imageFormat: String representation from qemu.conf for the compression
* image format being used (dump, save, or snapshot).
* @styleFormat: String representing the style of format (dump, save, snapshot)
*
* Returns:
* virQEMUSaveFormat - Integer representation of the compression
@ -3280,7 +3281,8 @@ qemuCompressProgramAvailable(virQEMUSaveFormat compress)
* indicating none.
*/
static virQEMUSaveFormat
qemuGetCompressionProgram(const char *imageFormat)
qemuGetCompressionProgram(const char *imageFormat,
const char *styleFormat)
{
virQEMUSaveFormat ret;
@ -3297,12 +3299,13 @@ qemuGetCompressionProgram(const char *imageFormat)
error:
if (ret < 0)
VIR_WARN("%s", _("Invalid dump image format specified in "
"configuration file, using raw"));
VIR_WARN("Invalid %s image format specified in "
"configuration file, using raw",
styleFormat);
else
VIR_WARN("%s", _("Compression program for dump image format "
"in configuration file isn't available, "
"using raw"));
VIR_WARN("Compression program for %s image format in "
"configuration file isn't available, using raw",
styleFormat);
/* Use "raw" as the format if the specified format is not valid,
* or the compress program is not available. */
@ -3590,7 +3593,7 @@ doCoreDump(virQEMUDriverPtr driver,
/* We reuse "save" flag for "dump" here. Then, we can support the same
* format in "save" and "dump". */
compress = qemuGetCompressionProgram(cfg->dumpImageFormat);
compress = qemuGetCompressionProgram(cfg->dumpImageFormat, "dump");
/* Create an empty file with appropriate ownership. */
if (dump_flags & VIR_DUMP_BYPASS_CACHE) {