From: Ian Campbell <ian.campbell@citrix.com>
To: xen-devel@lists.xen.org
Cc: ian.jackson@eu.citrix.com, Ian Campbell <ian.campbell@citrix.com>
Subject: [PATCH] tools: disable xend build by default
Date: Thu, 12 Sep 2013 11:34:12 +0100 [thread overview]
Message-ID: <1378982052-11270-1-git-send-email-ian.campbell@citrix.com> (raw)
Add big warnings to configure, xend initscript and xm.
The big xm warning is displayed once (per boot, or per tmpreaper clean),
afterwards a single line warning is displayed.
Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
tools/configure | 27 +++++++++++++++++++++++++--
tools/configure.ac | 15 ++++++++++++++-
tools/hotplug/Linux/init.d/xend | 4 ++++
tools/python/xen/xm/xm | 15 ++++++++++++++-
4 files changed, 57 insertions(+), 4 deletions(-)
diff --git a/tools/configure b/tools/configure
index 0dee92a..1da8652 100755
--- a/tools/configure
+++ b/tools/configure
@@ -1424,7 +1424,7 @@ Optional Features:
--disable-rombios Disable ROM BIOS (default is ENABLED)
--disable-seabios Disable SeaBIOS (default is ENABLED)
--disable-debug Disable debug build of tools (default is ENABLED)
- --disable-xend Disable xend toolstack (default is ENABLED)
+ --enable-xend Enable xend toolstack (default is DISABLED)
--enable-blktap1 Disable blktap1 tools (default is DISABLED)
--enable-qemu-traditional
Enable qemu traditional device model, (DEFAULT is on
@@ -3671,7 +3671,7 @@ elif test "x$enable_xend" = "xyes"; then :
elif test -z $ax_cv_xend; then :
- ax_cv_xend="y"
+ ax_cv_xend="n"
fi
xend=$ax_cv_xend
@@ -9235,3 +9235,26 @@ if test -n "$ac_unrecognized_opts" && test "$enable_option_checking" != no; then
$as_echo "$as_me: WARNING: unrecognized options: $ac_unrecognized_opts" >&2;}
fi
+
+if test "x$xend" = "xy" ; then :
+
+ { $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: You have enabled the xend toolstack.
+
+xend is deprecated and scheduled for removal. Please migrate to another
+toolstack ASAP.
+
+See http://wiki.xen.org/wiki/Choice_of_Toolstacks for information on
+other alternatives, including xl which is designed to be a drop in
+replacement for xm (http://wiki.xen.org/wiki/XL).
+" >&5
+$as_echo "$as_me: WARNING: You have enabled the xend toolstack.
+
+xend is deprecated and scheduled for removal. Please migrate to another
+toolstack ASAP.
+
+See http://wiki.xen.org/wiki/Choice_of_Toolstacks for information on
+other alternatives, including xl which is designed to be a drop in
+replacement for xm (http://wiki.xen.org/wiki/XL).
+" >&2;}
+
+fi
diff --git a/tools/configure.ac b/tools/configure.ac
index 2e57bfc..4f3c33a 100644
--- a/tools/configure.ac
+++ b/tools/configure.ac
@@ -57,7 +57,7 @@ AX_ARG_DEFAULT_DISABLE([ovmf], [Enable OVMF])
AX_ARG_DEFAULT_ENABLE([rombios], [Disable ROM BIOS])
AX_ARG_DEFAULT_ENABLE([seabios], [Disable SeaBIOS])
AX_ARG_DEFAULT_ENABLE([debug], [Disable debug build of tools])
-AX_ARG_DEFAULT_ENABLE([xend], [Disable xend toolstack])
+AX_ARG_DEFAULT_DISABLE([xend], [Enable xend toolstack])
AX_ARG_DEFAULT_DISABLE([blktap1], [Disable blktap1 tools])
AC_ARG_ENABLE([qemu-traditional],
@@ -220,3 +220,16 @@ AC_SUBST(libiconv)
AC_CHECK_HEADERS([yajl/yajl_version.h sys/eventfd.h])
AC_OUTPUT()
+
+AS_IF([test "x$xend" = "xy" ], [
+ AC_MSG_WARN(
+[You have enabled the xend toolstack.
+
+xend is deprecated and scheduled for removal. Please migrate to another
+toolstack ASAP.
+
+See http://wiki.xen.org/wiki/Choice_of_Toolstacks for information on
+other alternatives, including xl which is designed to be a drop in
+replacement for xm (http://wiki.xen.org/wiki/XL).
+])
+])
diff --git a/tools/hotplug/Linux/init.d/xend b/tools/hotplug/Linux/init.d/xend
index 5f92cdc..8921383 100755
--- a/tools/hotplug/Linux/init.d/xend
+++ b/tools/hotplug/Linux/init.d/xend
@@ -38,6 +38,10 @@ function await_daemons_up
case "$1" in
start)
+ echo "WARNING: Enabling the xend toolstack."
+ echo "xend is deprecated and scheduled for removal. Please migrate"
+ echo "to another toolstack ASAP."
+
if [ -z "`ps -C xenconsoled -o pid=`" ]; then
echo "xencommons should be started first."
exit 1
diff --git a/tools/python/xen/xm/xm b/tools/python/xen/xm/xm
index f4fd200..458f219 100755
--- a/tools/python/xen/xm/xm
+++ b/tools/python/xen/xm/xm
@@ -1,7 +1,20 @@
#!/usr/bin/env python
# -*- mode: python; -*-
-import sys
+import sys, os.path
from xen.xm import main
+print("WARNING: xend/xm is deprecated.")
+
+if not os.path.exists("/tmp/xm-deprecation-long-warning"):
+ print("""
+xend is deprecated and scheduled for removal. Please migrate to another
+toolstack ASAP.
+
+See http://wiki.xen.org/wiki/Choice_of_Toolstacks for information on
+other alternatives, including xl which is designed to be a drop in
+replacement for xm (http://wiki.xen.org/wiki/XL).
+""")
+ open("/tmp/xm-deprecation-long-warning", "w").close()
+
main.main(sys.argv)
--
1.7.10.4
next reply other threads:[~2013-09-12 10:34 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-12 10:34 Ian Campbell [this message]
2013-09-12 10:36 ` [PATCH] tools: disable xend build by default Ian Campbell
2013-09-12 11:12 ` Ian Jackson
2013-09-12 11:17 ` Ian Campbell
2013-09-12 13:03 ` Ian Jackson
2013-09-12 14:05 ` Ian Campbell
2013-09-12 14:22 ` Ian Jackson
2013-09-17 15:11 ` Ian Campbell
2013-09-17 16:36 ` Ian Jackson
2013-09-17 16:48 ` Ian Campbell
2013-09-12 10:42 ` Ian Campbell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1378982052-11270-1-git-send-email-ian.campbell@citrix.com \
--to=ian.campbell@citrix.com \
--cc=ian.jackson@eu.citrix.com \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).