Skip to content

Commit d4934fb

Browse files
jmagmanjmagman
authored
Add readlink -f flag to CocoaPods script to workaround Xcode 14.3 issue (#124062)
1 parent d32dc71 commit d4934fb

File tree

5 files changed

+223
-0
lines changed

5 files changed

+223
-0
lines changed

‎packages/flutter_tools/lib/src/macos/cocoapods.dart‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,12 @@ import '../base/logger.dart';
1313
import '../base/os.dart';
1414
import '../base/platform.dart';
1515
import '../base/process.dart';
16+
import '../base/project_migrator.dart';
1617
import '../base/version.dart';
1718
import '../build_info.dart';
1819
import '../cache.dart';
1920
import '../ios/xcodeproj.dart';
21+
import '../migrations/cocoapods_script_symlink.dart';
2022
import '../reporting/reporting.dart';
2123
import '../xcode_project.dart';
2224

@@ -166,6 +168,13 @@ class CocoaPods {
166168
throwToolExit('CocoaPods not installed or not in valid state.');
167169
}
168170
await _runPodInstall(xcodeProject, buildMode);
171+
172+
// This migrator works around a CocoaPods bug, and should be run after `pod install` is run.
173+
final ProjectMigration postPodMigration = ProjectMigration(<ProjectMigrator>[
174+
CocoaPodsScriptReadlink(xcodeProject, _xcodeProjectInterpreter, _logger),
175+
]);
176+
postPodMigration.run();
177+
169178
podsProcessed = true;
170179
}
171180
return podsProcessed;

‎packages/flutter_tools/lib/src/migrations/cocoapods_script_symlink.dart‎

Lines changed: 52 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,52 @@
1+
// Copyright 2014 The Flutter Authors. All rights reserved.
2+
// Use of this source code is governed by a BSD-style license that can be
3+
// found in the LICENSE file.
4+
5+
import '../base/file_system.dart';
6+
import '../base/project_migrator.dart';
7+
import '../base/version.dart';
8+
import '../ios/xcodeproj.dart';
9+
import '../xcode_project.dart';
10+
11+
// Xcode 14.3 changed the readlink symlink behavior to be relative from the script working directory, instead of the
12+
// relative path of the symlink. The -f flag returns the original "--canonicalize" behavior the CocoaPods script relies on.
13+
// This has been fixed upstream in CocoaPods, but migrate a copy of their workaround so users don't need to update.
14+
//
15+
// See https://github.com/flutter/flutter/issues/123890#issuecomment-1494825976.
16+
class CocoaPodsScriptReadlink extends ProjectMigrator {
17+
CocoaPodsScriptReadlink(
18+
XcodeBasedProject project,
19+
XcodeProjectInterpreter xcodeProjectInterpreter,
20+
super.logger,
21+
) : _podRunnerFrameworksScript = project.podRunnerFrameworksScript,
22+
_xcodeProjectInterpreter = xcodeProjectInterpreter;
23+
24+
final File _podRunnerFrameworksScript;
25+
final XcodeProjectInterpreter _xcodeProjectInterpreter;
26+
27+
@override
28+
void migrate() {
29+
if (!_podRunnerFrameworksScript.existsSync()) {
30+
logger.printTrace('CocoaPods Pods-Runner-frameworks.sh script not found, skipping "readlink -f" workaround.');
31+
return;
32+
}
33+
34+
final Version? version = _xcodeProjectInterpreter.version;
35+
36+
// If Xcode not installed or less than 14.3 with readlink behavior change, skip this migration.
37+
if (version == null || version < Version(14, 3, 0)) {
38+
logger.printTrace('Detected Xcode version is $version, below 14.3, skipping "readlink -f" workaround.');
39+
return;
40+
}
41+
42+
processFileLines(_podRunnerFrameworksScript);
43+
}
44+
45+
@override
46+
String? migrateLine(String line) {
47+
const String originalReadLinkLine = r'source="$(readlink "${source}")"';
48+
const String replacementReadLinkLine = r'source="$(readlink -f "${source}")"';
49+
50+
return line.replaceAll(originalReadLinkLine, replacementReadLinkLine);
51+
}
52+
}

‎packages/flutter_tools/lib/src/xcode_project.dart‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,13 @@ abstract class XcodeBasedProject extends FlutterProjectPlatform {
8989

9090
/// The CocoaPods 'Manifest.lock'.
9191
File get podManifestLock => hostAppRoot.childDirectory('Pods').childFile('Manifest.lock');
92+
93+
/// The CocoaPods generated 'Pods-Runner-frameworks.sh'.
94+
File get podRunnerFrameworksScript => hostAppRoot
95+
.childDirectory('Pods')
96+
.childDirectory('Target Support Files')
97+
.childDirectory('Pods-Runner')
98+
.childFile('Pods-Runner-frameworks.sh');
9299
}
93100

94101
/// Represents the iOS sub-project of a Flutter project.

‎packages/flutter_tools/test/general.shard/ios/ios_project_migration_test.dart‎

Lines changed: 105 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,16 @@ import 'package:file/file.dart';
66
import 'package:file/memory.dart';
77
import 'package:flutter_tools/src/base/logger.dart';
88
import 'package:flutter_tools/src/base/project_migrator.dart';
9+
import 'package:flutter_tools/src/base/version.dart';
910
import 'package:flutter_tools/src/ios/migrations/host_app_info_plist_migration.dart';
1011
import 'package:flutter_tools/src/ios/migrations/ios_deployment_target_migration.dart';
1112
import 'package:flutter_tools/src/ios/migrations/project_base_configuration_migration.dart';
1213
import 'package:flutter_tools/src/ios/migrations/project_build_location_migration.dart';
1314
import 'package:flutter_tools/src/ios/migrations/remove_bitcode_migration.dart';
1415
import 'package:flutter_tools/src/ios/migrations/remove_framework_link_and_embedding_migration.dart';
1516
import 'package:flutter_tools/src/ios/migrations/xcode_build_system_migration.dart';
17+
import 'package:flutter_tools/src/ios/xcodeproj.dart';
18+
import 'package:flutter_tools/src/migrations/cocoapods_script_symlink.dart';
1619
import 'package:flutter_tools/src/migrations/xcode_project_object_version_migration.dart';
1720
import 'package:flutter_tools/src/migrations/xcode_script_build_phase_migration.dart';
1821
import 'package:flutter_tools/src/migrations/xcode_thin_binary_build_phase_input_paths_migration.dart';
@@ -21,6 +24,7 @@ import 'package:flutter_tools/src/xcode_project.dart';
2124
import 'package:test/fake.dart';
2225

2326
import '../../src/common.dart';
27+
import '../../src/fake_process_manager.dart';
2428

2529
void main () {
2630
group('iOS migration', () {
@@ -901,6 +905,104 @@ platform :ios, '11.0'
901905
expect('Disabling deprecated bitcode Xcode build setting'.allMatches(testLogger.warningText).length, 1);
902906
});
903907
});
908+
909+
group('CocoaPods script readlink', () {
910+
late MemoryFileSystem memoryFileSystem;
911+
late BufferLogger testLogger;
912+
late FakeIosProject project;
913+
late File podRunnerFrameworksScript;
914+
late ProcessManager processManager;
915+
late XcodeProjectInterpreter xcode143ProjectInterpreter;
916+
917+
setUp(() {
918+
memoryFileSystem = MemoryFileSystem();
919+
podRunnerFrameworksScript = memoryFileSystem.file('Pods-Runner-frameworks.sh');
920+
testLogger = BufferLogger.test();
921+
project = FakeIosProject();
922+
processManager = FakeProcessManager.any();
923+
xcode143ProjectInterpreter = XcodeProjectInterpreter.test(processManager: processManager, version: Version(14, 3, 0));
924+
project.podRunnerFrameworksScript = podRunnerFrameworksScript;
925+
});
926+
927+
testWithoutContext('skipped if files are missing', () {
928+
final CocoaPodsScriptReadlink iosProjectMigration = CocoaPodsScriptReadlink(
929+
project,
930+
xcode143ProjectInterpreter,
931+
testLogger,
932+
);
933+
iosProjectMigration.migrate();
934+
expect(podRunnerFrameworksScript.existsSync(), isFalse);
935+
936+
expect(testLogger.traceText, contains('CocoaPods Pods-Runner-frameworks.sh script not found'));
937+
expect(testLogger.statusText, isEmpty);
938+
});
939+
940+
testWithoutContext('skipped if nothing to upgrade', () {
941+
const String contents = r'''
942+
if [ -L "${source}" ]; then
943+
echo "Symlinked..."
944+
source="$(readlink -f "${source}")"
945+
fi''';
946+
podRunnerFrameworksScript.writeAsStringSync(contents);
947+
948+
final CocoaPodsScriptReadlink iosProjectMigration = CocoaPodsScriptReadlink(
949+
project,
950+
xcode143ProjectInterpreter,
951+
testLogger,
952+
);
953+
iosProjectMigration.migrate();
954+
expect(podRunnerFrameworksScript.existsSync(), isTrue);
955+
expect(testLogger.traceText, isEmpty);
956+
expect(testLogger.statusText, isEmpty);
957+
});
958+
959+
testWithoutContext('skipped if Xcode version below 14.3', () {
960+
const String contents = r'''
961+
if [ -L "${source}" ]; then
962+
echo "Symlinked..."
963+
source="$(readlink "${source}")"
964+
fi''';
965+
podRunnerFrameworksScript.writeAsStringSync(contents);
966+
967+
final XcodeProjectInterpreter xcode142ProjectInterpreter = XcodeProjectInterpreter.test(
968+
processManager: processManager,
969+
version: Version(14, 2, 0),
970+
);
971+
972+
final CocoaPodsScriptReadlink iosProjectMigration = CocoaPodsScriptReadlink(
973+
project,
974+
xcode142ProjectInterpreter,
975+
testLogger,
976+
);
977+
iosProjectMigration.migrate();
978+
expect(podRunnerFrameworksScript.existsSync(), isTrue);
979+
expect(testLogger.traceText, contains('Detected Xcode version is 14.2.0, below 14.3, skipping "readlink -f" workaround'));
980+
expect(testLogger.statusText, isEmpty);
981+
});
982+
983+
testWithoutContext('Xcode project is migrated', () {
984+
const String contents = r'''
985+
if [ -L "${source}" ]; then
986+
echo "Symlinked..."
987+
source="$(readlink "${source}")"
988+
fi''';
989+
podRunnerFrameworksScript.writeAsStringSync(contents);
990+
991+
final CocoaPodsScriptReadlink iosProjectMigration = CocoaPodsScriptReadlink(
992+
project,
993+
xcode143ProjectInterpreter,
994+
testLogger,
995+
);
996+
iosProjectMigration.migrate();
997+
expect(podRunnerFrameworksScript.readAsStringSync(), r'''
998+
if [ -L "${source}" ]; then
999+
echo "Symlinked..."
1000+
source="$(readlink -f "${source}")"
1001+
fi
1002+
''');
1003+
expect(testLogger.statusText, contains('Upgrading Pods-Runner-frameworks.sh'));
1004+
});
1005+
});
9041006
});
9051007

