Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
lib: Change 'program_name' macro to avoid conflict with gnulib.
The gnulib 'error' module uses 'program_name'.  On some platforms --
but not Linux / glibc -- it references it as:

  extern char *program_name;

This means when you compile libguestfs on non-glibc (eg. Mac OS X)
gnulib requires 'program_name' as an external string reference, which
we don't provide.

This change doesn't define this string reference for gnulib, but it
does change the name of the macro we use to avoid conflicts if we
eventually need to export 'program_name' as a string.

Thanks: Margaret Lewicka
  • Loading branch information
rwmjones committed Feb 7, 2015
1 parent 7ddf6bc commit 5cafeda
Show file tree
Hide file tree
Showing 41 changed files with 244 additions and 244 deletions.
14 changes: 7 additions & 7 deletions align/scan.c
Expand Up @@ -71,7 +71,7 @@ usage (int status)
{
if (status != EXIT_SUCCESS)
fprintf (stderr, _("Try `%s --help' for more information.\n"),
program_name);
guestfs___program_name);
else {
fprintf (stdout,
_("%s: check alignment of virtual machine partitions\n"
Expand All @@ -91,8 +91,8 @@ usage (int status)
" -V|--version Display version and exit\n"
" -x Trace libguestfs API calls\n"
"For more information, see the manpage %s(1).\n"),
program_name, program_name, program_name,
program_name);
guestfs___program_name, guestfs___program_name, guestfs___program_name,
guestfs___program_name);
}
exit (status);
}
Expand Down Expand Up @@ -152,7 +152,7 @@ main (int argc, char *argv[])
uuid = 1;
} else {
fprintf (stderr, _("%s: unknown long option: %s (%d)\n"),
program_name, long_options[option_index].name, option_index);
guestfs___program_name, long_options[option_index].name, option_index);
exit (EXIT_FAILURE);
}
break;
Expand All @@ -171,7 +171,7 @@ main (int argc, char *argv[])

case 'P':
if (sscanf (optarg, "%zu", &max_threads) != 1) {
fprintf (stderr, _("%s: -P option is not numeric\n"), program_name);
fprintf (stderr, _("%s: -P option is not numeric\n"), guestfs___program_name);
exit (EXIT_FAILURE);
}
break;
Expand Down Expand Up @@ -229,13 +229,13 @@ main (int argc, char *argv[])
exit (EXIT_FAILURE);
#else
fprintf (stderr, _("%s: compiled without support for libvirt.\n"),
program_name);
guestfs___program_name);
exit (EXIT_FAILURE);
#endif
} else { /* Single guest. */
if (uuid) {
fprintf (stderr, _("%s: --uuid option cannot be used with -a or -d\n"),
program_name);
guestfs___program_name);
exit (EXIT_FAILURE);
}

Expand Down
18 changes: 9 additions & 9 deletions builder/index-validate.c
Expand Up @@ -39,7 +39,7 @@ extern int do_parse (struct parse_context *context, FILE *in);
static void
usage (int exit_status)
{
printf ("%s index\n", program_name);
printf ("%s index\n", guestfs___program_name);
exit (exit_status);
}

Expand Down Expand Up @@ -83,7 +83,7 @@ main (int argc, char *argv[])
compat_1_24_1 = 1;
else {
fprintf (stderr, _("%s: unknown long option: %s (%d)\n"),
program_name, long_options[option_index].name, option_index);
guestfs___program_name, long_options[option_index].name, option_index);
exit (EXIT_FAILURE);
}
break;
Expand Down Expand Up @@ -115,20 +115,20 @@ main (int argc, char *argv[])

if (fclose (in) == EOF) {
fprintf (stderr, _("%s: %s: error closing input file: %m (ignored)\n"),
program_name, input);
guestfs___program_name, input);
}

if (ret != 0) {
parse_context_free (&context);
fprintf (stderr, _("%s: '%s' could not be validated, see errors above\n"),
program_name, input);
guestfs___program_name, input);
exit (EXIT_FAILURE);
}

