[SyneRBI/SIRF] CMake] add STIR_REGISTRIES to Reg executables (PR #1268)

0 views
Skip to first unread message

Kris Thielemans

unread,
Jun 19, 2024, 1:54:17 PM (10 days ago) Jun 19
to SyneRBI/SIRF, Subscribed

I experienced linking problems with STIR (probably related to its use of "interdependent" libraries.
Adding STIR_REGISTRIES as dependency fixed it (and shouldn't do any harm).

To do that, I had to interchange the order of the ADD_SUBDIRECTORY statements (i.e. include STIR before Reg).

Note that Actions will likely fail until #1267 is fixed.


You can view, comment on, or merge this pull request online at:

  https://github.com/SyneRBI/SIRF/pull/1268

Commit Summary

  • fa63a99 CMake] add STIR_REGISTRIES to Reg executables

File Changes

(3 files)

Patch Links:


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <SyneRBI/SIRF/pull/1268@github.com>

Evgueni Ovtchinnikov

unread,
Jun 21, 2024, 5:52:28 AM (8 days ago) Jun 21
to SyneRBI/SIRF, Subscribed

@evgueni-ovtchinnikov approved this pull request.


Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <SyneRBI/SIRF/pull/1268/review/2132277885@github.com>

Kris Thielemans

unread,
Jun 25, 2024, 9:04:28 PM (4 days ago) Jun 25
to SyneRBI/SIRF, Subscribed

Merged #1268 into master.


Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <SyneRBI/SIRF/pull/1268/issue_event/13289750040@github.com>

Reply all
Reply to author
Forward
0 new messages