Um -- duh. You're right; I didn't pay attention to the fact that he
was patching the tarball, not the raw source.
On Jan 24, 2008, at 10:30 PM, Ralf Wildenhues wrote:
> 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
>>> 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.
>>> 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
>>> @@ -53,7 +53,8 @@
>>> t print
>>> : 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