Skip to content

[SYCL] Fix dependencies for Doxygen job #1206

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

Merged
merged 3 commits into from
Feb 27, 2020

Conversation

alexbatashev
Copy link
Contributor

configure.py requires Ninja to be installed on build machine, which is
missing from default GitHub Actions Linux environments. Install
ninja-build package before proceeding to CMake configuration.

Signed-off-by: Alexander Batashev [email protected]

configure.py requires Ninja to be installed on build machine, which is
missing from default GitHub Actions Linux environments. Install
ninja-build package before proceeding to CMake configuration.

Signed-off-by: Alexander Batashev <[email protected]>
@bader
Copy link
Contributor

bader commented Feb 27, 2020

I don't think it's enough. Did you test this?

@alexbatashev
Copy link
Contributor Author

I don't think it's enough. Did you test this?

I tested it on my fork, configuration step finishes successfully, which is enough for docs to be generated.

Is there anything particular that bothers you?

@alexbatashev
Copy link
Contributor Author

@bader by the way, we can temporarily change the trigger to pull request. Deployment will fail, but the rest of the steps must proceed, so that we can be sure that this is enough.

Alexander Batashev added 2 commits February 27, 2020 12:28
Signed-off-by: Alexander Batashev <[email protected]>
This reverts commit 55b7f37.

Signed-off-by: Alexander Batashev <[email protected]>
@bader bader merged commit c3aac3a into intel:sycl Feb 27, 2020
@alexbatashev alexbatashev deleted the private/abatashe/fix_gh_pages branch September 17, 2021 06:45
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Feb 23, 2023
…1206)

This commit removes the host run and any assumptions and operations
related to the host device from the tests in SYCL/ESIMD.

Co-authored-by: Sachkov, Alexey <[email protected]>
Signed-off-by: Larsen, Steffen <[email protected]>
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.

2 participants