[CCPPETMR/SIRF-SuperBuild] Handle different options for different versions of dependencies (#43)

0 views
Skip to first unread message

Kris Thielemans

unread,
Sep 27, 2017, 3:36:44 AM9/27/17
to CCPPETMR/SIRF-SuperBuild, Subscribed

With the version tracking, we can use the SuperBuild to use different versions of SIRF, STIR etc. However, some CMake options will be version specific. Example, CCPPETMR/SIRF#41 will need Matlab_ROOT_DIR as opposed to MATLAB_ROOT. The SuperBuild needs to know about this, but how would it?

In principle, we could keep a list of commits where the option changed, and then check if it's an ancester of the desired TAG using something like

git rev-list ${COMMIT_HASH_WITH_NEW_OPTION}| grep $(git rev-parse ${${SIRF_GIT_TAG})

(after CMake-ifying this) but keeping the list is error-prone and tedious.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Kris Thielemans

unread,
Sep 27, 2017, 3:56:43 AM9/27/17
to CCPPETMR/SIRF-SuperBuild, Subscribed

the rev-list is essentially impossible in the SuperBuild anyway as we have don't explicitly clone here, but rely on ExternProject_Add

Kris Thielemans

unread,
Feb 15, 2019, 5:56:17 AM2/15/19
to CCPPETMR/SIRF-SuperBuild, Subscribed

Closed #43.

Kris Thielemans

unread,
Feb 15, 2019, 5:56:18 AM2/15/19
to CCPPETMR/SIRF-SuperBuild, Subscribed

This seems not necessary since we've introduced version_config.cmake. If a user switches to an old STIR_TAG (or other dependency) and the options need to change, it's not really for us to handle that properly I think. So closing.

Reply all
Reply to author
Forward
0 new messages