Skip to content

Make sure correct include directory is used for FlexiBLAS toolchain component#3875

Merged
boegel merged 1 commit intoeasybuilders:developfrom
ocaisa:flexiblas_include
Oct 27, 2021
Merged

Make sure correct include directory is used for FlexiBLAS toolchain component#3875
boegel merged 1 commit intoeasybuilders:developfrom
ocaisa:flexiblas_include

Conversation

@ocaisa
Copy link
Copy Markdown
Member

@ocaisa ocaisa commented Oct 27, 2021

Addresses part of #3868

@ocaisa
Copy link
Copy Markdown
Member Author

ocaisa commented Oct 27, 2021

Testing:

[ocaisa@login1 ~]$ eb -x zarr-2.10.1-foss-2021a.eb | grep INC_DIR
  export BLAS_INC_DIR='/home/ocaisa/.local/easybuild/software/FlexiBLAS/3.0.4-GCC-10.3.0/include/flexiblas'
  export BLAS_LAPACK_INC_DIR='/home/ocaisa/.local/easybuild/software/FlexiBLAS/3.0.4-GCC-10.3.0/include/flexiblas'
  export FFTW_INC_DIR='/project/boegelbot/Rocky8/haswell/software/FFTW/3.3.9-gompi-2021a/include'
  export FFT_INC_DIR='/project/boegelbot/Rocky8/haswell/software/FFTW/3.3.9-gompi-2021a/include'
  export LAPACK_INC_DIR='/home/ocaisa/.local/easybuild/software/FlexiBLAS/3.0.4-GCC-10.3.0/include/flexiblas'
  export MPI_INC_DIR='/project/boegelbot/Rocky8/haswell/software/OpenMPI/4.1.1-GCC-10.3.0/include'
  export SCALAPACK_INC_DIR=''

@boegel boegel added the bug fix label Oct 27, 2021
@boegel boegel added this to the 4.5.0 (next release) milestone Oct 27, 2021
boegel
boegel approved these changes Oct 27, 2021
View reviewed changes
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@ocaisa ocaisa mentioned this pull request Oct 27, 2021
Closed
@boegel boegel merged commit a136f57 into easybuilders:develop Oct 27, 2021
@ocaisa ocaisa deleted the flexiblas_include branch October 27, 2021 12:09
@boegel boegel changed the title Make sure correct include directory is used for FlexiBLAS Make sure correct include directory is used for FlexiBLAS toolchain component Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.5.0

Development

Successfully merging this pull request may close these issues.

2 participants

@ocaisa @boegel