if (compat_1_24_1 && context.seen_comments) {
parse_context_free (&context);
fprintf (stderr, _("%s: %s contains comments which will not work with virt-builder 1.24.1\n"),
program_name, input);
guestfs___program_name, input);
exit (EXIT_FAILURE);
}

Expand All @@ -141,7 +141,7 @@ main (int argc, char *argv[])
if (strchr (sections->name, '_')) {
parse_context_free (&context);
fprintf (stderr, _("%s: %s: section [%s] has invalid characters which will not work with virt-builder 1.24.0\n"),
program_name, input, sections->name);
guestfs___program_name, input, sections->name);
exit (EXIT_FAILURE);
}
}
Expand All @@ -152,7 +152,7 @@ main (int argc, char *argv[])
strchr (fields->key, ']')) {
parse_context_free (&context);
fprintf (stderr, _("%s: %s: section [%s], field '%s' has invalid characters which will not work with virt-builder 1.24.0\n"),
program_name, input, sections->name, fields->key);
guestfs___program_name, input, sections->name, fields->key);
exit (EXIT_FAILURE);
}
}
Expand All @@ -161,7 +161,7 @@ main (int argc, char *argv[])
strchr (fields->key, ',')) {
parse_context_free (&context);
fprintf (stderr, _("%s: %s: section [%s], field '%s' has invalid characters which will not work with virt-builder 1.24.1\n"),
program_name, input, sections->name, fields->key);
guestfs___program_name, input, sections->name, fields->key);
exit (EXIT_FAILURE);
}
}
Expand All @@ -172,7 +172,7 @@ main (int argc, char *argv[])
if (compat_1_24_0 && !seen_sig) {
parse_context_free (&context);
fprintf (stderr, _("%s: %s: section [%s] is missing a 'sig' field which will not work with virt-builder 1.24.0\n"),
program_name, input, sections->name);
guestfs___program_name, input, sections->name);
exit (EXIT_FAILURE);
}
}
Expand Down
8 changes: 4 additions & 4 deletions cat/cat.c
Expand Up @@ -51,7 +51,7 @@ usage (int status)
{
if (status != EXIT_SUCCESS)
fprintf (stderr, _("Try `%s --help' for more information.\n"),
program_name);
guestfs___program_name);
else {
fprintf (stdout,
_("%s: display files in a virtual machine\n"
Expand All @@ -73,8 +73,8 @@ usage (int status)
" -V|--version Display version and exit\n"
" -x Trace libguestfs API calls\n"
"For more information, see the manpage %s(1).\n"),
program_name, program_name, program_name,
program_name);
guestfs___program_name, guestfs___program_name, guestfs___program_name,
guestfs___program_name);
}
exit (status);
}
Expand Down Expand Up @@ -139,7 +139,7 @@ main (int argc, char *argv[])
OPTION_format;
} else {
fprintf (stderr, _("%s: unknown long option: %s (%d)\n"),
program_name, long_options[option_index].name, option_index);
guestfs___program_name, long_options[option_index].name, option_index);
exit (EXIT_FAILURE);
}
break;
Expand Down
10 changes: 5 additions & 5 deletions cat/filesystems.c
Expand Up @@ -85,7 +85,7 @@ usage (int status)
{
if (status != EXIT_SUCCESS)
fprintf (stderr, _("Try `%s --help' for more information.\n"),
program_name);
guestfs___program_name);
else {
fprintf (stdout,
_("%s: list filesystems, partitions, block devices, LVM in a VM\n"
Expand Down Expand Up @@ -122,8 +122,8 @@ usage (int status)
" Display LVM volume groups\n"
" -x Trace libguestfs API calls\n"
"For more information, see the manpage %s(1).\n"),
program_name, program_name, program_name,
program_name);
guestfs___program_name, guestfs___program_name, guestfs___program_name,
guestfs___program_name);
}
exit (status);
}
Expand Down Expand Up @@ -240,7 +240,7 @@ main (int argc, char *argv[])
output |= OUTPUT_VGS;
} else {
fprintf (stderr, _("%s: unknown long option: %s (%d)\n"),
program_name, long_options[option_index].name, option_index);
guestfs___program_name, long_options[option_index].name, option_index);
exit (EXIT_FAILURE);
}
break;
Expand Down Expand Up @@ -304,7 +304,7 @@ main (int argc, char *argv[])
*/
if (human && csv) {
fprintf (stderr, _("%s: you cannot use -h and --csv options together.\n"),
program_name);
guestfs___program_name);
exit (EXIT_FAILURE);
}

