Fix CMake builds on Apple that use the Ninja generator#5769
Fix CMake builds on Apple that use the Ninja generator#5769martin-frbg merged 2 commits intoOpenMathLib:developfrom
Conversation
|
Note that this doesn't actually fix the issue, because the issue is not the won't work, because no But the thing is: because Ninja can handle long command lines directly, the static linking commands don't need to be changed regardless here, so the whole workaround is completely unnecessary. That's why I suggested in #5768 to simply replace the entire I just tested your variant of the patch and it fails the same as if that change wasn't there. |
|
Ah, I see. Didn't want to exclude the whole section as there appear to be unrelated settings in it, but didn't take a closer look |
|
As far as I can tell the entire section is just for the workaround, and it's just that complicated because basically the whole linking logic inside CMake for up to two libraries (the static one and potentially the shared one) has to be written by hand to make the workaround work properly. I can't test this on [edit] other [/edit] Macs, but at least on the M1 I have access to simply replacing the entire if (as I suggested) and having macOS use the same library linking logic as the other platforms "just works" for me. |
|
(But irrespective of this, many thanks for the very quick response to this. 😄) |
fixes #5768 as suggested there