kaistshadow / blockchain-sim

Scalable blockchain simulator/emulator running on shadow simulator
MIT License
9 stars 1 forks source link

[bug-fix] Update of Monero CMake script is not compatible with CMake lower than 3.17 #295

Closed ygnkim closed 3 years ago

ygnkim commented 3 years ago

Monero를 shared object로 사용할 수 있도록 include directory에 대한 interface를 추가해주는 아래의 코드가 CMake version 3.17보다 아래에서는 제대로 동작안함

https://github.com/kaistshadow/monero/blob/fb28ddbe1ccef0f9a84bb634b4f1b0f02aa18717/src/CMakeLists.txt#L96-L98

이는 CMAKE_CURRENT_FUNCTION_LIST_DIR 변수가 CMake 3.17이상에서만 지원되기 때문이다.

따라서 이에 대한 수정이 필요함

ygnkim commented 3 years ago

daemon plugin(shadow_monero_daemon)이 만들어진 이후 CMAKE_CURRENT_SOURCE_DIR으로 INTERFACE 설정하도록 바꿈

https://github.com/kaistshadow/monero/blob/179de2f3d925cdca12da571b96aaf890e6bc269b/src/daemon/CMakeLists.txt#L133-L135