Skip to content

Add macOS Universal Binary runtime support#625

Merged
bdovaz merged 1 commit intobdovaz:masterfrom
nowsprinting:fix/osxfat
Feb 5, 2026
Merged

Add macOS Universal Binary runtime support#625
bdovaz merged 1 commit intobdovaz:masterfrom
nowsprinting:fix/osxfat

Conversation

@nowsprinting
Copy link
Copy Markdown
Contributor

@nowsprinting nowsprinting commented Jan 17, 2026

Summary

  • Support osx (without architecture suffix) runtime identifier for native libraries
  • Enable Universal Binary (FAT) packages that work on both Intel and Apple Silicon Macs
  • Use UnityCpu.AnyCpu for osx Universal Binary native libraries

Changes

  • Modified NativeLibraries.cs to accept osx runtime identifier without architecture suffix
  • Added test case for osx Universal Binary native libraries

Examples of Universal Binary packages

  • MongoDB.Libmongocrypt
  • FlipBinding.CSharp

Related

Test plan

  • Run RegistryTests - all 457 tests passed
  • Verified installation via Unity Package Manager with local docker-compose (FlipBinding.CSharp) on macOS

🤖 Generated with Claude Code

Support osx (without architecture suffix) runtime identifier for native libraries, enabling Universal Binary packages that work on both Intel and Apple Silicon Macs.

Changes:
- Modified NativeLibraries.cs to accept osx runtime identifier without architecture
- Use UnityCpu.AnyCpu for osx Universal Binary native libraries
- Added test case for osx Universal Binary native libraries

Related: GlitchEnzo/NuGetForUnity#744

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@nowsprinting
Copy link
Copy Markdown
Contributor Author

@bdovaz san,
I just wanted to follow up on this PR. It’s been a couple of weeks, and I was wondering if there’s any additional information or changes needed from my side.

Please let me know if I can help move this forward. Thanks!

@bdovaz
Copy link
Copy Markdown
Owner

bdovaz commented Feb 5, 2026

I have my doubts that you used Claude to create the PR, that is, you didn't do it yourself, right? Having delegated to AI makes me doubt that you really know whether the changes that have been made make sense or not.

@nowsprinting
Copy link
Copy Markdown
Contributor Author

I instructed Claude, reviewed the fixes, checked tests passed, and confirmed they work on my local computer.
How can I prove it?

@bdovaz bdovaz merged commit 42d80a9 into bdovaz:master Feb 5, 2026
3 checks passed
@nowsprinting nowsprinting deleted the fix/osxfat branch February 5, 2026 16:01
@nowsprinting
Copy link
Copy Markdown
Contributor Author

Thanks!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants