search

Xilinx / dfx-mgr

Other
10 stars 5 forks source link

Stopped installing dfx-mgr-client include files #3

Open lool opened 2 years ago

lool commented 2 years ago

Hi,

ce24786eef2634af28c16d3462aae80edc85a2e0 renamed some cmake PROJECT variables but also caused at least the include files not to be make installed anymore:

-  install (FILES ${CMAKE_CURRENT_BINARY_DIR}/include/${PROJECT_NAME}/${f}
-           DESTINATION include RENAME ${PROJECT_NAME}/${f})
+  # install (FILES ${CMAKE_CURRENT_BINARY_DIR}/include/${PROJECT_NAME}/${f}
+  #          DESTINATION include RENAME ${PROJECT_NAME}/${f})

Seems like a leftover from local builds that shouldn't have been committed?

Best

rajimoha commented 4 months ago

Thanks.. seems to be commented off, will remove this..