MinGW: Do not include gomp

[lofidevops]

Steps to reproduce:

Build on a MinGW platform

What should happen:

Build successful.

What happens instead:

Fails due to dependency on GOMP/libgomp.

Notes:

MinGW is unusual in that it is both a GCC platform and a Windows platform. Unlike other GCC platforms, it is not guaranteed that GOMP is installed (afaik no MinGW installation currently includes GOMP).

Thus the CMake logic to include GOMP should be something like:

if (platform == GCC) and (platform != MinGW):
    include gomp

Workaround:

lofidevops@1e13351

Edit CMake files so that gomp is never included. This is obviously not a long-term solution.

[simonlynen]

We would just need to change the cmake to also check for mingw. Could you
open a pull-request that adds this check to cmake? Since I dont have a
setup to test this.

No further changes to the code should be needed.

Thanks

On Mon, Aug 3, 2015, 16:59 d❤vid seaward notifications@github.com wrote:

Steps to reproduce:

Build on a MinGW platform

What should happen:

Build successful.

What happens instead:

Fails due to dependency on GOMP/libgomp.

Notes:

MinGW is unusual in that it is both a GCC platform and a Windows platform.
Unlike other GCC platforms, it is not guaranteed that GOMP is installed
(afaik no MinGW installation currently includes GOMP).

Thus the CMake logic to include GOMP should be something like:

if (platform == GCC) and (platform != MinGW):
include gomp

Workaround:

kwill@1e13351
lofidevops@1e13351

Edit CMake files so that gomp is never included. This is obviously not a
long-term solution.

—
Reply to this email directly or view it on GitHub
#37.

[lofidevops]

See #66