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 minimal install script #819

Merged
merged 1 commit into from
Sep 1, 2022
Merged

Conversation

illuhad
Copy link
Collaborator

@illuhad illuhad commented Aug 31, 2022

This can be useful for training courses etc. when people just need to get a SYCL implementation asap with minimal dependencies.

@illuhad illuhad force-pushed the feature/add-minimal-install-script branch from a47aed8 to 39e829d Compare September 1, 2022 00:03
@illuhad illuhad merged commit 9890265 into develop Sep 1, 2022
@illuhad illuhad deleted the feature/add-minimal-install-script branch September 1, 2022 00:05
@Momellouky
Copy link
Contributor

Hello,
I have tried to run this script on ubuntu 22.04 LTS. I am facing a problem with the following include (#include "clang/AST/ASTContext.h"). It says that No such file or directory.
The complete error message:
fatal error: clang/AST/ASTContext.h: No such file or directory
38 | #include "clang/AST/ASTContext.h"
| ^~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

@illuhad
Copy link
Collaborator Author

illuhad commented Apr 12, 2023

It's possible that, if cmake finds an LLVM installation, the new SSCP compiler is enabled which then might fail if you are lacking a sufficiently new LLVM installation (in your case, probably the llvm -dev packages are missing).

Can you try:

  1. Open the minimal install script
  2. Edit the line where cmake is invoked and add -DWITH_SSCP_COMPILER=OFF
  3. Run the script again

@Momellouky
Copy link
Contributor

Hi.
Yes, you are right. In fact now it works perefectly. It compiles and everything is fine.
Thank you.

@illuhad
Copy link
Collaborator Author

illuhad commented Apr 12, 2023

Thanks for the feedback, I have opened PR #1009 to fix this issue in the script.

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.

None yet

2 participants