Skip to content

fix use of fftw3x_cdft library for old/new Intel MKL versions#579

Merged
boegel merged 3 commits intoeasybuilders:developfrom
boegel:intel_fftw
Mar 29, 2013
Merged

fix use of fftw3x_cdft library for old/new Intel MKL versions#579
boegel merged 3 commits intoeasybuilders:developfrom
boegel:intel_fftw

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Mar 28, 2013

No description provided.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 28, 2013

Without this, builds with ictce/3.2.2.u3 or ictce/5.2.0 are broken.

@stdweird
Copy link
Copy Markdown
Contributor

@boegel looks ok

boegel added a commit that referenced this pull request Mar 29, 2013
fix use of fftw3x_cdft library for old/new Intel MKL versions
@boegel boegel merged commit ee0ccc0 into easybuilders:develop Mar 29, 2013
@boegel boegel deleted the intel_fftw branch March 29, 2013 08:04
@fgeorgatos
Copy link
Copy Markdown
Contributor

Hi Ken,

Without this, builds with ictce/3.2.2.u3 or ictce/5.2.0 are broken.

that last one is a good catch. Can you elaborate?
Does it affect also 5.1.1 and other toolchains or what?
I have to admit I was surprised for 5.2.0; it wouldn't affect easily the bioinformatics packages but many other applications would suffer (notably for the fellows who are busy in material science and affiliated domains).

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 29, 2013

@fgeorgatos: The reason this broke is because of #518, i.e. we're a lot stricter now w.r.t. setting $LIBFFT, i.e. we only set it if the required libraries are actually there.
Before, the $LIBFFT was broken for ictce/3* and ictce/5*, but nobody noticed because $LIBFFT was not used by any of the builds using those toolchains.

@fgeorgatos
Copy link
Copy Markdown
Contributor

So, "the FFTW libs they provide are different", as per #easybuild discussion among the imkl APIs/ictce toolchains;

this now triggers me to provide the grand Pull Req. for ictce 5.2.0, since the chances of success have increased :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants