Skip to content

Fix MacOS build#837

Draft
zeyus wants to merge 1 commit intocommetchat:mainfrom
zeyus:feature/osx-build
Draft

Fix MacOS build#837
zeyus wants to merge 1 commit intocommetchat:mainfrom
zeyus:feature/osx-build

Conversation

@zeyus
Copy link

@zeyus zeyus commented Mar 13, 2026

I have access to an MacOS computer (and iPads), so I thought I could have a stab at getting the build working. I've been working a lot with dart and flutter for my research platform, and have had to deal with some similar issues (with the exception of rust libs, I've been using C FFI native libs).

I'm looking for some feedback on this PR, both in terms of scope and how you would like to have it chunked.

What this PR does already:

  • Makes the build for MacOS work
    • Adds required entitlements and Info.plist entries
  • Allows commet to launch

What doesn't work yet:

  • Critically, the build does not find the vodozemac / flutter_vodozemac library, and I haven't yet figured out why - the build does make the lib, but ends up outputting it as flutter_vodozemac (not, e.g. libflutter_vodozemac.dylib), but I made a quick test pointing directly to the lib file and it still didn't find it. I would need to spend more time investigating.

Untested yet:

  • iOS, while I added the required entries for camera and mic to the Info.plist, I haven't built and tested iOS yet

I guess my main question before turning this into a fully-fledged PR is:

  • Is it enough to merge this as an incremental step towards a MacOS / iOS build?
  • If not, how should I break it down to avoid this becoming too unweildy?
  • important: While using the main github version of the apple media_kit_* packages fixes the main problem that broke the build, obviously pointing to the head of a repo is dangerous. Would you prefer:
    • Pointing to a specific commit hash
    • That I make a fork / branch
    • That commetchat org makes a fork / branch

any is fine for me, once I know I can update the PR.

Thanks!

Edit: I wanted to make it clear that none of the changes in the PR, nor the text above used generative AI.

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.

1 participant