xl: remove parsing of "vncviewer" option in xl domain config file
authorWei Liu <wei.liu2@citrix.com>
Mon, 9 Jun 2014 12:43:16 +0000 (13:43 +0100)
committerIan Campbell <ian.campbell@citrix.com>
Tue, 10 Jun 2014 13:05:23 +0000 (14:05 +0100)
commitc28219f035562866934cdf6d39deac565553d6c2
tree11ba4360171b497613538d40f28d3e2156ac1af0
parent3a773be856b8ce55de6e4cd5aa8d69e920b4bba0
xl: remove parsing of "vncviewer" option in xl domain config file

Print out a warning and suggest user use "-V" option when invoking "xl
create". Also remove that option in manpage. This will introduce a
minor functional regression but it's very easy to work around.

The rationale behind this change is that, this option is actually not
part of domain configuration. It just affects whether a vncviewer
should be automatically spawn, but has nothing to do with how a domain
should be constructed. And this option is also bogus, considering if you
migrate a domain to a remote host and the receiver spawns a vncviewer on
the receiving side then it either dies silently or occupies resource.

Signed-off-by: Wei Liu <wei.liu2@citrix.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
docs/man/xl.cfg.pod.5
tools/libxl/xl_cmdimpl.c