tags 592892 + patch
thanks
* Daniel Schepler | 2010-08-13 10:25:31 [-0700]:
>Source: openmpi
>Version: 1.4.2-3
>Severity: serious
>
>../../../../ompi/datatype/dt_module.c:177: error: expected expression before
>')' token
In Line 177 OMPI_ALIGNMENT_FORTRAN_LOGICAL1 is not really set, it is:
/* Size of Fortran 77 LOGICAL*1 */
#define OMPI_SIZEOF_FORTRAN_LOGICAL1
which is set by OMPI_F77_CHECK defined in config/f77_check.m4. By now we
are in the middle of the hell. Lets go further.
That macro has the C default for $ofc_type_alignment and replace it with
the Fortran value in the macro OMPI_F77_GET_ALIGNMENT()
config/f77_get_alignment.m4.
The value should be set in third last line
| $2=AS_VAR_GET([type_var])
It was transformed earlier into
| ofc_type_alignment=$ompi_cv_f77_alignment_LOGICALp1
but become something totally differnt,
|ofc_type_alignment=`eval 'as_val=${'type_var'};$as_echo "$as_val"'`
After carefully replacing it with AS_VAR_COPY we get
| eval ofc_type_alignment=\$$as_type_var
which does the job for now. Puh. Now out quick!
Sebastian
>From a3d1266104a98d5f7be6159582017804871df034 Mon Sep 17 00:00:00 2001
From: Sebastian Andrzej Siewior <bige...@linutronix.de>
Date: Wed, 18 Aug 2010 16:33:23 +0200
Subject: [PATCH] Add fix for #592892
Signed-off-by: Sebastian Andrzej Siewior <bige...@linutronix.de>
---
debian/patches/fix_ftbfs_bts_592892 | 30 ++++++++++++++++++++++++++++++
debian/patches/series | 1 +
2 files changed, 31 insertions(+), 0 deletions(-)
create mode 100644 debian/patches/fix_ftbfs_bts_592892
diff --git a/debian/patches/fix_ftbfs_bts_592892
b/debian/patches/fix_ftbfs_bts_592892
new file mode 100644
index 0000000..e29cf55
--- /dev/null
+++ b/debian/patches/fix_ftbfs_bts_592892
@@ -0,0 +1,30 @@
+From 3fa47ccccdc5271e8d522ea708af418c95839dc2 Mon Sep 17 00:00:00 2001
+From: Sebastian Andrzej Siewior <bige...@linutronix.de>
+Date: Wed, 18 Aug 2010 15:51:01 +0200
+Subject: [PATCH] openmpi: get it built with newer autoconf
+
+Reported as Debian bug #592892. AS_VAR_GET missbehaves in newer
+autoconf. Since it is deprecated switch to AS_VAR_COPY which gets things
+done right. Both are cursed.
+
+Signed-off-by: Sebastian Andrzej Siewior <bige...@linutronix.de>
+---
+ config/f77_get_alignment.m4 | 2 +-
+ 1 files changed, 1 insertions(+), 1 deletions(-)
+
+diff --git a/config/f77_get_alignment.m4 b/config/f77_get_alignment.m4
+index aa3d318..ba1fb96 100644
+--- a/config/f77_get_alignment.m4
++++ b/config/f77_get_alignment.m4
+@@ -90,7 +90,7 @@ EOF
+ AC_MSG_ERROR([Could not determine alignment of $1])])])
+ rm -rf conftest*])
+
+- $2=AS_VAR_GET([type_var])
++ AS_VAR_COPY([$2], [type_var])
+ AS_VAR_POPDEF([type_var])dnl
+ OMPI_VAR_SCOPE_POP
+ ])
+--
+1.5.6.5
+
diff --git a/debian/patches/series b/debian/patches/series
index dee3552..acca808 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -4,3 +4,4 @@ manpage-errors-macro
build_hurd
manpage-spelling-errors
build_kfreebsd
+fix_ftbfs_bts_592892
--
1.5.6.5
