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

0 просмотров
Перейти к первому непрочитанному сообщению

Kris Thielemans

не прочитано,
27 сент. 2017 г., 03:36:4427.09.2017
– 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

не прочитано,
27 сент. 2017 г., 03:56:4327.09.2017
– 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

не прочитано,
15 февр. 2019 г., 05:56:1715.02.2019
– CCPPETMR/SIRF-SuperBuild, Subscribed

Closed #43.

Kris Thielemans

не прочитано,
15 февр. 2019 г., 05:56:1815.02.2019
– 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.

Ответить всем
Отправить сообщение автору
Переслать
0 новых сообщений