Message ID | 1365097110.5901.36.camel@auk59.llnl.gov (mailing list archive) |
---|---|
State | Superseded |
Delegated to: | Hal Rosenstock |
Headers | show |
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
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 --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,
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(-)