[mpich-commits] [mpich] MPICH primary repository branch, master, updated. v3.0.4-168-gc478a4f

mysql vizuser noreply at mpich.org
Wed May 8 11:19:06 CDT 2013


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "MPICH primary repository".

The branch, master has been updated
       via  c478a4fe8d4cad97a2412c13b02d445181f78a56 (commit)
       via  50887ce11493c5aa872e3257b2e406174b59a12b (commit)
       via  5d6aefa53ac174f8e1c4e3fbbd5fdf0225f48108 (commit)
      from  5c4f5de7563db2bee452a102614c53b8ed9583ed (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://git.mpich.org/mpich.git/commitdiff/c478a4fe8d4cad97a2412c13b02d445181f78a56

commit c478a4fe8d4cad97a2412c13b02d445181f78a56
Author: Pavan Balaji <balaji at mcs.anl.gov>
Date:   Thu May 2 22:46:41 2013 -0500

    Flag push/pop cleanup.
    
    PAC_VAR_PUSHFLAG and PAC_VAR_POPFLAG are duplicates of PAC_PUSH_FLAG
    and PAC_POP_FLAG.  Also, unlike the PAC_VAR_ macro variants, the flag
    is not updated by the PAC_PUSH/PAC_POP macros, so the older value is
    retained.
    
    Also cleaned up macros that do additional logic to replace empty
    action-if-true/action-if-false statements with ":".  This is not
    required.  The macros work fine without this replacement.
    
    Reviewed by gropp.

diff --git a/confdb/aclocal_runlog.m4 b/confdb/aclocal_runlog.m4
index 2fdbfa5..e7b5b32 100644
--- a/confdb/aclocal_runlog.m4
+++ b/confdb/aclocal_runlog.m4
@@ -44,82 +44,6 @@ PAC_COMMAND_IFELSE([$1 > $pac_TESTLOG],[
 rm -f $pac_TESTLOG
 ])
 
-
-dnl PAC_VAR_PUSHVAL(VARNAME, [LastSavedValue]))
-dnl
-dnl Save the content of the shell variable, VARNAME, onto a stack.
-dnl The saved value of VARNAME is restorable with respect to the nesting
-dnl of the macro.
-dnl
-dnl The Last saved value of VARNAME on the stack is stored in shell variable
-dnl pac_LastSavedValueOf_$VARNAME if the 2nd argument is NOT supplied.
-dnl If the 2nd argument is present, the last saved value will be stored
-dnl in the 2nd argument instead.
-dnl
-dnl The First saved value of VARNAME on the stack is stored in shell variable
-dnl dnl pac_FirstSavedValueOf_$VARNAME.
-dnl
-AC_DEFUN([PAC_VAR_PUSHVAL],[
-# START of PUSHVAL
-dnl define local m4-name pac_stk_level.
-AS_VAR_PUSHDEF([pac_stk_level], [pac_stk_$1_level])
-AS_VAR_SET_IF([pac_stk_level],[
-    dnl autoconf < 2.64 does not have AS_VAR_ARITH, so use expr instead.
-    AS_VAR_SET([pac_stk_level], [`expr $pac_stk_level + 1`])
-],[
-    AS_VAR_SET([pac_stk_level], [0])
-])
-dnl AS_ECHO_N(["PUSHVAL: pac_stk_level = $pac_stk_level, "])
-dnl Save the content of VARNAME, i.e. $VARNAME, onto the stack.
-AS_VAR_SET([pac_stk_$1_$pac_stk_level],[$$1])
-AS_VAR_IF([pac_stk_level], [0], [
-    dnl Save the 1st pushed value of VARNAME as pac_FirstSavedValueOf_$VARNAME
-    AS_VAR_COPY([pac_FirstSavedValueOf_$1],[pac_stk_$1_$pac_stk_level])
-])
-ifelse([$2],[],[
-    dnl Save the last pushed value of VARNAME as pac_LastSavedValueOf_$VARNAME
-    AS_VAR_COPY([pac_LastSavedValueOf_$1],[pac_stk_$1_$pac_stk_level])
-    dnl AS_ECHO(["pac_LastSavedValueOf_$1 = $pac_LastSavedValueOf_$1"])
-],[
-    dnl Save the last pushed value of VARNAME as $2
-    AS_VAR_COPY([$2],[pac_stk_$1_$pac_stk_level])
-    dnl AS_ECHO(["$2 = $$2"])
-])
-AS_VAR_POPDEF([pac_stk_level])
-# END of PUSHVAL
-])
-dnl
-dnl
-dnl
-dnl PAC_VAR_POPVAL(VARNAME) 
-dnl
-dnl Restore variable, VARNAME, from the stack.
-dnl This macro is safe with respect to the nesting.
-dnl Some minimal checking of nesting balance of PAC_VAR_PUSH[POP]VAL()
-dnl is done here.
-dnl
-AC_DEFUN([PAC_VAR_POPVAL],[
-# START of POPVAL
-dnl define local m4-name pac_stk_level.
-AS_VAR_PUSHDEF([pac_stk_level], [pac_stk_$1_level])
-AS_VAR_SET_IF([pac_stk_level],[
-    AS_VAR_IF([pac_stk_level],[-1],[
-        AC_MSG_WARN(["Imbalance of PUSHVAL/POPVAL of $1"])
-    ],[
-        dnl AS_ECHO_N(["POPVAL: pac_stk_level = $pac_stk_level, "])
-        AS_VAR_COPY([$1],[pac_stk_$1_$pac_stk_level])
-        dnl AS_ECHO(["popped_val = $$1"])
-        dnl autoconf < 2.64 does not have AS_VAR_ARITH, so use expr instead.
-        AS_VAR_SET([pac_stk_level], [`expr $pac_stk_level - 1`])
-    ])
-],[
-    AC_MSG_WARN(["Uninitialized PUSHVAL/POPVAL of $1"])
-])
-AS_VAR_POPDEF([pac_stk_level])
-# END of POPVAL
-])
-dnl
-dnl
 dnl
 dnl PAC_COMPILE_IFELSE_LOG is a wrapper around AC_COMPILE_IFELSE with the
 dnl output of ac_compile to a specified logfile instead of AS_MESSAGE_LOG_FD
