Skip to content
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: New package #19948

Merged
merged 3 commits into from Feb 6, 2024
Merged

libobjc2: New package #19948

merged 3 commits into from Feb 6, 2024

Conversation

qmfrederik
Copy link
Contributor

@qmfrederik qmfrederik commented Feb 4, 2024

libobjc2 is a modern Objective C runtime.

The v2.2 release of libobjc2 supports building on msys2 (at least mingw64 and ucrt64, via gnustep/libobjc2#254 and gnustep/libobjc2#267); and the clang patches which enable this (llvm/llvm-project#77797, llvm/llvm-project#77385, llvm/llvm-project#77255) have been backported to msys (#19767).

This PR adds libobjc2 to msys2.

Closes #11942
Closes #8287

@lazka
Copy link
Member

lazka commented Feb 5, 2024

The installed files look wrong:

  +/ucrt64/lib/objc.dll
  +/ucrt64/lib/objc.lib

This should be:

  +/ucrt64/bin/objc.dll
  +/ucrt64/lib/objc.dll.a

@qmfrederik
Copy link
Contributor Author

@lazka Thanks, I created a patch to fix the bin vs lib mismatch and hope to get it upstreamed. The .dll.a vs .lib difference seems to stem from using clang instead of gcc (libobjc2 supports clang only). I couldn't find a CMake variable which would influence this behavior, so I renamed the file in the package phase.

@qmfrederik
Copy link
Contributor Author

ok - that patch landed upstream just now: gnustep/libobjc2#275

@lazka lazka merged commit c061ef4 into msys2:master Feb 6, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants