Skip to content

Commit 990b5d2

Browse files
akosthekissyichoi
authored andcommitted
Fix include directories of jerry-ext (#2464)
The distinction between public and private include directories was not maintained and the install paths got confusing. JerryScript-DCO-1.0-Signed-off-by: Akos Kiss [email protected]
1 parent b31e430 commit 990b5d2

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

jerry-ext/CMakeLists.txt

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@ set(JERRY_EXT_NAME jerry-ext)
1717
project (${JERRY_EXT_NAME} C)
1818

1919
# Include directories
20-
set(INCLUDE_EXT "${CMAKE_CURRENT_SOURCE_DIR}/include" "${CMAKE_CURRENT_SOURCE_DIR}/common")
20+
set(INCLUDE_EXT_PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/include")
21+
set(INCLUDE_EXT_PRIVATE "${CMAKE_CURRENT_SOURCE_DIR}/common")
2122

2223
if(FEATURE_INIT_FINI)
2324
set(DEFINES_EXT ${DEFINES_EXT} ENABLE_INIT_FINI)
@@ -39,9 +40,10 @@ set(SOURCE_EXT
3940

4041
add_library(${JERRY_EXT_NAME} ${SOURCE_EXT})
4142

42-
target_include_directories(${JERRY_EXT_NAME} PUBLIC ${INCLUDE_EXT})
43+
target_include_directories(${JERRY_EXT_NAME} PUBLIC ${INCLUDE_EXT_PUBLIC})
44+
target_include_directories(${JERRY_EXT_NAME} PRIVATE ${INCLUDE_EXT_PRIVATE})
4345
target_compile_definitions(${JERRY_EXT_NAME} PUBLIC ${DEFINES_EXT})
4446
target_link_libraries(${JERRY_EXT_NAME} jerry-core)
4547

4648
install(TARGETS ${JERRY_EXT_NAME} DESTINATION lib)
47-
install(DIRECTORY ${INCLUDE_EXT}/ DESTINATION include)
49+
install(DIRECTORY ${INCLUDE_EXT_PUBLIC}/ DESTINATION include)

0 commit comments

Comments
 (0)