@@ -143,20 +67,12 @@ dnl
 dnl Replace ">&AS_MESSAGE_LOG_FD" by "> FILE 2>&1" in ac_compile.
 dnl Save a copy of ac_compile on a stack
 dnl which is safe through nested invocations of this macro.
-PAC_VAR_PUSHVAL([ac_compile])
-dnl Modify ac_compile based on the unmodified ac_compile.
-ac_compile="`echo $pac_FirstSavedValueOf_ac_compile | sed -e 's|>.*$|> $1 2>\&1|g'`"
-AC_COMPILE_IFELSE([$2],[
-    ifelse([$3],[],[:],[$3])
-],[
-    ifelse([$4],[],[:],[$4])
+PAC_PUSH_FLAG([ac_compile])
+ac_compile="`echo $ac_compile | sed -e 's|>.*$|> $1 2>\&1|g'`"
+AC_COMPILE_IFELSE([$2],[$3],[$4])
+PAC_POP_FLAG([ac_compile])
 ])
-dnl Restore the original ac_compile from the stack.
-PAC_VAR_POPVAL([ac_compile])
-])
-dnl
-dnl
-dnl
+
 dnl PAC_LINK_IFELSE_LOG is a wrapper around AC_LINK_IFELSE with the
 dnl output of ac_link to a specified logfile instead of AS_MESSAGE_LOG_FD
 dnl
@@ -179,20 +95,12 @@ dnl
 dnl Replace ">&AS_MESSAGE_LOG_FD" by "> FILE 2>&1" in ac_link.
 dnl Save a copy of ac_link on a stack
 dnl which is safe through nested invocations of this macro.
-PAC_VAR_PUSHVAL([ac_link])
-dnl Modify ac_link based on the unmodified ac_link.
-ac_link="`echo $pac_FirstSavedValueOf_ac_link | sed -e 's|>.*$|> $1 2>\&1|g'`"
-dnl
-AC_LINK_IFELSE([$2],[
-    ifelse([$3],[],[:],[$3])
-],[
-    ifelse([$4],[],[:],[$4])
-])
-dnl Restore the original ac_link from the stack.
-PAC_VAR_POPVAL([ac_link])
+PAC_PUSH_FLAG([ac_link])
+ac_link="`echo $ac_link | sed -e 's|>.*$|> $1 2>\&1|g'`"
+AC_LINK_IFELSE([$2],[$3],[$4])
+PAC_POP_FLAG([ac_link])
 ])
