Commit df946dd8 authored by Chinmay Garde's avatar Chinmay Garde

iOS: Treat the initial Info.plist string as a mustache template.

Use the project name to initialize the bundle name and identifier
parent 5009f44a
......@@ -10,6 +10,7 @@ import 'package:xml/xml.dart' as xml;
import 'artifacts.dart';
import 'build_configuration.dart';
import 'ios/plist_utils.dart';
abstract class ApplicationPackage {
/// Path to the actual apk or bundle.
......@@ -81,13 +82,25 @@ class AndroidApk extends ApplicationPackage {
}
class IOSApp extends ApplicationPackage {
static const String _defaultId = 'io.flutter.runner.Runner';
static const String _defaultPath = 'ios/.generated';
IOSApp({
String localPath: _defaultPath,
String id: _defaultId
}) : super(localPath: localPath, id: id);
String iosProjectDir,
String iosProjectBundleId
}) : super(localPath: iosProjectDir, id: iosProjectBundleId);
factory IOSApp.fromBuildConfiguration(BuildConfiguration config) {
if (getCurrentHostPlatform() != HostPlatform.mac) {
return null;
}
String plistPath = path.join("ios", "Info.plist");
String id = plistValueForKey(plistPath, kCFBundleIdentifierKey);
if (id == "") {
return null;
}
String projectDir = path.join("ios", ".generated");
return new IOSApp(iosProjectDir: projectDir, iosProjectBundleId: id);
}
}
class ApplicationPackageStore {
......@@ -138,12 +151,12 @@ class ApplicationPackageStore {
case TargetPlatform.iOS:
assert(iOS == null);
iOS = new IOSApp();
iOS = new IOSApp.fromBuildConfiguration(config);
break;
case TargetPlatform.iOSSimulator:
assert(iOSSimulator == null);
iOSSimulator = new IOSApp();
iOSSimulator = new IOSApp.fromBuildConfiguration(config);
break;
case TargetPlatform.mac:
......
// Copyright 2016 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
import 'package:path/path.dart' as path;
import '../base/process.dart';
const String kCFBundleIdentifierKey = "CFBundleIdentifier";
String plistValueForKey(String plistFilePath, String plistKey) {
// TODO(chinmaygarde): For now, we only need to read from plist files on a
// mac host. If this changes, we will need our own Dart plist reader.
// Don't use PlistBuddy since that is not guaranteed to be installed.
// 'defaults' requires the path to be absolute and without the 'plist'
// extension.
String normalizedPlistPath = path.withoutExtension(path.absolute(plistFilePath));
try {
return runCheckedSync([
'/usr/bin/defaults',
'read',
normalizedPlistPath,
plistKey]).trim();
} catch (error) {
return "";
}
}
......@@ -91,18 +91,6 @@ Future<bool> _inflateXcodeArchive(String directory, List<int> archiveBytes) asyn
return true;
}
void _writeUserEditableFilesIfNecessary(String directory) {
iosTemplateFiles.forEach((String filePath, String contents) {
File file = new File(filePath);
if (!file.existsSync()) {
file.parent.createSync(recursive: true);
file.writeAsStringSync(contents);
printStatus('Created $filePath.');
}
});
}
void _setupXcodeProjXcconfig(String filePath) {
StringBuffer localsBuffer = new StringBuffer();
......@@ -160,19 +148,15 @@ Future<int> setupXcodeProjectHarness() async {
return 1;
}
// Step 3: Setup default user editable files if this is the first run of
// the init command.
_writeUserEditableFilesIfNecessary(iosFilesPath);
// Step 4: Populate the Local.xcconfig with project specific paths
// Step 3: Populate the Local.xcconfig with project specific paths
_setupXcodeProjXcconfig(path.join(xcodeprojPath, 'Local.xcconfig'));
// Step 5: Write the REVISION file
// Step 4: Write the REVISION file
File revisionFile = new File(path.join(xcodeprojPath, 'REVISION'));
revisionFile.createSync();
revisionFile.writeAsStringSync(ArtifactStore.engineRevision);
// Step 6: Tell the user the location of the generated project.
// Step 5: Tell the user the location of the generated project.
printStatus('Xcode project created at $xcodeprojPath/.');
printStatus('User editable settings are in $iosFilesPath/.');
......@@ -189,11 +173,11 @@ final String _infoPlistInitialContents = '''
<key>CFBundleExecutable</key>
<string>Runner</string>
<key>CFBundleIdentifier</key>
<string>io.flutter.runner.Runner</string>
<string>com.example.{{projectName}}</string>
<key>CFBundleInfoDictionaryVersion</key>
<string>6.0</string>
<key>CFBundleName</key>
<string>Flutter</string>
<string>{{projectName}}</string>
<key>CFBundlePackageType</key>
<string>APPL</string>
<key>CFBundleShortVersionString</key>
......
......@@ -14,8 +14,8 @@ import 'package:mockito/mockito.dart';
class MockApplicationPackageStore extends ApplicationPackageStore {
MockApplicationPackageStore() : super(
android: new AndroidApk(localPath: '/mock/path/to/android/SkyShell.apk'),
iOS: new IOSApp(localPath: '/mock/path/to/iOS/SkyShell.app'),
iOSSimulator: new IOSApp(localPath: '/mock/path/to/iOSSimulator/SkyShell.app'));
iOS: new IOSApp(iosProjectDir: '/mock/path/to/iOS/SkyShell.app'),
iOSSimulator: new IOSApp(iosProjectDir: '/mock/path/to/iOSSimulator/SkyShell.app'));
}
class MockCompiler extends Mock implements Compiler {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment