llvm-shlib: Fix libLLVM-${MAJOR}.so symlink on MacOS#85163
Conversation
This is a partial revert of 10c48a7 with a fix for the symlink target name on MacOS See llvm#84637
|
This broke building the dylib for mingw platforms, see e.g. https://github.com/mstorsjo/llvm-mingw/actions/runs/8303983325/job/22728993479: Could we revert this until we sort it out, to get my builds back to green? |
|
@mstorsjo What symlinks / library names do you expect for mingw? |
I see the other recent changes here have regressed how libLLVM is set up for mingw, #82660 and/or #79376), so I see this behaves worse for 18.1.0 final, while it was ok in earlier RCs. The ideal situation for mingw is like it was before all these changes; |
|
@mstorsjo What files are currently produced? Is there a libLLVM.dll.18.1 file ? |
Currently, before this change, the build produces |
|
Can you try this fix: #85554 |
Thanks, this fixes the immediate build break. I can look into fixing the rest of the situation later when I have time (in a day or two). Previously, the effective “soname” on windows was |
This is a partial revert of 10c48a7 with a fix for the symlink target name on MacOS See llvm#84637 (cherry picked from commit ec2b752)
This is a partial revert of 10c48a7 with a fix for the symlink target name on MacOS See llvm#84637 (cherry picked from commit ec2b752)
This is a partial revert of 10c48a7 with a fix for the symlink target name on MacOS
See #84637