Skip to content

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 alexander.batashev@intel.com

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 <alexander.batashev@intel.com>
@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 <alexander.batashev@intel.com>
This reverts commit 55b7f37.

Signed-off-by: Alexander Batashev <alexander.batashev@intel.com>
@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 <alexey.sachkov@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
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