-
Notifications
You must be signed in to change notification settings - Fork 984
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
Upgrade clang and patch Glog for fixing iOS builds on MacOS Sonoma, Xcode 15 #18108
Conversation
Jenkins BuildsClick to see older builds (10)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you men by:
which conflicts with flags already set in nix iOS shell.
Which flags set in Nix iOS shell?
69% of end-end tests have passed
Failed tests (9)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Expected to fail tests (6)Click to expandClass TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Passed tests (34)Click to expandClass TestDeepLinksOneDevice:
Class TestActivityMultipleDevicePRTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityCenterContactRequestMultipleDevicePR:
|
2bd9fca
to
10c946d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's better, but I would be worried about weird future failures if we unser CC
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done :)
10c946d
to
6644a98
Compare
I understand, the new patch makes it more explicit that we're just avoid 2 flags being set by react-native :
|
6644a98
to
b90c0b1
Compare
@status-im/mobile-qa : we don't need QA for this since this PR just fixes build issues for MacOS Sonoma. |
b90c0b1
to
4afd104
Compare
I was facing the glog issue while trying to run
Here's the complete log file. Did anyone else face this issue ? |
@flexsurfer : faced this |
hey @shivekkhurana glad to see you :) so i don't know what helped but i cleaned entire project, |
Glad to see you too Andrey 👋👋 |
After upgrading MacOS to Sonoma and Xcode to 15, pod install stage would fail with clang errors. Turns out that react-native's configure glog script sets 2 additional flags, CC and CXX which conflicts with flags already set in nix iOS shell. In this commit we : - bump clang from 10.0.0 to 15.0.7 - patch glog configure script to avoid setting those flags This fixes the iOS builds.
fixes #17682
After upgrading
MacOS
toSonoma
andXcode
to 15, pod install stage would fail withclang
errors.Turns out that
react-native
's configureglog
script sets 2 additional flags,CC
andCXX
which conflicts with flags already set innix
iOS shell.In this PR we :
clang
from10.0.0
to15.0.7
glog
configure script to avoid setting those flagsThis fixes the
iOS
builds.status: ready