#1 FTBFS fix
Merged 2 years ago by tdawson. Opened 2 years ago by merlinm.

file modified
+13 -1

@@ -14,7 +14,7 @@

  

  Name:           fftw

  Version:        3.3.5

- Release:        7%{?dist}

+ Release:        8%{?dist}

  Summary:        A Fast Fourier Transform library

  Group:          System Environment/Libraries

  License:        GPLv2+

@@ -407,6 +407,13 @@

   make %{?_smp_mflags} -C $ver check

  done

  # MPI

+ %if %{with openmpi}

+ %ifarch %{ix86}

+ # disable Open MPI's vader byte transfer layer while running tests on 32-bit x86 platforms

+ # as it is known to be troublesome <https://github.com/open-mpi/ompi/issues/4260>

+ export OMPI_MCA_btl="^vader"

+ %endif

+ %endif

  for mpi in %{mpi_list}

  do

   module load mpi/${mpi}-%{_arch}

@@ -543,6 +550,11 @@

  %endif

  

  %changelog

+ * Tue Nov 28 2017 Merlin Mathesius <mmathesi@redhat.com> - 3.3.5-8

+ - Disable Open MPI's vader byte transfer layer while running tests on 32-bit x86 platforms

+   as it is known to be troublesome <https://github.com/open-mpi/ompi/issues/4260>.

+   This resolves FTBFS issue (BZ#1518038).

+ 

  * Wed Aug 02 2017 Fedora Release Engineering <releng@fedoraproject.org> - 3.3.5-7

  - Rebuilt for https://fedoraproject.org/wiki/Fedora_27_Binutils_Mass_Rebuild

  

This PR disables Open MPI's vader byte transfer layer while running tests on 32-bit x86 platforms as it is known to be troublesome (https://github.com/open-mpi/ompi/issues/4260).

This resolves FTBFS issue BZ#1518038.

Successful Rawhide scratch build of this patch: https://koji.fedoraproject.org/koji/taskinfo?taskID=23444109

Please consider merging this PR--or implementing an alternate fix--so that fftw can build successfully again.

Koji has purged all artifacts from my earlier scratch build, so here's a link to a fresh one: https://koji.fedoraproject.org/koji/taskinfo?taskID=24129915

The current version of the package is still FTBFS, as show by this scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=24129893

Please merge this PR or implement an alternate resolution.

LGTM
Thank you for doing this.

Pull-Request has been merged by tdawson

2 years ago
Metadata