9061008
group('update Xcode script build phase', () {
@@ -1134,6 +1236,9 @@ class FakeIosProject extends Fake implements IosProject {
11341236

11351237
@override
11361238
File podfile = MemoryFileSystem.test().file('Podfile');
1239+
1240+
@override
1241+
File podRunnerFrameworksScript = MemoryFileSystem.test().file('podRunnerFrameworksScript');
11371242
}
11381243

11391244
class FakeIOSMigrator extends ProjectMigrator {

‎packages/flutter_tools/test/general.shard/macos/cocoapods_test.dart‎

Lines changed: 50 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import 'package:file/file.dart';
66
import 'package:file/memory.dart';
77
import 'package:flutter_tools/src/base/logger.dart';
88
import 'package:flutter_tools/src/base/platform.dart';
9+
import 'package:flutter_tools/src/base/version.dart';
910
import 'package:flutter_tools/src/build_info.dart';
1011
import 'package:flutter_tools/src/cache.dart';
1112
import 'package:flutter_tools/src/flutter_plugins.dart';
@@ -740,6 +741,55 @@ Note: as of CocoaPods 1.0, `pod repo update` does not happen on `pod install` by
740741
);
741742
expect(didInstall, isTrue);
742743
expect(fakeProcessManager, hasNoRemainingExpectations);
744+
expect(logger.traceText, contains('CocoaPods Pods-Runner-frameworks.sh script not found'));
745+
});
746+
747+
testUsingContext('runs CocoaPods Pod runner script migrator', () async {
748+
final FlutterProject projectUnderTest = setupProjectUnderTest();
749+
pretendPodIsInstalled();
750+
pretendPodVersionIs('100.0.0');
751+
projectUnderTest.ios.podfile
752+
..createSync()
753+
..writeAsStringSync('Existing Podfile');
754+
projectUnderTest.ios.podfileLock
755+
..createSync()
756+
..writeAsStringSync('Existing lock file.');
757+
projectUnderTest.ios.podManifestLock
758+
..createSync(recursive: true)
759+
..writeAsStringSync('Existing lock file.');
760+
projectUnderTest.ios.podRunnerFrameworksScript
761+
..createSync(recursive: true)
762+
..writeAsStringSync(r'source="$(readlink "${source}")"');
763+
764+
fakeProcessManager.addCommands(const <FakeCommand>[
765+
FakeCommand(
766+
command: <String>['pod', 'install', '--verbose'],
767+
workingDirectory: 'project/ios',
768+
environment: <String, String>{'COCOAPODS_DISABLE_STATS': 'true', 'LANG': 'en_US.UTF-8'},
769+
),
770+
FakeCommand(
771+
command: <String>['touch', 'project/ios/Podfile.lock'],
772+
),
773+
]);
774+
775+
final CocoaPods cocoaPodsUnderTestXcode143 = CocoaPods(
776+
fileSystem: fileSystem,
777+
processManager: fakeProcessManager,
778+
logger: logger,
779+
platform: FakePlatform(operatingSystem: 'macos'),
780+
xcodeProjectInterpreter: XcodeProjectInterpreter.test(processManager: fakeProcessManager, version: Version(14, 3, 0)),
781+
usage: usage,
782+
);
783+
784+
final bool didInstall = await cocoaPodsUnderTestXcode143.processPods(
785+
xcodeProject: projectUnderTest.ios,
786+
buildMode: BuildMode.debug,
787+
);
788+
expect(didInstall, isTrue);
789+
expect(fakeProcessManager, hasNoRemainingExpectations);
790+
// Now has readlink -f flag.
791+
expect(projectUnderTest.ios.podRunnerFrameworksScript.readAsStringSync(), contains(r'source="$(readlink -f "${source}")"'));
792+
expect(logger.statusText, contains('Upgrading Pods-Runner-frameworks.sh'));
743793
});
744794

745795
testUsingContext('runs pod install, if Podfile.lock is older than Podfile', () async {

0 commit comments

Comments
 (0)