-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add C# data provider support for macOS #264
Comments
Here is some Q&A from Marc Durdin: (Q's from me, A's from him)
|
I must say that all of that is not encouraging. 😖 |
For what it's worth I publish the electron app lameta for macs using only vscode, but that's typescript only. I don't publish to the store, but I do have to sign and get the notarization. Apple completely changed this last month, and I found getting the electron app notarized in the new system to be quite a puzzle for lack of documentation. Eventually got it working, so that github action yml may be of use as a reference: https://github.com/onset/lameta |
Also, Marc was talking about an app with native UI. If you're doing electron, in my experience,
|
Unit testing will look like reenabling unit tests that didn't work on mac before. |
Discovered a bug in the icu4c library causing test failures. We need to try rolling back to an older icu4c build on macOS if they are available to avoid these errors. |
Turns out that wasn't a bug in icu4c but a bug in icu-dotnet's handling of UnicodeSet data. |
@lyonsil I'd be interested to know where you settle on getting the mac native icu4c library from for Paratext, I just ran into the Microsoft nuget package that has windows an linux binaries, but wasn't sure what to do about mac. |
@hahn-kev We haven't sorted it out yet, but as a first cut I'm assuming we will lean on MacPorts (which is what we are using in the macOS builds for icu-dotnet's GitHub build actions). MacPorts has both binary and build from source options available. Presumably we can bundle a universal build of icu4c libs into the platform installer for macOS. |
To decide:
Regarding how Paratext Lite handles this:
Additional resources of interest:
The text was updated successfully, but these errors were encountered: