-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Consider supporting macOS App Sandbox #101
Comments
With this issue as-is, I am not going to fix this. Not worth the time and it would be an absolute pain. Even once/if LMDB is patched as necessary, |
If anyone stumbles across this issue worried, please do know that iOS/iPadOS should be ok for the App Store. I will support iOS for submission to the App Store in mimir since I need it for my application. |
Web support is only going to be possible through a different backend (something other than LMDB, probably redb), which we could actually just set as the default in macOS, which would in turn give us App Sandbox support back! See #10. If that gets fixed, it is likely App Sandbox support will come back for macOS, but with the tradeoff that all old data will be unreadable by new versions of mimir which is problematic for app authors. |
Update: unlikely this will be supported as I am probably just going to provide an IndexedDB shim over in heed. If milli ever switches to a different K/V backend, then this issue can be reconsidered. |
This might be possible once we have Native Assets; we can add an environment variable of the application group id to the embedded_milli Cargo.toml and then check for the environment variable in lmdb-sys |
I'm going to mess around with a |
Right now, we are getting sandbox violations from POSIX semaphores not following the proper naming scheme.
See here:
Currently they are
/MDB...
. This issue really sucks because I looked briefly in the LMDB source and did not really see great support for customizing POSIX semaphore prefixes as-is. There is someMUTEXNAME_PREFIX
macro, but it really doesn't allow for customization as far as I can tell. This would require submitting an issue to LMDB directly and see what they recommend.Another yippee.
...And this would require library-users to go through plenty of hoops during compilation and to change entitlements.
The text was updated successfully, but these errors were encountered: