Fixed separator in CMAKE_INSTALL_RPATH

[serguei-k]

Description

Running commands on OSX when AliceVision was installed into a custom prefix was failing as the command could not find the linked dynamic libraries (because the LC_RPATH was set to path @loader_path/../lib:@loader_path).

The issues is that the CMAKE_INSTALL_RPATH list used a colon as a separator for the path list, instead of a semicolon.

For reference:
https://cmake.org/cmake/help/latest/variable/CMAKE_INSTALL_RPATH.html

Features list

• Fix CMAKE_INSTALL_RPATH value

[fabiencastan]

@serguei-k Thanks for the fix! And sorry for the delay to merge it.

[fabiencastan]

Hi @serguei-k,
If you have a bit of time and motivation :), it would be really helpful if you could make a PR with some documentation for the macos build.
And hopefully one day, we could setup the macos build in the github action!
Cheers,

[serguei-k]

Hi @serguei-k, If you have a bit of time and motivation :), it would be really helpful if you could make a PR with some documentation for the macos build. And hopefully one day, we could setup the macos build in the github action! Cheers,

Hi @fabiencastan, I will give it a shot when I find some time.
vcpkg actually gets you 90% there, clp package wasn't working for some reason so I had to build Clp, CoinUtils and Osi manually. But this is something that probably could be resolved.
Using OpenMP on macos is not straight forward as it does not ship with clang anymore and I had trouble getting it to pick up external installation everywhere.
I am going to build on arch64 at some point too so that should be more interesting.