diff mbox

[Xen-devel,GIT,PULL,v2] remove xend and libxen for 4.5

Message ID 1397035429.31448.37.camel@kazak.uk.xensource.com
State Accepted
Commit 28d9b22d1b5308bb25e857635fc75cfbe23c3b63
Headers show

Commit Message

Ian Campbell April 9, 2014, 9:23 a.m. UTC
On Wed, 2014-04-09 at 11:14 +0200, Olaf Hering wrote:
> On Wed, Apr 09, Ian Campbell wrote:
> 
> > On Wed, 2014-04-09 at 11:03 +0200, Olaf Hering wrote:
> > > On Wed, Apr 09, Ian Campbell wrote:
> > > 
> > > > Looks like some stray CONFIG_XENDs remained after rebasing for v2.
> > > > 
> > > > However you should also have rerun configure so CONFIG_XEND should not
> > > > be set anyway. The following takes care of the first issue.
> > > 
> > > Thanks, with this patch the rpmbuild makes progress past the 'make' call.
> > 
> > Thanks, I'll take this as a Tested-by: Olaf Hering <olaf@aepfle.de> if
> > you don't mind.
> 
> Yes, please.
> 
> > How was CONFIG_XEND getting set in your build though?
> 
> I had configure --enable-xend in my rpm spec file.

But nothing should understand enable-xend any more!

Ah, I also missed the change to docs/configure.ac. Sigh....

-----8<---------

From c781e81173de31077ac79d67aab7ce4879636bc3 Mon Sep 17 00:00:00 2001
From: Ian Campbell <ian.campbell@citrix.com>
Date: Wed, 9 Apr 2014 09:26:23 +0100
Subject: [PATCH v2] docs: remove stray CONFIG_XENDs and configure option from
 docs.

These were added by 7dbfc2f8b054 "docs: Honour --{en, dis}able-xend when
building docs" between v1 and the (eventually committed) v2 of 9e8672f1c36d
"tools: remove xend and associated python modules" and were missed when
rebasing for v2.

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Tested-by: Olaf Hering <olaf@aepfle.de>
---
v2: Also remove option from docs/configure.
---
 config/Docs.mk.in |    3 ---
 docs/Makefile     |    2 --
 docs/configure    |   34 ----------------------------------
 docs/configure.ac |    3 ---
 4 files changed, 42 deletions(-)

Comments

Olaf Hering April 9, 2014, 9:40 a.m. UTC | #1
On Wed, Apr 09, Ian Campbell wrote:

> But nothing should understand enable-xend any more!
> 
> Ah, I also missed the change to docs/configure.ac. Sigh....

This works as well for me. Thanks.

Olaf
Ian Jackson April 9, 2014, 10:42 a.m. UTC | #2
Ian Campbell writes ("Re: [Xen-devel] [GIT PULL v2] remove xend and libxen for 4.5"):
> Ah, I also missed the change to docs/configure.ac. Sigh....

Oops.

> From: Ian Campbell <ian.campbell@citrix.com>
> Date: Wed, 9 Apr 2014 09:26:23 +0100
> Subject: [PATCH v2] docs: remove stray CONFIG_XENDs and configure option from
>  docs.
> 
> These were added by 7dbfc2f8b054 "docs: Honour --{en, dis}able-xend when
> building docs" between v1 and the (eventually committed) v2 of 9e8672f1c36d
> "tools: remove xend and associated python modules" and were missed when
> rebasing for v2.
> 
> Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> Tested-by: Olaf Hering <olaf@aepfle.de>

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

Maybe
  git-grep -i xend
will spot more ?

Ian.
Ian Campbell April 9, 2014, 10:56 a.m. UTC | #3
On Wed, 2014-04-09 at 11:42 +0100, Ian Jackson wrote:
> Ian Campbell writes ("Re: [Xen-devel] [GIT PULL v2] remove xend and libxen for 4.5"):
> > Ah, I also missed the change to docs/configure.ac. Sigh....
> 
> Oops.
> 
> > From: Ian Campbell <ian.campbell@citrix.com>
> > Date: Wed, 9 Apr 2014 09:26:23 +0100
> > Subject: [PATCH v2] docs: remove stray CONFIG_XENDs and configure option from
> >  docs.
> > 
> > These were added by 7dbfc2f8b054 "docs: Honour --{en, dis}able-xend when
> > building docs" between v1 and the (eventually committed) v2 of 9e8672f1c36d
> > "tools: remove xend and associated python modules" and were missed when
> > rebasing for v2.
> > 
> > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> > Tested-by: Olaf Hering <olaf@aepfle.de>
> 
> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
> 
> Maybe
>   git-grep -i xend
> will spot more ?

Sigh, yes, because I forgot about -i...

