Processed: Re: Bug#853324: bagel: ftbfs with GCC-7

[Debian Bug Tracking System]

Processing control commands:

> reassign -1 libmpich-dev
Bug #853324 [src:bagel] bagel: ftbfs with GCC-7
Bug reassigned from package 'src:bagel' to 'libmpich-dev'.
No longer marked as found in versions bagel/0.0~git20170109-1.
Ignoring request to alter fixed versions of bug #853324 to the same values 
previously set
> reassign 853532 libmpich-dev
Bug #853532 [src:madness] madness: ftbfs with GCC-7
Bug reassigned from package 'src:madness' to 'libmpich-dev'.
No longer marked as found in versions madness/0.10.1~gite4aa500e-10.
Ignoring request to alter fixed versions of bug #853532 to the same values 
previously set
> reassign 853562 libmpich-dev
Bug #853562 [src:mpqc3] mpqc3: ftbfs with GCC-7
Bug reassigned from package 'src:mpqc3' to 'libmpich-dev'.
No longer marked as found in versions mpqc3/0.0~git20170114-4.
Ignoring request to alter fixed versions of bug #853562 to the same values 
previously set
> reassign 853404 libmpich-dev
Bug #853404 [src:form] form: ftbfs with GCC-7
Bug reassigned from package 'src:form' to 'libmpich-dev'.
No longer marked as found in versions form/4.1-1.
Ignoring request to alter fixed versions of bug #853404 to the same values 
previously set
> forcemerge -1 853532 853562 853404
Bug #853324 [libmpich-dev] bagel: ftbfs with GCC-7
Bug #853404 [libmpich-dev] form: ftbfs with GCC-7
Bug #853532 [libmpich-dev] madness: ftbfs with GCC-7
Bug #853562 [libmpich-dev] mpqc3: ftbfs with GCC-7
Merged 853324 853404 853532 853562
> retitle -1 libmpich-dev: drop the gcc version check in mpicxx.h
Bug #853324 [libmpich-dev] bagel: ftbfs with GCC-7
Bug #853404 [libmpich-dev] form: ftbfs with GCC-7
Bug #853532 [libmpich-dev] madness: ftbfs with GCC-7
Bug #853562 [libmpich-dev] mpqc3: ftbfs with GCC-7
Changed Bug title to 'libmpich-dev: drop the gcc version check in mpicxx.h' 
from 'bagel: ftbfs with GCC-7'.
Changed Bug title to 'libmpich-dev: drop the gcc version check in mpicxx.h' 
from 'form: ftbfs with GCC-7'.
Changed Bug title to 'libmpich-dev: drop the gcc version check in mpicxx.h' 
from 'madness: ftbfs with GCC-7'.
Changed Bug title to 'libmpich-dev: drop the gcc version check in mpicxx.h' 
from 'mpqc3: ftbfs with GCC-7'.
> affects -1 src:bagel src:madness src:mpqc3 src:form
Bug #853324 [libmpich-dev] libmpich-dev: drop the gcc version check in mpicxx.h
Bug #853404 [libmpich-dev] libmpich-dev: drop the gcc version check in mpicxx.h
Bug #853532 [libmpich-dev] libmpich-dev: drop the gcc version check in mpicxx.h
Bug #853562 [libmpich-dev] libmpich-dev: drop the gcc version check in mpicxx.h
Added indication that 853324 affects src:bagel, src:madness, src:mpqc3, and 
src:form
Added indication that 853404 affects src:bagel, src:madness, src:mpqc3, and 
src:form
Added indication that 853532 affects src:bagel, src:madness, src:mpqc3, and 
src:form
Added indication that 853562 affects src:bagel, src:madness, src:mpqc3, and 
src:form
> tags -1 patch
Bug #853324 [libmpich-dev] libmpich-dev: drop the gcc version check in mpicxx.h
Bug #853404 [libmpich-dev] libmpich-dev: drop the gcc version check in mpicxx.h
Bug #853532 [libmpich-dev] libmpich-dev: drop the gcc version check in mpicxx.h
Bug #853562 [libmpich-dev] libmpich-dev: drop the gcc version check in mpicxx.h
Added tag(s) patch.
Added tag(s) patch.
Added tag(s) patch.
Added tag(s) patch.

-- 
853324: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=853324
853404: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=853404
853532: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=853532
853562: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=853562
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems

Bug#853324: bagel: ftbfs with GCC-7

[Adrian Bunk]

Control: reassign -1 libmpich-dev
Control: reassign 853532 libmpich-dev
Control: reassign 853562 libmpich-dev
Control: reassign 853404 libmpich-dev
Control: forcemerge -1 853532 853562 853404
Control: retitle -1 libmpich-dev: drop the gcc version check in mpicxx.h
Control: affects -1 src:bagel src:madness src:mpqc3 src:form
Control: tags -1 patch

On Tue, Jan 31, 2017 at 09:29:53AM +, Matthias Klose wrote:
>...
> /usr/include/mpich/mpicxx.h:21:4: error: #error 'Please use the same version 
> of GCC and g++ for compiling MPICH and user MPI programs'
>  #  error 'Please use the same version of GCC and g++ for compiling MPICH and 
> user MPI programs'
>...

The recent binNMU has "fixed" that, but we will see the same again with 
gcc 8 and the current check really doesn't make much sense:
# if __GNUC__ > 7
#  error 'Please use the same version of GCC and g++ for compiling MPICH and 
user MPI programs'
# endif

Attached is a patch to drop the whole check.

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

Description: Remove the gcc version check
Author: Adrian Bunk 
Bug-Debian: https://bugs.debian.org/811210
Bug-Debian: https://bugs.debian.org/853324

--- mpich-3.2.orig/src/binding/cxx/mpicxx.h.in
+++ mpich-3.2/src/binding/cxx/mpicxx.h.in
@@ -12,18 +12,6 @@
 #error "You cannot define MPI; that name is reserved for the MPI namespace"
 #endif
 
-// Check for incompatible GCC versions
-// GCC (specifically) g++ changed the calling convention
-// between 3.2.3 and 3.4.3 (!!)  Normally such changes
-// should only occur at major releases (e.g., version 3 to 4)
-#ifdef __GNUC__ 
-# if __GNUC__ >= @GNUCXX_VERSION@ 
-#  if __GNUC_MINOR__ > 2 && @GNUCXX_MINORVERSION@ == 2 
-#  error 'Please use the same version of GCC and g++ for compiling MPICH and user MPI programs'
-#  endif
-# endif 
-#endif
-
 /* 
  * Because the API is defined, some methods have parameters that are 
  * not used.  The following definition allows us to suppress warnings