[Libvir] Latest MinGW patch

Jim Meyering jim at meyering.net
Thu Dec 6 12:35:56 UTC 2007


"Richard W.M. Jones" <rjones at redhat.com> wrote:
> Index: src/virsh.c
> ===================================================================
> RCS file: /data/cvs/libvirt/src/virsh.c,v
> retrieving revision 1.117
> diff -u -r1.117 virsh.c
...
> @@ -3705,7 +3713,9 @@
>      {"autostart", cmdAutostart, opts_autostart, info_autostart},
>      {"capabilities", cmdCapabilities, NULL, info_capabilities},
>      {"connect", cmdConnect, opts_connect, info_connect},
> +#ifndef __MINGW32__
>      {"console", cmdConsole, opts_console, info_console},
> +#endif /* !__MINGW32__ */
>      {"create", cmdCreate, opts_create, info_create},
>      {"start", cmdStart, opts_start, info_start},
>      {"destroy", cmdDestroy, opts_destroy, info_destroy},

Please don't make the existence of an option depend on #ifdefs.
Then we'd have to adjust the manual and --help output, too.

In this particular case, it'd make --co and --con work (as aliases
for --connect) on mingw, while they don't work anywhere else
due to the ambiguity.

Instead, it'd be nice to handle the option and
give a diagnostic saying it's not supported on mingw.




More information about the libvir-list mailing list