Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add mkl to fedora #25

Merged
merged 2 commits into from
Dec 1, 2023
Merged

Add mkl to fedora #25

merged 2 commits into from
Dec 1, 2023

Conversation

streeve
Copy link
Member

@streeve streeve commented Dec 1, 2023

No longer comes by default? Breaking in ECP-copa/Cabana#720

@streeve streeve requested a review from junghans December 1, 2023 17:03
@streeve streeve force-pushed the fixup_mkl branch 2 times, most recently from 7c36905 to 044ef5f Compare December 1, 2023 17:37
@streeve
Copy link
Member Author

streeve commented Dec 1, 2023

@junghans not sure what I'm doing wrong here, but this isn't installing MKL for intel as I'm intending

@dalg24
Copy link
Collaborator

dalg24 commented Dec 1, 2023

@masterleinad please review

@streeve
Copy link
Member Author

streeve commented Dec 1, 2023

isn't installing MKL for intel as I'm intending

Fixed with the strategy from kokkos/ci-containers

run: |
echo "args<<EOF" >> $GITHUB_OUTPUT
echo ${{ matrix.config.build_args }} | sed 's/,/\n/g' >> $GITHUB_OUTPUT
echo "EOF" >> $GITHUB_OUTPUT
Copy link
Member

Choose a reason for hiding this comment

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

Thanks for stealing my code ;-)

@junghans
Copy link
Member

junghans commented Dec 1, 2023

@masterleinad please review

This is not kokkos, so nothing for @masterleinad to do here.

@junghans junghans merged commit e0086a5 into master Dec 1, 2023
10 checks passed
@junghans junghans deleted the fixup_mkl branch December 1, 2023 19:04
@dalg24
Copy link
Collaborator

dalg24 commented Dec 1, 2023

@masterleinad please review

This is not kokkos, so nothing for @masterleinad to do here.

My bad :)

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.

3 participants