Expand Down
18 changes: 9 additions & 9 deletions cat/log.c
Expand Up @@ -60,7 +60,7 @@ usage (int status)
{
if (status != EXIT_SUCCESS)
fprintf (stderr, _("Try `%s --help' for more information.\n"),
program_name);
guestfs___program_name);
else {
fprintf (stdout,
_("%s: display log files in a virtual machine\n"
Expand All @@ -80,8 +80,8 @@ usage (int status)
" -V|--version Display version and exit\n"
" -x Trace libguestfs API calls\n"
"For more information, see the manpage %s(1).\n"),
program_name, program_name, program_name,
program_name);
guestfs___program_name, guestfs___program_name, guestfs___program_name,
guestfs___program_name);
}
exit (status);
}
Expand Down Expand Up @@ -141,7 +141,7 @@ main (int argc, char *argv[])
OPTION_format;
} else {
fprintf (stderr, _("%s: unknown long option: %s (%d)\n"),
program_name, long_options[option_index].name, option_index);
guestfs___program_name, long_options[option_index].name, option_index);
exit (EXIT_FAILURE);
}
break;
Expand Down Expand Up @@ -244,7 +244,7 @@ do_log (void)

fprintf (stderr,
_("%s: Windows Event Log for pre-Vista guests is not supported.\n"),
program_name);
guestfs___program_name);
return -1;
}

Expand Down Expand Up @@ -352,7 +352,7 @@ do_log_journal (void)
if (strftime (buf, sizeof buf, "%b %d %H:%M:%S",
localtime_r (&t, &tm)) <= 0) {
fprintf (stderr, _("%s: could not format journal entry timestamp\n"),
program_name);
guestfs___program_name);
errors++;
continue;
}
Expand Down Expand Up @@ -421,7 +421,7 @@ do_log_windows_evtx (void)
"in order to parse Windows Event Logs. If you cannot install this, then\n"
"use virt-copy-out(1) to copy the contents of /Windows/System32/winevt/Logs\n"
"from this guest, and examine in a binary file viewer.\n"),
program_name);
guestfs___program_name);
return -1;
}

Expand All @@ -437,7 +437,7 @@ do_log_windows_evtx (void)
GUESTFS_IS_FILE_OPTS_FOLLOWSYMLINKS, 1,
-1) <= 0) {
fprintf (stderr, _("%s: Windows Event Log file (%s) not found\n"),
program_name, filename);
guestfs___program_name, filename);
return -1;
}

Expand Down Expand Up @@ -469,7 +469,7 @@ do_log_windows_evtx (void)
if (status) {
char buf[256];
fprintf (stderr, "%s: %s\n",
program_name,
guestfs___program_name,
guestfs___exit_status_to_string (status, "evtxdump.py",
buf, sizeof buf));
return -1;
Expand Down
12 changes: 6 additions & 6 deletions cat/ls.c
Expand Up @@ -81,7 +81,7 @@ usage (int status)
{
if (status != EXIT_SUCCESS)
fprintf (stderr, _("Try `%s --help' for more information.\n"),
program_name);
guestfs___program_name);
else {
fprintf (stdout,
_("%s: list files in a virtual machine\n"
Expand Down Expand Up @@ -114,8 +114,8 @@ usage (int status)
" -V|--version Display version and exit\n"
" -x Trace libguestfs API calls\n"
"For more information, see the manpage %s(1).\n"),
program_name, program_name, program_name,
program_name);
guestfs___program_name, guestfs___program_name, guestfs___program_name,
guestfs___program_name);
}
exit (status);
}
Expand Down Expand Up @@ -229,7 +229,7 @@ main (int argc, char *argv[])
enable_uids = 1;
} else {
fprintf (stderr, _("%s: unknown long option: %s (%d)\n"),
program_name, long_options[option_index].name, option_index);
guestfs___program_name, long_options[option_index].name, option_index);
exit (EXIT_FAILURE);
}
break;
Expand Down Expand Up @@ -335,7 +335,7 @@ main (int argc, char *argv[])
(csv || human || enable_uids || enable_times || enable_extra_stats ||
checksum)) {
fprintf (stderr, _("%s: used a flag which can only be combined with -lR mode\nFor more information, read the virt-ls(1) man page.\n"),
program_name);
guestfs___program_name);
exit (EXIT_FAILURE);
}