-dnl
-dnl
+
 dnl
 dnl PAC_COMPLINK_IFELSE (input1, input2, [action-if-true], [action-if-false])
 dnl
@@ -205,16 +113,10 @@ dnl
 AC_DEFUN([PAC_COMPLINK_IFELSE],[
 AC_COMPILE_IFELSE([$1],[
     PAC_RUNLOG([mv conftest.$OBJEXT pac_conftest.$OBJEXT])
-    PAC_VAR_PUSHVAL([LIBS])
-    LIBS="pac_conftest.$OBJEXT $pac_FirstSavedValueOf_LIBS"
-    AC_LINK_IFELSE([$2],[
-        ifelse([$3],[],[:],[$3])
-    ],[
-        ifelse([$4],[],[:],[$4])
-    ])
-    PAC_VAR_POPVAL([LIBS])
+    PAC_PUSH_FLAG([LIBS])
+    LIBS="pac_conftest.$OBJEXT $LIBS"
+    AC_LINK_IFELSE([$2],[$3],[$4])
+    PAC_POP_FLAG([LIBS])
     rm -f pac_conftest.$OBJEXT
-],[
-    ifelse([$4],[],[:],[$4])
-])
+],[$4])
 ])

http://git.mpich.org/mpich.git/commitdiff/50887ce11493c5aa872e3257b2e406174b59a12b

commit 50887ce11493c5aa872e3257b2e406174b59a12b
Author: Pavan Balaji <balaji at mcs.anl.gov>
Date:   Thu May 2 22:39:50 2013 -0500

    Get rid of PAC_EVAL_IFELSE which is a duplicate of PAC_COMMAND_IFELSE.
    
    Reviewed by gropp.

diff --git a/confdb/aclocal_runlog.m4 b/confdb/aclocal_runlog.m4
index d78c3e6..2fdbfa5 100644
--- a/confdb/aclocal_runlog.m4
+++ b/confdb/aclocal_runlog.m4
@@ -9,6 +9,7 @@ AC_DEFUN([PAC_RUNLOG],[
   ac_status=$?
   AS_ECHO(["$as_me:$LINENO: \$? = $ac_status"]) >&AS_MESSAGE_LOG_FD
   test $ac_status = 0; }])
+
 dnl
 dnl PAC_COMMAND_IFELSE is written to replace AC_TRY_EVAL with added logging
 dnl to config.log, i.e. AC_TRY_EVAL does not log anything to config.log.
@@ -18,8 +19,6 @@ dnl
 dnl PAC_COMMAND_IFELSE(COMMMAND,[ACTION-IF-RUN-OK],[ACTION-IF-RUN-FAIL])
 dnl
 AC_DEFUN([PAC_COMMAND_IFELSE],[
-dnl Should use _AC_DO_TOKENS but use AC_RUN_LOG instead
-dnl because _AC_XX is autoconf's undocumented macro.
 AS_IF([PAC_RUNLOG([$1])],[
     $2
 ],[
@@ -30,25 +29,7 @@ AS_IF([PAC_RUNLOG([$1])],[
     ])
 ])
 ])
-dnl
-dnl
-dnl
-AC_DEFUN([PAC_EVAL_IFELSE],[
-dnl Should use _AC_DO_TOKENS but use AC_RUN_LOG instead
-dnl because _AC_XX is autoconf's undocumented macro.
-AS_IF([PAC_RUNLOG([$$1])],[
-    $2
-],[
-    AS_ECHO(["$as_me: program exited with status $ac_status"]) >&AS_MESSAGE_LOG_FD
-    m4_ifvaln([$3],[
-        (exit $ac_status)
-        $3
-    ])
-])
-])
-dnl
-dnl
-dnl
+
 AC_DEFUN([PAC_RUNLOG_IFELSE],[
 dnl pac_TESTLOG is the internal temporary logfile for this macro.
 pac_TESTLOG="pac_test.log"

http://git.mpich.org/mpich.git/commitdiff/5d6aefa53ac174f8e1c4e3fbbd5fdf0225f48108

commit 5d6aefa53ac174f8e1c4e3fbbd5fdf0225f48108
Author: Pavan Balaji <balaji at mcs.anl.gov>
Date:   Thu May 2 22:34:58 2013 -0500

    PAS_VAR_COPY cleanup.
    
    We no longer need PAS_VAR_COPY since AS_VAR_COPY is present in newer
    autoconf versions.
    
    Reviewed by gropp.

diff --git a/confdb/aclocal_runlog.m4 b/confdb/aclocal_runlog.m4
index 9b85db5..d78c3e6 100644
--- a/confdb/aclocal_runlog.m4
+++ b/confdb/aclocal_runlog.m4
@@ -62,21 +62,8 @@ PAC_COMMAND_IFELSE([$1 > $pac_TESTLOG],[
 ])
 rm -f $pac_TESTLOG
 ])
