xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
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

             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).