-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
libobjc2: Add clang64 support #20024
Conversation
The is a mistake, the import library should be named in gnustep/libobjc2#276, it should be if (MINGW)
set(CMAKE_IMPORT_LIBRARY_SUFFIX ".dll.a")
set(CMAKE_IMPORT_LIBRARY_PREFIX "lib")
set(CMAKE_LINK_LIBRARY_SUFFIX "")
set(CMAKE_SHARED_LIBRARY_PREFIX "")
set(CMAKE_OBJCXX_IMPLICIT_INCLUDE_DIRECTORIES ${CMAKE_CXX_IMPLICIT_INCLUDE_DIRECTORIES})
endif () |
- Use "lib{library}.dll.a" for import libraries - Use "{library.dll}" for shared libraries See msys2/MINGW-packages#20024 (comment)
Thanks @MehdiChinoune , I submitted a patch for upstream and included it in this PR. With this, we now have: /mingw64/lib/libobjc.dll.a Is this what you were expecting? |
@qmfrederik could you try cmake from #20028 |
We no longer need to manually set these suffixes/prefixes as CMake in MSYS2 now knows about Objective C. See msys2/MINGW-packages#20028 See https://gitlab.kitware.com/cmake/cmake/-/merge_requests/9244 See msys2/MINGW-packages#20024 (comment)
@MehdiChinoune Thank you for all your help. The libobjc2 patch got merged and I updated PKGBUILD to use the merged version of that patch. The installed files are now:
which I believe is what we want. |
The name of dll was changed, so I think all reverse-dependencies should be rebuilt. |
Upstream libobjc2 now has clang64 support (via gnustep/libobjc2#276). Cherry-pick that commit and add clang64 to the build matrix for libobjc2. This cherry-picked commit also fixes the ".dll.a" suffix for library objects.
80bd572
to
3d982e7
Compare
We no longer need to manually set these suffixes/prefixes as CMake in MSYS2 now knows about Objective C. See msys2/MINGW-packages#20028 See https://gitlab.kitware.com/cmake/cmake/-/merge_requests/9244 See msys2/MINGW-packages#20024 (comment)
Upstream libobjc2 now has clang64 support (via gnustep/libobjc2#276).
Cherry-pick that commit and add clang64 to the build matrix for libobjc2. This cherry-picked commit also fixes the ".dll.a" suffix for library objects, so we no longer need to manually rename that file.