Diff 14625

native/chat/chat-input-bar.react.js

	require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")			require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")
	require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")			require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")
	require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")			require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")

	platform :ios, '12.0'			platform :ios, '12.0'

	require 'json'			require 'json'
	podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}

	target 'Comm' do			def common_comm_target_pods
	pod 'ReactNativeKeyboardTrackingView', :path => '../../node_modules/react-native-keyboard-tracking-view'
	pod 'ReactNativeKeyboardInput', :path => '../../node_modules/react-native-keyboard-input'
	pod 'react-native-ffmpeg/min-lts', :podspec => '../../node_modules/react-native-ffmpeg/react-native-ffmpeg.podspec'
	pod 'react-native-video/VideoCaching', :podspec => '../../node_modules/react-native-video/react-native-video.podspec'
	pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'			pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'

	pod 'gRPC-C++', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'			pod 'gRPC-C++', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'
	pod 'gRPC-C++/Protobuf', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'			pod 'gRPC-C++/Protobuf', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'
	pod 'gRPC-Core', :podspec => './pod-patch/.patched/gRPC-Core/1.40.0/gRPC-Core.podspec.json'			pod 'gRPC-Core', :podspec => './pod-patch/.patched/gRPC-Core/1.40.0/gRPC-Core.podspec.json'
	pod 'Protobuf-C++', '3.15.8'			pod 'Protobuf-C++', '3.15.8'
				end

				target 'Comm' do
				pod 'ReactNativeKeyboardTrackingView', :path => '../../node_modules/react-native-keyboard-tracking-view'
				pod 'ReactNativeKeyboardInput', :path => '../../node_modules/react-native-keyboard-input'
				pod 'react-native-video/VideoCaching', :podspec => '../../node_modules/react-native-video/react-native-video.podspec'
				pod 'react-native-ffmpeg/min-lts', :podspec => '../../node_modules/react-native-ffmpeg/react-native-ffmpeg.podspec'
				common_comm_target_pods

	use_expo_modules!			use_expo_modules!
	config = use_native_modules!			config = use_native_modules!
				podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}
	use_react_native!(			use_react_native!(
	:path => config[:reactNativePath],			:path => config[:reactNativePath],
	# to enable hermes on iOS, change `false` to `true` and then install pods			# to enable hermes on iOS, change `false` to `true` and then install pods
	:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'			:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'
	)			)

	# Enables Flipper.			# Enables Flipper.
	#			#
	# Note that if you have use_frameworks! enabled, Flipper will not work and			# Note that if you have use_frameworks! enabled, Flipper will not work and
	# you should disable these next few lines.			# you should disable these next few lines.
	use_flipper!({ "Flipper-DoubleConversion" => "1.1.7" })			use_flipper!({ "Flipper-DoubleConversion" => "1.1.7" })
				end

				target 'NotificationService' do
				common_comm_target_pods
				pod 'OLMKit', :path => "../../node_modules/olm"
				pod 'RCT-Folly', :podspec => "../../node_modules/react-native/third-party-podspecs/RCT-Folly.podspec"

				# Lines below are only needed to compile and use Expo Secure Store in Notification Service.
				# If Apple disapproves 'APPLICATION_EXTENSION_API_ONLY' flag then we can safely delete them and use
				# customized Expo Secure Store that does not depend on ExpoModulesCore
				podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}
				use_expo_modules!
				use_react_native!(
				:path => "../../node_modules/react-native",
				:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'
				)
				end

	post_install do \|installer\|			post_install do \|installer\|
				ashoatUnsubmitted Not Done Inline Actions Does this apply to all targets now? Are we sure we want to apply it to all targets? Is it possibly that we're inadvertedly applying it to some targets we haven't considered? (Targets that aren't the Comm app, or the notification service extension) ashoat: Does this apply to all targets now? Are we sure we want to apply it to all targets? Is it…
				marcinAuthorUnsubmitted Done Inline Actions There are two changes here compared to master branch: post_install hook is moved from target 'Comm' scope to global Podfile scope some lines are added that disable APPLICATION_EXTENSION_API_ONLY flag to enable CommonCpp sources compilation in 'NotificationService' target. Let me explain why those changes should not raise concern. Ad. 1) According to this doc: https://www.rubydoc.info/github/CocoaPods/Core/Pod%2FPodfile%2FDSL:post_install post_install is a hook applied just before XCode project is written to disk. According to this doc: https://www.rubydoc.info/gems/cocoapods-core/0.34.1/Pod/Podfile/DSL Podfile hooks always exist in global scope. Even if this post_install hook was written inside target 'Comm' scope its behaviour would be the same as if it was in global scope. This is even more confirmed by the fact that an attempt to write multiple post_install hooks (e.g. each per target) leads to errors: https://github.com/CocoaPods/CocoaPods/issues/6172. Ad. 2) Now that we established that we cannot have multiple post_install hooks and the one that we have should be in global scope we can admit it is impossible to use this flag anywhere else. marcin: There are two changes here compared to master branch: 1) post_install hook is moved from target…
				ashoatUnsubmitted Not Done Inline Actions Thanks for the explanation! ashoat: Thanks for the explanation!
	react_native_post_install(installer)			react_native_post_install(installer)
	__apply_Xcode_12_5_M1_post_install_workaround(installer)			__apply_Xcode_12_5_M1_post_install_workaround(installer)

	# Excluding arm64 because M1 simulator isn't supported yet			# Excluding arm64 because M1 simulator isn't supported yet
	# Excluding 32-bit x86 because nobody uses it and it causes compilation issues			# Excluding 32-bit x86 because nobody uses it and it causes compilation issues
	installer.pods_project.build_configurations.each do \|config\|			installer.pods_project.build_configurations.each do \|config\|
	config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = "arm64 i386"			config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = "arm64 i386"
				ashoatUnsubmitted Not Done Inline Actions It's a bit confusing to me that we exclude both of these here, but in the `project.pbxproj` only `arm64` is mentioned. Do you know why that is? ashoat: It's a bit confusing to me that we exclude both of these here, but in the `project.pbxproj`…
				marcinAuthorUnsubmitted Done Inline Actions Build settings in the XCode project refer to targets: Comm and NotificationService. This line changes build settings for pods. I did not modify this line and according to the explanation above it has alway been that we exclude both architectures for all pods. I do not know why it is that, but it is not related to any change introduced in this diff. Target 'Comm' excluded only amr64 architecture before I started to work on this issue and the very recent changes I introduced were to unify excluded architectures for both Comm and NotificationService. Would you like me to try whether we can exclude i386 arch for XCode targets as well? marcin: Build settings in the XCode project refer to targets: Comm and NotificationService. This line…
				ashoatUnsubmitted Not Done Inline Actions No, that's alright – thanks for this explanation ashoat: No, that's alright – thanks for this explanation
	config.build_settings['OTHER_CPLUSPLUSFLAGS'] = '-DDONT_AUTOINSTALL_REANIMATED'			config.build_settings['OTHER_CPLUSPLUSFLAGS'] = '-DDONT_AUTOINSTALL_REANIMATED'
	end			end

				# Lines below are only needed to compile and use Expo Secure Store in Notification Service.
				# If Apple disapproves 'APPLICATION_EXTENSION_API_ONLY' flag then we can safely delete them and use
				# customized Expo Secure Store that does not depend on ExpoModulesCore
				installer.pods_project.targets.each do \|target\|
				target.build_configurations.each do \|config\|
				config.build_settings['APPLICATION_EXTENSION_API_ONLY'] = 'NO'
				end
	end			end
				ashoatUnsubmitted Not Done Inline Actions There's an extraneous space at the end of this line ashoat: There's an extraneous space at the end of this line
	end			end

View Options

native/chat/chat-thread-list.react.js

	require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")			require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")
	require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")			require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")
	require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")			require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")

	platform :ios, '12.0'			platform :ios, '12.0'

	require 'json'			require 'json'
	podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}

	target 'Comm' do			def common_comm_target_pods
	pod 'ReactNativeKeyboardTrackingView', :path => '../../node_modules/react-native-keyboard-tracking-view'
	pod 'ReactNativeKeyboardInput', :path => '../../node_modules/react-native-keyboard-input'
	pod 'react-native-ffmpeg/min-lts', :podspec => '../../node_modules/react-native-ffmpeg/react-native-ffmpeg.podspec'
	pod 'react-native-video/VideoCaching', :podspec => '../../node_modules/react-native-video/react-native-video.podspec'
	pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'			pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'

	pod 'gRPC-C++', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'			pod 'gRPC-C++', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'
	pod 'gRPC-C++/Protobuf', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'			pod 'gRPC-C++/Protobuf', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'
	pod 'gRPC-Core', :podspec => './pod-patch/.patched/gRPC-Core/1.40.0/gRPC-Core.podspec.json'			pod 'gRPC-Core', :podspec => './pod-patch/.patched/gRPC-Core/1.40.0/gRPC-Core.podspec.json'
	pod 'Protobuf-C++', '3.15.8'			pod 'Protobuf-C++', '3.15.8'
				end

				target 'Comm' do
				pod 'ReactNativeKeyboardTrackingView', :path => '../../node_modules/react-native-keyboard-tracking-view'
				pod 'ReactNativeKeyboardInput', :path => '../../node_modules/react-native-keyboard-input'
				pod 'react-native-video/VideoCaching', :podspec => '../../node_modules/react-native-video/react-native-video.podspec'
				pod 'react-native-ffmpeg/min-lts', :podspec => '../../node_modules/react-native-ffmpeg/react-native-ffmpeg.podspec'
				common_comm_target_pods

	use_expo_modules!			use_expo_modules!
	config = use_native_modules!			config = use_native_modules!
				podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}
	use_react_native!(			use_react_native!(
	:path => config[:reactNativePath],			:path => config[:reactNativePath],
	# to enable hermes on iOS, change `false` to `true` and then install pods			# to enable hermes on iOS, change `false` to `true` and then install pods
	:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'			:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'
	)			)

	# Enables Flipper.			# Enables Flipper.
	#			#
	# Note that if you have use_frameworks! enabled, Flipper will not work and			# Note that if you have use_frameworks! enabled, Flipper will not work and
	# you should disable these next few lines.			# you should disable these next few lines.
	use_flipper!({ "Flipper-DoubleConversion" => "1.1.7" })			use_flipper!({ "Flipper-DoubleConversion" => "1.1.7" })
				end

				target 'NotificationService' do
				common_comm_target_pods
				pod 'OLMKit', :path => "../../node_modules/olm"
				pod 'RCT-Folly', :podspec => "../../node_modules/react-native/third-party-podspecs/RCT-Folly.podspec"

				# Lines below are only needed to compile and use Expo Secure Store in Notification Service.
				# If Apple disapproves 'APPLICATION_EXTENSION_API_ONLY' flag then we can safely delete them and use
				# customized Expo Secure Store that does not depend on ExpoModulesCore
				podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}
				use_expo_modules!
				use_react_native!(
				:path => "../../node_modules/react-native",
				:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'
				)
				end

	post_install do \|installer\|			post_install do \|installer\|
				ashoatUnsubmitted Not Done Inline Actions Does this apply to all targets now? Are we sure we want to apply it to all targets? Is it possibly that we're inadvertedly applying it to some targets we haven't considered? (Targets that aren't the Comm app, or the notification service extension) ashoat: Does this apply to all targets now? Are we sure we want to apply it to all targets? Is it…
				marcinAuthorUnsubmitted Done Inline Actions There are two changes here compared to master branch: post_install hook is moved from target 'Comm' scope to global Podfile scope some lines are added that disable APPLICATION_EXTENSION_API_ONLY flag to enable CommonCpp sources compilation in 'NotificationService' target. Let me explain why those changes should not raise concern. Ad. 1) According to this doc: https://www.rubydoc.info/github/CocoaPods/Core/Pod%2FPodfile%2FDSL:post_install post_install is a hook applied just before XCode project is written to disk. According to this doc: https://www.rubydoc.info/gems/cocoapods-core/0.34.1/Pod/Podfile/DSL Podfile hooks always exist in global scope. Even if this post_install hook was written inside target 'Comm' scope its behaviour would be the same as if it was in global scope. This is even more confirmed by the fact that an attempt to write multiple post_install hooks (e.g. each per target) leads to errors: https://github.com/CocoaPods/CocoaPods/issues/6172. Ad. 2) Now that we established that we cannot have multiple post_install hooks and the one that we have should be in global scope we can admit it is impossible to use this flag anywhere else. marcin: There are two changes here compared to master branch: 1) post_install hook is moved from target…
				ashoatUnsubmitted Not Done Inline Actions Thanks for the explanation! ashoat: Thanks for the explanation!
	react_native_post_install(installer)			react_native_post_install(installer)
	__apply_Xcode_12_5_M1_post_install_workaround(installer)			__apply_Xcode_12_5_M1_post_install_workaround(installer)

	# Excluding arm64 because M1 simulator isn't supported yet			# Excluding arm64 because M1 simulator isn't supported yet
	# Excluding 32-bit x86 because nobody uses it and it causes compilation issues			# Excluding 32-bit x86 because nobody uses it and it causes compilation issues
	installer.pods_project.build_configurations.each do \|config\|			installer.pods_project.build_configurations.each do \|config\|
	config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = "arm64 i386"			config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = "arm64 i386"
				ashoatUnsubmitted Not Done Inline Actions It's a bit confusing to me that we exclude both of these here, but in the `project.pbxproj` only `arm64` is mentioned. Do you know why that is? ashoat: It's a bit confusing to me that we exclude both of these here, but in the `project.pbxproj`…
				marcinAuthorUnsubmitted Done Inline Actions Build settings in the XCode project refer to targets: Comm and NotificationService. This line changes build settings for pods. I did not modify this line and according to the explanation above it has alway been that we exclude both architectures for all pods. I do not know why it is that, but it is not related to any change introduced in this diff. Target 'Comm' excluded only amr64 architecture before I started to work on this issue and the very recent changes I introduced were to unify excluded architectures for both Comm and NotificationService. Would you like me to try whether we can exclude i386 arch for XCode targets as well? marcin: Build settings in the XCode project refer to targets: Comm and NotificationService. This line…
				ashoatUnsubmitted Not Done Inline Actions No, that's alright – thanks for this explanation ashoat: No, that's alright – thanks for this explanation
	config.build_settings['OTHER_CPLUSPLUSFLAGS'] = '-DDONT_AUTOINSTALL_REANIMATED'			config.build_settings['OTHER_CPLUSPLUSFLAGS'] = '-DDONT_AUTOINSTALL_REANIMATED'
	end			end

				# Lines below are only needed to compile and use Expo Secure Store in Notification Service.
				# If Apple disapproves 'APPLICATION_EXTENSION_API_ONLY' flag then we can safely delete them and use
				# customized Expo Secure Store that does not depend on ExpoModulesCore
				installer.pods_project.targets.each do \|target\|
				target.build_configurations.each do \|config\|
				config.build_settings['APPLICATION_EXTENSION_API_ONLY'] = 'NO'
				end
	end			end
				ashoatUnsubmitted Not Done Inline Actions There's an extraneous space at the end of this line ashoat: There's an extraneous space at the end of this line
	end			end

View Options

native/chat/chat.react.js

	require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")			require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")
	require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")			require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")
	require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")			require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")

	platform :ios, '12.0'			platform :ios, '12.0'

	require 'json'			require 'json'
	podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}

	target 'Comm' do			def common_comm_target_pods
	pod 'ReactNativeKeyboardTrackingView', :path => '../../node_modules/react-native-keyboard-tracking-view'
	pod 'ReactNativeKeyboardInput', :path => '../../node_modules/react-native-keyboard-input'
	pod 'react-native-ffmpeg/min-lts', :podspec => '../../node_modules/react-native-ffmpeg/react-native-ffmpeg.podspec'
	pod 'react-native-video/VideoCaching', :podspec => '../../node_modules/react-native-video/react-native-video.podspec'
	pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'			pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'

	pod 'gRPC-C++', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'			pod 'gRPC-C++', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'
	pod 'gRPC-C++/Protobuf', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'			pod 'gRPC-C++/Protobuf', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'
	pod 'gRPC-Core', :podspec => './pod-patch/.patched/gRPC-Core/1.40.0/gRPC-Core.podspec.json'			pod 'gRPC-Core', :podspec => './pod-patch/.patched/gRPC-Core/1.40.0/gRPC-Core.podspec.json'
	pod 'Protobuf-C++', '3.15.8'			pod 'Protobuf-C++', '3.15.8'
				end

				target 'Comm' do
				pod 'ReactNativeKeyboardTrackingView', :path => '../../node_modules/react-native-keyboard-tracking-view'
				pod 'ReactNativeKeyboardInput', :path => '../../node_modules/react-native-keyboard-input'
				pod 'react-native-video/VideoCaching', :podspec => '../../node_modules/react-native-video/react-native-video.podspec'
				pod 'react-native-ffmpeg/min-lts', :podspec => '../../node_modules/react-native-ffmpeg/react-native-ffmpeg.podspec'
				common_comm_target_pods

	use_expo_modules!			use_expo_modules!
	config = use_native_modules!			config = use_native_modules!
				podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}
	use_react_native!(			use_react_native!(
	:path => config[:reactNativePath],			:path => config[:reactNativePath],
	# to enable hermes on iOS, change `false` to `true` and then install pods			# to enable hermes on iOS, change `false` to `true` and then install pods
	:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'			:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'
	)			)

	# Enables Flipper.			# Enables Flipper.
	#			#
	# Note that if you have use_frameworks! enabled, Flipper will not work and			# Note that if you have use_frameworks! enabled, Flipper will not work and
	# you should disable these next few lines.			# you should disable these next few lines.
	use_flipper!({ "Flipper-DoubleConversion" => "1.1.7" })			use_flipper!({ "Flipper-DoubleConversion" => "1.1.7" })
				end

				target 'NotificationService' do
				common_comm_target_pods
				pod 'OLMKit', :path => "../../node_modules/olm"
				pod 'RCT-Folly', :podspec => "../../node_modules/react-native/third-party-podspecs/RCT-Folly.podspec"

				# Lines below are only needed to compile and use Expo Secure Store in Notification Service.
				# If Apple disapproves 'APPLICATION_EXTENSION_API_ONLY' flag then we can safely delete them and use
				# customized Expo Secure Store that does not depend on ExpoModulesCore
				podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}
				use_expo_modules!
				use_react_native!(
				:path => "../../node_modules/react-native",
				:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'
				)
				end

	post_install do \|installer\|			post_install do \|installer\|
				ashoatUnsubmitted Not Done Inline Actions Does this apply to all targets now? Are we sure we want to apply it to all targets? Is it possibly that we're inadvertedly applying it to some targets we haven't considered? (Targets that aren't the Comm app, or the notification service extension) ashoat: Does this apply to all targets now? Are we sure we want to apply it to all targets? Is it…
				marcinAuthorUnsubmitted Done Inline Actions There are two changes here compared to master branch: post_install hook is moved from target 'Comm' scope to global Podfile scope some lines are added that disable APPLICATION_EXTENSION_API_ONLY flag to enable CommonCpp sources compilation in 'NotificationService' target. Let me explain why those changes should not raise concern. Ad. 1) According to this doc: https://www.rubydoc.info/github/CocoaPods/Core/Pod%2FPodfile%2FDSL:post_install post_install is a hook applied just before XCode project is written to disk. According to this doc: https://www.rubydoc.info/gems/cocoapods-core/0.34.1/Pod/Podfile/DSL Podfile hooks always exist in global scope. Even if this post_install hook was written inside target 'Comm' scope its behaviour would be the same as if it was in global scope. This is even more confirmed by the fact that an attempt to write multiple post_install hooks (e.g. each per target) leads to errors: https://github.com/CocoaPods/CocoaPods/issues/6172. Ad. 2) Now that we established that we cannot have multiple post_install hooks and the one that we have should be in global scope we can admit it is impossible to use this flag anywhere else. marcin: There are two changes here compared to master branch: 1) post_install hook is moved from target…
				ashoatUnsubmitted Not Done Inline Actions Thanks for the explanation! ashoat: Thanks for the explanation!
	react_native_post_install(installer)			react_native_post_install(installer)
	__apply_Xcode_12_5_M1_post_install_workaround(installer)			__apply_Xcode_12_5_M1_post_install_workaround(installer)

	# Excluding arm64 because M1 simulator isn't supported yet			# Excluding arm64 because M1 simulator isn't supported yet
	# Excluding 32-bit x86 because nobody uses it and it causes compilation issues			# Excluding 32-bit x86 because nobody uses it and it causes compilation issues
	installer.pods_project.build_configurations.each do \|config\|			installer.pods_project.build_configurations.each do \|config\|
	config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = "arm64 i386"			config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = "arm64 i386"
				ashoatUnsubmitted Not Done Inline Actions It's a bit confusing to me that we exclude both of these here, but in the `project.pbxproj` only `arm64` is mentioned. Do you know why that is? ashoat: It's a bit confusing to me that we exclude both of these here, but in the `project.pbxproj`…
				marcinAuthorUnsubmitted Done Inline Actions Build settings in the XCode project refer to targets: Comm and NotificationService. This line changes build settings for pods. I did not modify this line and according to the explanation above it has alway been that we exclude both architectures for all pods. I do not know why it is that, but it is not related to any change introduced in this diff. Target 'Comm' excluded only amr64 architecture before I started to work on this issue and the very recent changes I introduced were to unify excluded architectures for both Comm and NotificationService. Would you like me to try whether we can exclude i386 arch for XCode targets as well? marcin: Build settings in the XCode project refer to targets: Comm and NotificationService. This line…
				ashoatUnsubmitted Not Done Inline Actions No, that's alright – thanks for this explanation ashoat: No, that's alright – thanks for this explanation
	config.build_settings['OTHER_CPLUSPLUSFLAGS'] = '-DDONT_AUTOINSTALL_REANIMATED'			config.build_settings['OTHER_CPLUSPLUSFLAGS'] = '-DDONT_AUTOINSTALL_REANIMATED'
	end			end

				# Lines below are only needed to compile and use Expo Secure Store in Notification Service.
				# If Apple disapproves 'APPLICATION_EXTENSION_API_ONLY' flag then we can safely delete them and use
				# customized Expo Secure Store that does not depend on ExpoModulesCore
				installer.pods_project.targets.each do \|target\|
				target.build_configurations.each do \|config\|
				config.build_settings['APPLICATION_EXTENSION_API_ONLY'] = 'NO'
				end
	end			end
				ashoatUnsubmitted Not Done Inline Actions There's an extraneous space at the end of this line ashoat: There's an extraneous space at the end of this line
	end			end

View Options

native/chat/compose-subchannel.react.js

	require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")			require File.join(File.dirname(`node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")
	require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")			require File.join(File.dirname(`node --print "require.resolve('react-native/package.json')"`), "scripts/react_native_pods")
	require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")			require File.join(File.dirname(`node --print "require.resolve('@react-native-community/cli-platform-ios/package.json')"`), "native_modules")

	platform :ios, '12.0'			platform :ios, '12.0'

	require 'json'			require 'json'
	podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}

	target 'Comm' do			def common_comm_target_pods
	pod 'ReactNativeKeyboardTrackingView', :path => '../../node_modules/react-native-keyboard-tracking-view'
	pod 'ReactNativeKeyboardInput', :path => '../../node_modules/react-native-keyboard-input'
	pod 'react-native-ffmpeg/min-lts', :podspec => '../../node_modules/react-native-ffmpeg/react-native-ffmpeg.podspec'
	pod 'react-native-video/VideoCaching', :podspec => '../../node_modules/react-native-video/react-native-video.podspec'
	pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'			pod 'SQLCipher-Amalgamation', :path => '../../node_modules/@commapp/sqlcipher-amalgamation'

	pod 'gRPC-C++', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'			pod 'gRPC-C++', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'
	pod 'gRPC-C++/Protobuf', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'			pod 'gRPC-C++/Protobuf', :podspec => './pod-patch/.patched/gRPC-C++/1.40.0/gRPC-C++.podspec.json'
	pod 'gRPC-Core', :podspec => './pod-patch/.patched/gRPC-Core/1.40.0/gRPC-Core.podspec.json'			pod 'gRPC-Core', :podspec => './pod-patch/.patched/gRPC-Core/1.40.0/gRPC-Core.podspec.json'
	pod 'Protobuf-C++', '3.15.8'			pod 'Protobuf-C++', '3.15.8'
				end

				target 'Comm' do
				pod 'ReactNativeKeyboardTrackingView', :path => '../../node_modules/react-native-keyboard-tracking-view'
				pod 'ReactNativeKeyboardInput', :path => '../../node_modules/react-native-keyboard-input'
				pod 'react-native-video/VideoCaching', :podspec => '../../node_modules/react-native-video/react-native-video.podspec'
				pod 'react-native-ffmpeg/min-lts', :podspec => '../../node_modules/react-native-ffmpeg/react-native-ffmpeg.podspec'
				common_comm_target_pods

	use_expo_modules!			use_expo_modules!
	config = use_native_modules!			config = use_native_modules!
				podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}
	use_react_native!(			use_react_native!(
	:path => config[:reactNativePath],			:path => config[:reactNativePath],
	# to enable hermes on iOS, change `false` to `true` and then install pods			# to enable hermes on iOS, change `false` to `true` and then install pods
	:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'			:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'
	)			)

	# Enables Flipper.			# Enables Flipper.
	#			#
	# Note that if you have use_frameworks! enabled, Flipper will not work and			# Note that if you have use_frameworks! enabled, Flipper will not work and
	# you should disable these next few lines.			# you should disable these next few lines.
	use_flipper!({ "Flipper-DoubleConversion" => "1.1.7" })			use_flipper!({ "Flipper-DoubleConversion" => "1.1.7" })
				end

				target 'NotificationService' do
				common_comm_target_pods
				pod 'OLMKit', :path => "../../node_modules/olm"
				pod 'RCT-Folly', :podspec => "../../node_modules/react-native/third-party-podspecs/RCT-Folly.podspec"

				# Lines below are only needed to compile and use Expo Secure Store in Notification Service.
				# If Apple disapproves 'APPLICATION_EXTENSION_API_ONLY' flag then we can safely delete them and use
				# customized Expo Secure Store that does not depend on ExpoModulesCore
				podfile_properties = JSON.parse(File.read('./Podfile.properties.json')) rescue {}
				use_expo_modules!
				use_react_native!(
				:path => "../../node_modules/react-native",
				:hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes'
				)
				end

	post_install do \|installer\|			post_install do \|installer\|
				ashoatUnsubmitted Not Done Inline Actions Does this apply to all targets now? Are we sure we want to apply it to all targets? Is it possibly that we're inadvertedly applying it to some targets we haven't considered? (Targets that aren't the Comm app, or the notification service extension) ashoat: Does this apply to all targets now? Are we sure we want to apply it to all targets? Is it…
				marcinAuthorUnsubmitted Done Inline Actions There are two changes here compared to master branch: post_install hook is moved from target 'Comm' scope to global Podfile scope some lines are added that disable APPLICATION_EXTENSION_API_ONLY flag to enable CommonCpp sources compilation in 'NotificationService' target. Let me explain why those changes should not raise concern. Ad. 1) According to this doc: https://www.rubydoc.info/github/CocoaPods/Core/Pod%2FPodfile%2FDSL:post_install post_install is a hook applied just before XCode project is written to disk. According to this doc: https://www.rubydoc.info/gems/cocoapods-core/0.34.1/Pod/Podfile/DSL Podfile hooks always exist in global scope. Even if this post_install hook was written inside target 'Comm' scope its behaviour would be the same as if it was in global scope. This is even more confirmed by the fact that an attempt to write multiple post_install hooks (e.g. each per target) leads to errors: https://github.com/CocoaPods/CocoaPods/issues/6172. Ad. 2) Now that we established that we cannot have multiple post_install hooks and the one that we have should be in global scope we can admit it is impossible to use this flag anywhere else. marcin: There are two changes here compared to master branch: 1) post_install hook is moved from target…
				ashoatUnsubmitted Not Done Inline Actions Thanks for the explanation! ashoat: Thanks for the explanation!
	react_native_post_install(installer)			react_native_post_install(installer)
	__apply_Xcode_12_5_M1_post_install_workaround(installer)			__apply_Xcode_12_5_M1_post_install_workaround(installer)

	# Excluding arm64 because M1 simulator isn't supported yet			# Excluding arm64 because M1 simulator isn't supported yet
	# Excluding 32-bit x86 because nobody uses it and it causes compilation issues			# Excluding 32-bit x86 because nobody uses it and it causes compilation issues
	installer.pods_project.build_configurations.each do \|config\|			installer.pods_project.build_configurations.each do \|config\|
	config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = "arm64 i386"			config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = "arm64 i386"
				ashoatUnsubmitted Not Done Inline Actions It's a bit confusing to me that we exclude both of these here, but in the `project.pbxproj` only `arm64` is mentioned. Do you know why that is? ashoat: It's a bit confusing to me that we exclude both of these here, but in the `project.pbxproj`…
				marcinAuthorUnsubmitted Done Inline Actions Build settings in the XCode project refer to targets: Comm and NotificationService. This line changes build settings for pods. I did not modify this line and according to the explanation above it has alway been that we exclude both architectures for all pods. I do not know why it is that, but it is not related to any change introduced in this diff. Target 'Comm' excluded only amr64 architecture before I started to work on this issue and the very recent changes I introduced were to unify excluded architectures for both Comm and NotificationService. Would you like me to try whether we can exclude i386 arch for XCode targets as well? marcin: Build settings in the XCode project refer to targets: Comm and NotificationService. This line…
				ashoatUnsubmitted Not Done Inline Actions No, that's alright – thanks for this explanation ashoat: No, that's alright – thanks for this explanation
	config.build_settings['OTHER_CPLUSPLUSFLAGS'] = '-DDONT_AUTOINSTALL_REANIMATED'			config.build_settings['OTHER_CPLUSPLUSFLAGS'] = '-DDONT_AUTOINSTALL_REANIMATED'
	end			end

				# Lines below are only needed to compile and use Expo Secure Store in Notification Service.
				# If Apple disapproves 'APPLICATION_EXTENSION_API_ONLY' flag then we can safely delete them and use
				# customized Expo Secure Store that does not depend on ExpoModulesCore
				installer.pods_project.targets.each do \|target\|
				target.build_configurations.each do \|config\|
				config.build_settings['APPLICATION_EXTENSION_API_ONLY'] = 'NO'
				end
	end			end
				ashoatUnsubmitted Not Done Inline Actions There's an extraneous space at the end of this line ashoat: There's an extraneous space at the end of this line
	end			end