-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
OV file path migration to std lib interface #26595
base: master
Are you sure you want to change the base?
Conversation
…te_files, join_paths
src/common/util/CMakeLists.txt
Outdated
@@ -37,7 +37,9 @@ add_library(${TARGET_NAME} STATIC ${LIBRARY_SRC} ${PUBLIC_HEADERS}) | |||
add_library(openvino::util ALIAS ${TARGET_NAME}) | |||
set_target_properties(${TARGET_NAME} PROPERTIES EXPORT_NAME util) | |||
|
|||
target_link_libraries(${TARGET_NAME} PRIVATE ${CMAKE_DL_LIBS} PUBLIC openvino::pugixml) | |||
# TODO: make lib linkage portable e.g.: openvino/src/bindings/python/thirdparty/pybind11/tests/CMakeLists.txt | |||
target_link_libraries(${TARGET_NAME} PRIVATE ${CMAKE_DL_LIBS} PUBLIC openvino::pugixml stdc++fs ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does it required new C++ standard ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, the feature will be C++11 compatible. Where possible I plan to use experimental part of std filesystem lib and in reference tests for custom implementation. I would like the interface to be as close to std as possible to smoothly adapt future fixups to the newer CPP standard.
Details:
Tickets: