Hi Jeff, * Jeff Squyres wrote on Fri, Jan 25, 2008 at 01:00:23AM CET: > On Jan 19, 2008, at 3:38 AM, Ralf Wildenhues wrote: > > >This patch will not work with Solaris sh. Please consider using this > >instead, it should avoid the issues with both shells. (Untested.) > > Ralf -- the original patch changed both ompi_get_version.m4 and > ompi_get_version.sh; your patch only changes the m4. > > Is there equivalent magic for the sh file as well?
Sure. Just run autogen.sh, it will regenerate the sh file. Sorry for not mentioning that right away. Cheers, Ralf > >Index: config/ompi_get_version.m4 > >=================================================================== > >--- config/ompi_get_version.m4 (Revision 17165) > >+++ config/ompi_get_version.m4 (Arbeitskopie) > >@@ -41,7 +41,7 @@ > > > > dnl quote eval to suppress macro expansion with non-GNU m4 > > if test -f "$1"; then > >- [eval] "`sed -n \"\ > >+ ompi_vers=`sed -n " > > t clear > > : clear > > s/^major/$2_MAJOR_VERSION/ > >@@ -53,7 +53,8 @@ > > t print > > b > > : print > >- p\" < \"\$1\"`" > >+ p" < "$1"` > >+ [eval] "$ompi_vers" > > > > # Only print release version if it isn't 0 > > if test $$2_RELEASE_VERSION -ne 0 ; then
