Re: [OMPI devel] non-portable test operator in configure

2012-02-20 Thread Paul H. Hargrove
For those keeping score at home, that should have said "/usr/ucb" instead of "/usr/ucb/bin". I make mistakes too (as Ralph's observation of breakage w/ r25966 shows quite clearly). -Paul On 2/20/2012 2:37 PM, Paul H. Hargrove wrote: Short version: The "expr: Paren problem" comes from having /

Re: [OMPI devel] non-portable test operator in configure

2012-02-20 Thread Paul H. Hargrove
Short version: The "expr: Paren problem" comes from having /usr/ucb/bin ahead of /usr/bin in one's $PATH. So, I needed to fix my $PATH. Long version: This error is coming from configure's own argument parsing logic when the ROMIO sub-configure is invoked. The issue appears to be that the expr

Re: [OMPI devel] non-portable test operator in configure

2012-02-20 Thread Jeffrey Squyres
Committed -- thanks. Confirmed that it's not in v1.5, too. Just for good measure. On Feb 20, 2012, at 4:12 PM, Paul H. Hargrove wrote: > Not what I had expected to find, but a pretty simple fix (missing line > continuation): > > Index: orte/mca/ess/alps/configure.m4 > ===

Re: [OMPI devel] non-portable test operator in configure

2012-02-20 Thread Paul H. Hargrove
Argh!! I am now trying to track down "expr: Paren problem" on Solaris. The dash shell on Linux doesn't reproduce this one, unfortunately. -Paul On 2/20/2012 1:12 PM, Paul H. Hargrove wrote: I'll report back ASAP on my slowlaris10 results. -- Paul H. Hargrove phhargr..

Re: [OMPI devel] non-portable test operator in configure

2012-02-20 Thread Paul H. Hargrove
Not what I had expected to find, but a pretty simple fix (missing line continuation): Index: orte/mca/ess/alps/configure.m4 === --- orte/mca/ess/alps/configure.m4 (revision 25970) +++ orte/mca/ess/alps/configure.m4 (workin

Re: [OMPI devel] non-portable test operator in configure

2012-02-20 Thread Jeffrey Squyres
Ah, ok. On Feb 20, 2012, at 3:45 PM, Paul H. Hargrove wrote: > Jeff, > > The one in config/ompi_load_platform.m4 was on my original hit-list. > Getting PAST that one shows a new problem that appears NOT to be a "==". > The autoconf manual warns about use of "-a" and "-o" together with variables

Re: [OMPI devel] non-portable test operator in configure

2012-02-20 Thread Paul H. Hargrove
Jeff, The one in config/ompi_load_platform.m4 was on my original hit-list. Getting PAST that one shows a new problem that appears NOT to be a "==". The autoconf manual warns about use of "-a" and "-o" together with variables that may expand to the empty string, and I suspect that is the new pro

Re: [OMPI devel] non-portable test operator in configure

2012-02-20 Thread Jeffrey Squyres
grep == configure | grep test only shows one more. I found it in config/ompi_load_platform.m4 and fixed it on the trunk. On Feb 20, 2012, at 3:38 PM, Paul H. Hargrove wrote: > I am afraid that with the $with_platform instance fixed, configure on Solaris > 10 gets far enough to find another p

Re: [OMPI devel] non-portable test operator in configure

2012-02-20 Thread Paul H. Hargrove
I am afraid that with the $with_platform instance fixed, configure on Solaris 10 gets far enough to find another problem. I'll provide a patch once I've tracked this down. Sigh. FYI: One can root out bashisms by using the "dash" shell on a Debian or Ubuntu system: $ env CONFIG_SHELL=dash dash

Re: [OMPI devel] non-portable test operator in configure

2012-02-20 Thread Jeffrey Squyres
Fixed -- thanks! On Feb 20, 2012, at 4:11 AM, Paul H. Hargrove wrote: > Please note that "==" is NOT a portable binary operator for the "test" > utility. > It is supported only by the bash built-in version of "test". > The correct operator is a simple "=". > > The following appear in the svn tr