-dnl
-dnl
-dnl
-dnl PAS_VAR_COPY -  A portable layer that mimics AS_VAR_COPY when it is not
-dnl                 defined as in older autoconf, e.g. 2.63 and older.
-dnl                 This macro is absolutely necessary, because AS_VAR_GET in
-dnl                 some newer autoconf, e.g. 2.64, seems to be totally broken,
-dnl                 or behave very different from older autoconf, i.e. 2.63.
-dnl
-AC_DEFUN([PAS_VAR_COPY],[
-m4_ifdef([AS_VAR_COPY], [AS_VAR_COPY([$1],[$2])], [$1=AS_VAR_GET([$2])])
-])
-dnl
-dnl
-dnl
+
+
 dnl PAC_VAR_PUSHVAL(VARNAME, [LastSavedValue]))
 dnl
 dnl Save the content of the shell variable, VARNAME, onto a stack.
@@ -106,15 +93,15 @@ dnl Save the content of VARNAME, i.e. $VARNAME, onto the stack.
 AS_VAR_SET([pac_stk_$1_$pac_stk_level],[$$1])
 AS_VAR_IF([pac_stk_level], [0], [
     dnl Save the 1st pushed value of VARNAME as pac_FirstSavedValueOf_$VARNAME
-    PAS_VAR_COPY([pac_FirstSavedValueOf_$1],[pac_stk_$1_$pac_stk_level])
+    AS_VAR_COPY([pac_FirstSavedValueOf_$1],[pac_stk_$1_$pac_stk_level])
 ])
 ifelse([$2],[],[
     dnl Save the last pushed value of VARNAME as pac_LastSavedValueOf_$VARNAME
-    PAS_VAR_COPY([pac_LastSavedValueOf_$1],[pac_stk_$1_$pac_stk_level])
+    AS_VAR_COPY([pac_LastSavedValueOf_$1],[pac_stk_$1_$pac_stk_level])
     dnl AS_ECHO(["pac_LastSavedValueOf_$1 = $pac_LastSavedValueOf_$1"])
 ],[
     dnl Save the last pushed value of VARNAME as $2
-    PAS_VAR_COPY([$2],[pac_stk_$1_$pac_stk_level])
+    AS_VAR_COPY([$2],[pac_stk_$1_$pac_stk_level])
     dnl AS_ECHO(["$2 = $$2"])
 ])
 AS_VAR_POPDEF([pac_stk_level])
@@ -139,7 +126,7 @@ AS_VAR_SET_IF([pac_stk_level],[
         AC_MSG_WARN(["Imbalance of PUSHVAL/POPVAL of $1"])
     ],[
         dnl AS_ECHO_N(["POPVAL: pac_stk_level = $pac_stk_level, "])
-        PAS_VAR_COPY([$1],[pac_stk_$1_$pac_stk_level])
+        AS_VAR_COPY([$1],[pac_stk_$1_$pac_stk_level])
         dnl AS_ECHO(["popped_val = $$1"])
         dnl autoconf < 2.64 does not have AS_VAR_ARITH, so use expr instead.
         AS_VAR_SET([pac_stk_level], [`expr $pac_stk_level - 1`])

-----------------------------------------------------------------------

Summary of changes:
 confdb/aclocal_runlog.m4 |  166 +++++-----------------------------------------
 1 files changed, 18 insertions(+), 148 deletions(-)


hooks/post-receive
-- 
MPICH primary repository


More information about the commits mailing list