Ian.
Ian Campbell April 10, 2014, 11:36 a.m. UTC | #4
On Wed, 2014-04-09 at 11:42 +0100, Ian Jackson wrote:
> Ian Campbell writes ("Re: [Xen-devel] [GIT PULL v2] remove xend and libxen for 4.5"):
> > Ah, I also missed the change to docs/configure.ac. Sigh....
> 
> Oops.
> 
> > From: Ian Campbell <ian.campbell@citrix.com>
> > Date: Wed, 9 Apr 2014 09:26:23 +0100
> > Subject: [PATCH v2] docs: remove stray CONFIG_XENDs and configure option from
> >  docs.
> > 
> > These were added by 7dbfc2f8b054 "docs: Honour --{en, dis}able-xend when
> > building docs" between v1 and the (eventually committed) v2 of 9e8672f1c36d
> > "tools: remove xend and associated python modules" and were missed when
> > rebasing for v2.
> > 
> > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> > Tested-by: Olaf Hering <olaf@aepfle.de>
> 
> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

Applied, thanks.

> Maybe
>   git-grep -i xend
> will spot more ?

I've got a branch.... Needs more cleanup though...

Ian.
diff mbox

Patch

diff --git a/config/Docs.mk.in b/config/Docs.mk.in
index 03fac60..a2a72fd 100644
--- a/config/Docs.mk.in
+++ b/config/Docs.mk.in
@@ -12,6 +12,3 @@  POD2HTML            := @POD2HTML@
 POD2TEXT            := @POD2TEXT@
 MARKDOWN            := @MARKDOWN@
 PERL                := @PERL@
-
-# Subsets of documentation to build
-CONFIG_XEND         := @xend@
diff --git a/docs/Makefile b/docs/Makefile
index 7cd7440..5153bcb 100644
--- a/docs/Makefile
+++ b/docs/Makefile
@@ -9,10 +9,8 @@  DOC_ARCHES      := arm x86_32 x86_64
 # Documentation sources to build
 MAN1SRC-y := $(wildcard man/xl*.pod.1)
 MAN1SRC-y += $(wildcard man/xenstore*.pod.1)
-MAN1SRC-$(CONFIG_XEND) += man/xm.pod.1
 
 MAN5SRC-y := $(wildcard man/xl*.pod.5)
-MAN5SRC-$(CONFIG_XEND) += man/xend-config.sxp.pod.5 man/xmdomain.cfg.pod.5
 
 MARKDOWNSRC-y := $(wildcard misc/*.markdown)
 
diff --git a/docs/configure b/docs/configure
index c54f431..d76e4d4 100755
--- a/docs/configure
+++ b/docs/configure
@@ -588,7 +588,6 @@  ac_unique_file="misc/xen-command-line.markdown"
 ac_subst_vars='LTLIBOBJS
 LIBOBJS
 PERL
-xend
 MARKDOWN
 POD2TEXT
 POD2HTML
@@ -635,7 +634,6 @@  SHELL'
 ac_subst_files=''
 ac_user_opts='
 enable_option_checking
-enable_xend
 '
       ac_precious_vars='build_alias
 host_alias
@@ -1251,13 +1249,6 @@  if test -n "$ac_init_help"; then
    esac
   cat <<\_ACEOF
 
-Optional Features:
-  --disable-option-checking  ignore unrecognized --enable/--with options
-  --disable-FEATURE       do not include FEATURE (same as --enable-FEATURE=no)
-  --enable-FEATURE[=ARG]  include FEATURE [ARG=yes]
-  --enable-xend           Enable xend toolstack documentation (default is
-                          DISABLED)
-
 Some influential environment variables:
   FIG2DEV     Path to fig2dev tool
   POD2MAN     Path to pod2man tool
@@ -1996,31 +1987,6 @@  $as_echo "$as_me: WARNING: markdown is not available so some documentation won't
 fi
 
 
-# Enable/disable options
-
-# Check whether --enable-xend was given.
-if test "${enable_xend+set}" = set; then :
-  enableval=$enable_xend;
-fi
-
-
-if test "x$enable_xend" = "xno"; then :
-
-    ax_cv_xend="n"
-
-elif test "x$enable_xend" = "xyes"; then :
-
-    ax_cv_xend="y"
-
-elif test -z $ax_cv_xend; then :
-
-    ax_cv_xend="n"
-
-fi
-xend=$ax_cv_xend
-
-
-
 
 # Extract the first word of "perl", so it can be a program name with args.
 set dummy perl; ac_word=$2
diff --git a/docs/configure.ac b/docs/configure.ac
index 7e6d1b1..e545b00 100644
--- a/docs/configure.ac
+++ b/docs/configure.ac
@@ -19,9 +19,6 @@  AX_DOCS_TOOL_PROG([POD2HTML], [pod2html])
 AX_DOCS_TOOL_PROG([POD2TEXT], [pod2text])
 AX_DOCS_TOOL_PROGS([MARKDOWN], [markdown], [markdown markdown_py])
 
-# Enable/disable options
-AX_ARG_DEFAULT_DISABLE([xend], [Enable xend toolstack documentation])
-
 AC_ARG_VAR([PERL], [Path to Perl parser])
 AX_PATH_PROG_OR_FAIL([PERL], [perl])