Skip to content

Fix: add some missing *_LIBRARY_DIRS to target_link_directories() #785

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

Conversation

ldorau
Copy link
Contributor

@ldorau ldorau commented Oct 9, 2024

Description

Fix: add some missing *_LIBRARY_DIRS to target_link_directories():

The following examples:

  • umf_example_ipc_ipcapi_anon_fd
  • umf_example_ipc_ipcapi_shm
  • umf_example_custom_file_provider

require TBB and they have to know TBB_LIBRARY_DIRS.

The umf_example_dram_and_fsdax example requires HWLOC and it has to know LIBHWLOC_LIBRARY_DIRS.

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly

The following examples:
- umf_example_ipc_ipcapi_anon_fd
- umf_example_ipc_ipcapi_shm
- umf_example_custom_file_provider
require TBB and they have to know TBB_LIBRARY_DIRS.

The umf_example_dram_and_fsdax example requires HWLOC
and it has to know LIBHWLOC_LIBRARY_DIRS.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
@ldorau ldorau requested a review from a team as a code owner October 9, 2024 08:05
@ldorau ldorau requested a review from bratpiorka October 9, 2024 08:45
@bratpiorka bratpiorka merged commit db47b9e into oneapi-src:main Oct 9, 2024
72 checks passed
@ldorau ldorau deleted the Fix_add_some_missing_LIBRARY_DIRS_to_target_link_directories branch October 10, 2024 06:07
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