[SSSD] [PATCH] Add support for experimental features

Sumit Bose sbose at redhat.com
Thu Jun 23 11:41:42 UTC 2011


and now with patch ...

On Thu, Jun 23, 2011 at 01:40:51PM +0200, Sumit Bose wrote:
> Hi,
> 
> We will have some experimental features added in the near future and
> this patch tries to help to identify them. One part is a new configure
> option which enables all experimental features which check if the option
> is set. The second is a small include file for man pages to indicate
> that a certain feature is experimental.
> 
> bye,
> Sumit
-------------- next part --------------
From 035666e2bcc56354c9dc48cc757754647fc8b41c Mon Sep 17 00:00:00 2001
From: Sumit Bose <sbose at redhat.com>
Date: Tue, 21 Jun 2011 14:17:22 +0200
Subject: [PATCH] Add support for experimental features

New experimental features should have their own configure switch to
enable or disable them at compile time. Additionally they can check if
the configure variable build_all_experimental_features is set and enable
the feature. This variable will be set if the command line option
--enable-all-experimental-features is used to configure sssd. This will
make it easy to enable all experimental features.

Experimental features should be marked in the man pages. To simplify
this include/experimental.xml can be used.
---
 src/conf_macros.m4               |    6 ++++++
 src/man/include/experimental.xml |    4 ++++
 2 files changed, 10 insertions(+), 0 deletions(-)
 create mode 100644 src/man/include/experimental.xml

diff --git a/src/conf_macros.m4 b/src/conf_macros.m4
index af9cb4f..1e503b3 100644
--- a/src/conf_macros.m4
+++ b/src/conf_macros.m4
@@ -331,3 +331,9 @@ AC_DEFUN([WITH_NOLOGIN_SHELL],
     fi
     AC_DEFINE_UNQUOTED(NOLOGIN_SHELL, "$nologin_shell", [The shell used to deny access to users])
   ])
+
+AC_ARG_ENABLE([all-experimental-features],
+              [AS_HELP_STRING([--enable-all-experimental-features],
+                              [build all experimental features])],
+              [build_all_experimental_features=$enableval],
+              [build_all_experimental_features=no])
diff --git a/src/man/include/experimental.xml b/src/man/include/experimental.xml
new file mode 100644
index 0000000..134bac9
--- /dev/null
+++ b/src/man/include/experimental.xml
@@ -0,0 +1,4 @@
+<emphasis>
+This is an experimental feature, please use http://fedorahosted.org/sssd to
+report any issues.
+</emphasis>
-- 
1.7.5.4



More information about the sssd-devel mailing list