Skip to content

added c-interface (lapacke) support - #699

Open
DirkdeDraak wants to merge 1 commit into
easybuilders:developfrom
DirkdeDraak:lapack-with-c-interface
Open

added c-interface (lapacke) support#699
DirkdeDraak wants to merge 1 commit into
easybuilders:developfrom
DirkdeDraak:lapack-with-c-interface

Conversation

@DirkdeDraak

Copy link
Copy Markdown

No description provided.

@hpcugentbot

Copy link
Copy Markdown

Automatic reply from Jenkins: Can I test this?

@boegel

boegel commented Sep 22, 2015

Copy link
Copy Markdown
Member

Jenkins: ok to test

@boegel

boegel commented Sep 22, 2015

Copy link
Copy Markdown
Member

looks fine, but we'll need to make sure this works with all existing LAPACK easyconfig we have (there should be no regressions)

I'll take a look at that.

@boegel boegel added this to the v2.4.0 milestone Sep 22, 2015
@hpcugentbot

Copy link
Copy Markdown

Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1218/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how about just checking the version? this should be there in LAPACK 3.4.0 and up

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see https://github.com/DirkdeDraak/easybuild-easyblocks/pull/1, if you merge my PR to your branch, it'll be included here

@boegel

boegel commented Sep 22, 2015

Copy link
Copy Markdown
Member

@DirkdeDraak: hmm, this doesn't work as expected...

It results in running this:

make -j 10  OPTS="$FFLAGS -m64" NOOPT="-fPIC -m64 -O0"  BLASLIB=""  lib  lapackelib

which yields:

make: *** No rule to make

because you can't specify two targets (lib and lapackelib) at the same time...

@boegel boegel removed this from the v2.4.0 milestone Sep 22, 2015
@boegel boegel modified the milestone: v2.7.0 Jan 23, 2016
@boegel boegel modified the milestones: v2.7.0, v2.x Mar 9, 2016
@boegel boegel modified the milestones: 3.x, 4.x Feb 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants