Skip to content

WIP: add oneapi compilers#19033

Closed
rscohn2 wants to merge 2 commits intospack:developfrom
rscohn2:dev/oneapi
Closed

WIP: add oneapi compilers#19033
rscohn2 wants to merge 2 commits intospack:developfrom
rscohn2:dev/oneapi

Conversation

@rscohn2
Copy link
Copy Markdown
Member

@rscohn2 rscohn2 commented Sep 28, 2020

Add support for installing icc/icpc/ifort from oneapi installer
Differences versus PSXE:

  • no license file needed
  • directory structure and environment variable setup has changed
  • installer does not need a config file, you can pass the required arguments on the command line

I am new to spack, so not sure if I did things the best way. Feel free to use this as a guide on what has to change and redo if you prefer.

@rscohn2 rscohn2 changed the title add oneapi compilers WIP: add oneapi compilers Sep 29, 2020
@rscohn2
Copy link
Copy Markdown
Member Author

rscohn2 commented Sep 29, 2020

Want to make the version more flexible. Currently, version must match the version identifier in the downloaded file name. Also, oneAPI is beta software so oneapi version should not be default.

@tgamblin tgamblin self-requested a review October 2, 2020 06:39
@tgamblin tgamblin self-assigned this Oct 2, 2020
@rscohn2
Copy link
Copy Markdown
Member Author

rscohn2 commented Oct 6, 2020

@tgamblin : I have been reconsidering this patch. So much has changed between parallel studio and oneapi. It might be better to make a clean break instead of doing the minimal changes to make it work again. It would help me if you could recommend someone who could answer questions about the current code.

@rscohn2 rscohn2 closed this Oct 6, 2020
@tgamblin tgamblin requested a review from scheibelp October 10, 2020 00:01
@scheibelp scheibelp mentioned this pull request Oct 24, 2020
1 task
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