diff mbox

opensm: Add configure output messages for several configure options

Message ID 1365097110.5901.36.camel@auk59.llnl.gov (mailing list archive)
State Superseded
Delegated to: Hal Rosenstock
Headers show

Commit Message

Al Chu April 4, 2013, 5:38 p.m. UTC
The --enable-console-loopback, --enable-console-socket, --enable-perf-mgr,
--enable-perf-mgr-profile, and --enable-default-event-plugin did not
have a checking message or notice message indicating if they were
enabled/disabled when running configure.

This made things difficult when perusing old build logs to determine
if things were enabled/disabled.

Signed-off-by: Albert Chu <chu11@llnl.gov>
---
 config/osmvsel.m4 |   14 ++++++++++++++
 1 files changed, 14 insertions(+), 0 deletions(-)

Comments

Hal Rosenstock April 5, 2013, 8:30 p.m. UTC | #1
On 4/4/2013 1:38 PM, Albert Chu wrote:
> The --enable-console-loopback, --enable-console-socket, --enable-perf-mgr,
> --enable-perf-mgr-profile, and --enable-default-event-plugin did not
> have a checking message or notice message indicating if they were
> enabled/disabled when running configure.
> 
> This made things difficult when perusing old build logs to determine
> if things were enabled/disabled.
> 
> Signed-off-by: Albert Chu <chu11@llnl.gov>
> ---
>  config/osmvsel.m4 |   14 ++++++++++++++
>  1 files changed, 14 insertions(+), 0 deletions(-)
> 
> diff --git a/config/osmvsel.m4 b/config/osmvsel.m4
> index 6c98c07..f249b1c 100644
> --- a/config/osmvsel.m4
> +++ b/config/osmvsel.m4
> @@ -183,6 +183,7 @@ AC_DEFUN([OPENIB_OSM_CONSOLE_SOCKET_SEL], [
>  # --- BEGIN OPENIB_OSM_CONSOLE_SOCKET_SEL ---
>  
>  dnl Console over a loopback socket is default if libwrap is available
> +AC_MSG_CHECKING([to enable console loopback])
>  AC_ARG_ENABLE(console-loopback,
>  [  --enable-console-loopback Enable a console socket on the loopback interface, requires tcp_wrappers (default yes)],
>  [case $enableval in
> @@ -190,6 +191,7 @@ AC_ARG_ENABLE(console-loopback,
>       no)  console_loopback=no ;;
>     esac],
>     console_loopback=yes)
> +AC_MSG_RESULT([yes])

Should this be AC_MSG_RESULT[$console_loopback] ?

>  if test $console_loopback = yes; then
>  AC_CHECK_LIB(wrap, request_init, [], [console_loopback=no
> @@ -202,6 +204,7 @@ if test $console_loopback = yes; then
>  fi
>  
>  dnl Console over a socket connection
> +AC_MSG_CHECKING([to enable console socket])
>  AC_ARG_ENABLE(console-socket,
>  [  --enable-console-socket Enable a console socket, requires --enable-console-loopback (default no)],
>  [case $enableval in
> @@ -209,6 +212,8 @@ AC_ARG_ENABLE(console-socket,
>       no)  console_socket=no ;;
>     esac],
>     console_socket=no)
> +AC_MSG_RESULT([no])

Similar to console_loopback question.

> +
>  if test $console_socket = yes; then
>    if test $console_loopback = no; then
>      AC_MSG_ERROR([--enable-console-socket requires --enable-console-loopback])
> @@ -228,6 +233,7 @@ AC_DEFUN([OPENIB_OSM_PERF_MGR_SEL], [
>  # --- BEGIN OPENIB_OSM_PERF_MGR_SEL ---
>  
>  dnl enable the perf-mgr
> +AC_MSG_CHECKING([to enable perf mgr])
>  AC_ARG_ENABLE(perf-mgr,
>  [  --enable-perf-mgr Enable the performance manager (default yes)],
>     [case $enableval in
> @@ -235,6 +241,9 @@ AC_ARG_ENABLE(perf-mgr,
>       no)  perf_mgr=no ;;
>     esac],
>     perf_mgr=yes)
> +AC_MSG_RESULT([yes])

Ditto, etc.

-- Hal

> +
> +AC_MSG_CHECKING([to enable perf mgr profiling])
>  AC_ARG_ENABLE(perf-mgr-profile,
>  [  --enable-perf-mgr-profile Enable the performance manager profiling (default no)],
>  	[case $enableval in
> @@ -242,6 +251,8 @@ AC_ARG_ENABLE(perf-mgr-profile,
>  	no)  perf_mgr_profile=no ;;
>  	esac],
>  	perf_mgr_profile=no)
> +AC_MSG_RESULT([no])
> +
>  if test $perf_mgr = yes; then
>    AC_DEFINE(ENABLE_OSM_PERF_MGR,
>  	1,
> @@ -261,6 +272,7 @@ AC_DEFUN([OPENIB_OSM_DEFAULT_EVENT_PLUGIN_SEL], [
>  # --- BEGIN OPENIB_OSM_DEFAULT_EVENT_PLUGIN_SEL ---
>  
>  dnl enable the default-event-plugin
> +AC_MSG_CHECKING([to enable default event plugin])
>  AC_ARG_ENABLE(default-event-plugin,
>  [  --enable-default-event-plugin  Enable a default event plugin "osmeventplugin" (default no)],
>     [case $enableval in
> @@ -268,6 +280,8 @@ AC_ARG_ENABLE(default-event-plugin,
>       no)  default_event_plugin=no ;;
>     esac],
>     default_event_plugin=no)
> +AC_MSG_RESULT([no])
> +
>  if test $default_event_plugin = yes; then
>    AC_DEFINE(ENABLE_OSM_DEFAULT_EVENT_PLUGIN,
>  	    1,

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Al Chu April 5, 2013, 9:23 p.m. UTC | #2
On Fri, 2013-04-05 at 16:30 -0400, Hal Rosenstock wrote:
> On 4/4/2013 1:38 PM, Albert Chu wrote:
> > The --enable-console-loopback, --enable-console-socket, --enable-perf-mgr,
> > --enable-perf-mgr-profile, and --enable-default-event-plugin did not
> > have a checking message or notice message indicating if they were
> > enabled/disabled when running configure.
> > 
> > This made things difficult when perusing old build logs to determine
> > if things were enabled/disabled.
> > 
> > Signed-off-by: Albert Chu <chu11@llnl.gov>
> > ---
> >  config/osmvsel.m4 |   14 ++++++++++++++
> >  1 files changed, 14 insertions(+), 0 deletions(-)
> > 
> > diff --git a/config/osmvsel.m4 b/config/osmvsel.m4
> > index 6c98c07..f249b1c 100644
> > --- a/config/osmvsel.m4
> > +++ b/config/osmvsel.m4
> > @@ -183,6 +183,7 @@ AC_DEFUN([OPENIB_OSM_CONSOLE_SOCKET_SEL], [
> >  # --- BEGIN OPENIB_OSM_CONSOLE_SOCKET_SEL ---
> >  
> >  dnl Console over a loopback socket is default if libwrap is available
> > +AC_MSG_CHECKING([to enable console loopback])
> >  AC_ARG_ENABLE(console-loopback,
> >  [  --enable-console-loopback Enable a console socket on the loopback interface, requires tcp_wrappers (default yes)],
> >  [case $enableval in
> > @@ -190,6 +191,7 @@ AC_ARG_ENABLE(console-loopback,
> >       no)  console_loopback=no ;;
> >     esac],
> >     console_loopback=yes)
> > +AC_MSG_RESULT([yes])
> 
> Should this be AC_MSG_RESULT[$console_loopback] ?

Argh!  I could have swore I had this working correctly.  I'll fix this
up and repost.

Al

> >  if test $console_loopback = yes; then
> >  AC_CHECK_LIB(wrap, request_init, [], [console_loopback=no
> > @@ -202,6 +204,7 @@ if test $console_loopback = yes; then
> >  fi
> >  
> >  dnl Console over a socket connection
> > +AC_MSG_CHECKING([to enable console socket])
> >  AC_ARG_ENABLE(console-socket,
> >  [  --enable-console-socket Enable a console socket, requires --enable-console-loopback (default no)],
> >  [case $enableval in
> > @@ -209,6 +212,8 @@ AC_ARG_ENABLE(console-socket,
> >       no)  console_socket=no ;;
> >     esac],
> >     console_socket=no)
> > +AC_MSG_RESULT([no])
> 
> Similar to console_loopback question.
> 
> > +
> >  if test $console_socket = yes; then
> >    if test $console_loopback = no; then
> >      AC_MSG_ERROR([--enable-console-socket requires --enable-console-loopback])
> > @@ -228,6 +233,7 @@ AC_DEFUN([OPENIB_OSM_PERF_MGR_SEL], [
> >  # --- BEGIN OPENIB_OSM_PERF_MGR_SEL ---
> >  
> >  dnl enable the perf-mgr
> > +AC_MSG_CHECKING([to enable perf mgr])
> >  AC_ARG_ENABLE(perf-mgr,
> >  [  --enable-perf-mgr Enable the performance manager (default yes)],
> >     [case $enableval in
> > @@ -235,6 +241,9 @@ AC_ARG_ENABLE(perf-mgr,
> >       no)  perf_mgr=no ;;
> >     esac],
> >     perf_mgr=yes)
> > +AC_MSG_RESULT([yes])
> 
> Ditto, etc.
> 
> -- Hal
> 
> > +
> > +AC_MSG_CHECKING([to enable perf mgr profiling])
> >  AC_ARG_ENABLE(perf-mgr-profile,
> >  [  --enable-perf-mgr-profile Enable the performance manager profiling (default no)],
> >  	[case $enableval in
> > @@ -242,6 +251,8 @@ AC_ARG_ENABLE(perf-mgr-profile,
> >  	no)  perf_mgr_profile=no ;;
> >  	esac],
> >  	perf_mgr_profile=no)
> > +AC_MSG_RESULT([no])
> > +
> >  if test $perf_mgr = yes; then
> >    AC_DEFINE(ENABLE_OSM_PERF_MGR,
> >  	1,
> > @@ -261,6 +272,7 @@ AC_DEFUN([OPENIB_OSM_DEFAULT_EVENT_PLUGIN_SEL], [
> >  # --- BEGIN OPENIB_OSM_DEFAULT_EVENT_PLUGIN_SEL ---
> >  
> >  dnl enable the default-event-plugin
> > +AC_MSG_CHECKING([to enable default event plugin])
> >  AC_ARG_ENABLE(default-event-plugin,
> >  [  --enable-default-event-plugin  Enable a default event plugin "osmeventplugin" (default no)],
> >     [case $enableval in
> > @@ -268,6 +280,8 @@ AC_ARG_ENABLE(default-event-plugin,
> >       no)  default_event_plugin=no ;;
> >     esac],
> >     default_event_plugin=no)
> > +AC_MSG_RESULT([no])
> > +
> >  if test $default_event_plugin = yes; then
> >    AC_DEFINE(ENABLE_OSM_DEFAULT_EVENT_PLUGIN,
> >  	    1,
>
diff mbox

Patch

diff --git a/config/osmvsel.m4 b/config/osmvsel.m4
index 6c98c07..f249b1c 100644
--- a/config/osmvsel.m4
+++ b/config/osmvsel.m4
@@ -183,6 +183,7 @@  AC_DEFUN([OPENIB_OSM_CONSOLE_SOCKET_SEL], [
 # --- BEGIN OPENIB_OSM_CONSOLE_SOCKET_SEL ---
 
 dnl Console over a loopback socket is default if libwrap is available
+AC_MSG_CHECKING([to enable console loopback])
 AC_ARG_ENABLE(console-loopback,
 [  --enable-console-loopback Enable a console socket on the loopback interface, requires tcp_wrappers (default yes)],
 [case $enableval in
@@ -190,6 +191,7 @@  AC_ARG_ENABLE(console-loopback,
      no)  console_loopback=no ;;
    esac],
    console_loopback=yes)
+AC_MSG_RESULT([yes])
 
 if test $console_loopback = yes; then
 AC_CHECK_LIB(wrap, request_init, [], [console_loopback=no
@@ -202,6 +204,7 @@  if test $console_loopback = yes; then
 fi
 
 dnl Console over a socket connection
+AC_MSG_CHECKING([to enable console socket])
 AC_ARG_ENABLE(console-socket,
 [  --enable-console-socket Enable a console socket, requires --enable-console-loopback (default no)],
 [case $enableval in
@@ -209,6 +212,8 @@  AC_ARG_ENABLE(console-socket,
      no)  console_socket=no ;;
    esac],
    console_socket=no)
+AC_MSG_RESULT([no])
+
 if test $console_socket = yes; then
   if test $console_loopback = no; then
     AC_MSG_ERROR([--enable-console-socket requires --enable-console-loopback])
@@ -228,6 +233,7 @@  AC_DEFUN([OPENIB_OSM_PERF_MGR_SEL], [
 # --- BEGIN OPENIB_OSM_PERF_MGR_SEL ---
 
 dnl enable the perf-mgr
+AC_MSG_CHECKING([to enable perf mgr])
 AC_ARG_ENABLE(perf-mgr,
 [  --enable-perf-mgr Enable the performance manager (default yes)],
    [case $enableval in
@@ -235,6 +241,9 @@  AC_ARG_ENABLE(perf-mgr,
      no)  perf_mgr=no ;;
    esac],
    perf_mgr=yes)
+AC_MSG_RESULT([yes])
+
+AC_MSG_CHECKING([to enable perf mgr profiling])
 AC_ARG_ENABLE(perf-mgr-profile,
 [  --enable-perf-mgr-profile Enable the performance manager profiling (default no)],
 	[case $enableval in
@@ -242,6 +251,8 @@  AC_ARG_ENABLE(perf-mgr-profile,
 	no)  perf_mgr_profile=no ;;
 	esac],
 	perf_mgr_profile=no)
+AC_MSG_RESULT([no])
+
 if test $perf_mgr = yes; then
   AC_DEFINE(ENABLE_OSM_PERF_MGR,
 	1,
@@ -261,6 +272,7 @@  AC_DEFUN([OPENIB_OSM_DEFAULT_EVENT_PLUGIN_SEL], [
 # --- BEGIN OPENIB_OSM_DEFAULT_EVENT_PLUGIN_SEL ---
 
 dnl enable the default-event-plugin
+AC_MSG_CHECKING([to enable default event plugin])
 AC_ARG_ENABLE(default-event-plugin,
 [  --enable-default-event-plugin  Enable a default event plugin "osmeventplugin" (default no)],
    [case $enableval in
@@ -268,6 +280,8 @@  AC_ARG_ENABLE(default-event-plugin,
      no)  default_event_plugin=no ;;
    esac],
    default_event_plugin=no)
+AC_MSG_RESULT([no])
+
 if test $default_event_plugin = yes; then
   AC_DEFINE(ENABLE_OSM_DEFAULT_EVENT_PLUGIN,
 	    1,