search

mattermost / mattermost-mobile

Next generation iOS and Android apps for Mattermost in React Native
https://about.mattermost.com/
Apache License 2.0
2.26k stars 1.37k forks source link

Add fixes for xCode 16 changes #8254

Closed larkox closed 1 month ago

larkox commented 1 month ago

Summary

With XCode 16 some changes were needed to properly compile the app.

This PR addresses those changes.

Ticket Link

NONE

Release Note

NONE
chrisfromredfin commented 1 month ago

Oof, can confirm that after days of trying to build these should do it (finally built about 12 hours ago - seeing this issue is 2 hours old 🤣)

For reference, the way I fixed it was while the npm run build:ios was running, but after the pods installed, I ran the following three commands from the mattermost-mobile directory.

These were the more base-level patches that were needed but are encapsulated by the typealias and the bump of @sentry/react-native packages that appear in this MR.

sed -i '' '41i\
#include <sys/_types/_ucontext64.h>
' ios/Pods/Sentry/Sources/SentryCrash/Recording/Tools/SentryCrashMachineContext.c

find ios/Gekidou/Sources/Gekidou/Storage/ -name "*.swift" -exec sed -i '' 's/= Expression/= SQLite.Expression/g' {} +

find ios/Gekidou/Sources/Gekidou/Storage/ -name "*.swift" -exec sed -i '' 's/user.get(Expression</user.get(SQLite.Expression</g' {} +

Note the sed syntax is BSD-style, since, ya know, Mac.