Expand All @@ -344,7 +344,7 @@ main (int argc, char *argv[])
*/
if (human && csv) {
fprintf (stderr, _("%s: you cannot use -h and --csv options together.\n"),
program_name);
guestfs___program_name);
exit (EXIT_FAILURE);
}

Expand Down
4 changes: 2 additions & 2 deletions cat/visit.c
Expand Up @@ -100,7 +100,7 @@ _visit (guestfs_h *g, int depth, const char *dir,

if (xattrs->val[xattrp].attrval_len == 0) {
fprintf (stderr, _("%s: error getting extended attrs for %s %s\n"),
program_name, dir, names[i]);
guestfs___program_name, dir, names[i]);
return -1;
}
/* attrval is not \0-terminated. */
Expand All @@ -110,7 +110,7 @@ _visit (guestfs_h *g, int depth, const char *dir,
attrval[xattrs->val[xattrp].attrval_len] = '\0';
if (sscanf (attrval, "%zu", &nr_xattrs) != 1) {
fprintf (stderr, _("%s: error: cannot parse xattr count for %s %s\n"),
program_name, dir, names[i]);
guestfs___program_name, dir, names[i]);
return -1;
}

Expand Down
10 changes: 5 additions & 5 deletions df/domains.c
Expand Up @@ -83,7 +83,7 @@ get_all_libvirt_domains (const char *libvirt_uri)
err = virGetLastError ();
fprintf (stderr,
_("%s: could not connect to libvirt (code %d, domain %d): %s\n"),
program_name, err->code, err->domain, err->message);
guestfs___program_name, err->code, err->domain, err->message);
exit (EXIT_FAILURE);
}

Expand All @@ -92,7 +92,7 @@ get_all_libvirt_domains (const char *libvirt_uri)
err = virGetLastError ();
fprintf (stderr,
_("%s: could not get number of running domains (code %d, domain %d): %s\n"),
program_name, err->code, err->domain, err->message);
guestfs___program_name, err->code, err->domain, err->message);
exit (EXIT_FAILURE);
}

Expand All @@ -102,7 +102,7 @@ get_all_libvirt_domains (const char *libvirt_uri)
err = virGetLastError ();
fprintf (stderr,
_("%s: could not list running domains (code %d, domain %d): %s\n"),
program_name, err->code, err->domain, err->message);
guestfs___program_name, err->code, err->domain, err->message);
exit (EXIT_FAILURE);
}

Expand All @@ -113,7 +113,7 @@ get_all_libvirt_domains (const char *libvirt_uri)
err = virGetLastError ();
fprintf (stderr,
_("%s: could not get number of inactive domains (code %d, domain %d): %s\n"),
program_name, err->code, err->domain, err->message);
guestfs___program_name, err->code, err->domain, err->message);
exit (EXIT_FAILURE);
}

Expand All @@ -123,7 +123,7 @@ get_all_libvirt_domains (const char *libvirt_uri)
err = virGetLastError ();
fprintf (stderr,
_("%s: could not list inactive domains (code %d, domain %d): %s\n"),
program_name, err->code, err->domain, err->message);
guestfs___program_name, err->code, err->domain, err->message);
exit (EXIT_FAILURE);
}

Expand Down

0 comments on commit 5cafeda

Please sign in to comment.