Skip to content

[UR] Ensure mock target depends on UR_EXTERNAL_DEPENDENCIES #18844

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 1 commit into from
Jun 11, 2025

Conversation

kbenzie
Copy link
Contributor

@kbenzie kbenzie commented Jun 6, 2025

Firstly, rename ur_mock_headers to ur_mock as this is a shared library not header only library.

Secondly, when part of a subbuild, ensure that the ur_mock target depends on the specified UR_EXTERNAL_DEPENDENCIES, this is handled by using add_ur_library() in place of add_library().

Firstly, rename `ur_mock_headers` to `ur_mock` as this is a shared
library not header only library.

Secondly, when part of a subbuild, ensure that the `ur_mock` target
depends on the specified `UR_EXTERNAL_DEPENDENCIES`, this is handled by
using `add_ur_library()` in place of `add_library()`.
@kbenzie kbenzie temporarily deployed to WindowsCILock June 6, 2025 13:07 — with GitHub Actions Inactive
@kbenzie kbenzie temporarily deployed to WindowsCILock June 6, 2025 13:07 — with GitHub Actions Inactive
@kbenzie kbenzie merged commit 5cb4647 into intel:sycl Jun 11, 2025
43 of 45 checks passed
@kbenzie kbenzie deleted the benie/ur-mock-header-dep branch June 11, 2025 12:29
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