Commit 845c1b7f authored by Chris Bracken's avatar Chris Bracken Committed by GitHub

Re-land Lazily initialise Xcode installation status (#10952)

This reverts commit 3e265e9e.

Rather than pre-compute Xcode install path, version, and EULA status,
compute and cache on demand.
parent 3e265e9e
...@@ -80,54 +80,58 @@ class IMobileDevice { ...@@ -80,54 +80,58 @@ class IMobileDevice {
} }
class Xcode { class Xcode {
Xcode() { bool get isInstalledAndMeetsVersionCheck => isInstalled && xcodeVersionSatisfactory;
_eulaSigned = false;
try { String _xcodeSelectPath;
_xcodeSelectPath = runSync(<String>['xcode-select', '--print-path'])?.trim(); String get xcodeSelectPath {
if (_xcodeSelectPath == null || _xcodeSelectPath.isEmpty) { if (_xcodeSelectPath == null) {
_isInstalled = false; try {
return; _xcodeSelectPath = runSync(<String>['/usr/bin/xcode-select', '--print-path'])?.trim();
} } on ProcessException {
_isInstalled = true; // Ignore: return null below.
_xcodeVersionText = runSync(<String>['xcodebuild', '-version']).replaceAll('\n', ', ');
if (!xcodeVersionRegex.hasMatch(_xcodeVersionText)) {
_isInstalled = false;
} else {
try {
final ProcessResult result = processManager.runSync(<String>['/usr/bin/xcrun', 'clang']);
if (result.stdout != null && result.stdout.contains('license'))
_eulaSigned = false;
else if (result.stderr != null && result.stderr.contains('license'))
_eulaSigned = false;
else
_eulaSigned = true;
} catch (error) {
_eulaSigned = false;
}
} }
} catch (error) {
_isInstalled = false;
} }
return _xcodeSelectPath;
} }
bool get isInstalledAndMeetsVersionCheck => isInstalled && xcodeVersionSatisfactory; bool get isInstalled {
if (xcodeSelectPath == null || xcodeSelectPath.isEmpty)
String _xcodeSelectPath; return false;
String get xcodeSelectPath => _xcodeSelectPath; if (xcodeVersionText == null || !xcodeVersionRegex.hasMatch(xcodeVersionText))
return false;
bool _isInstalled; return true;
bool get isInstalled => _isInstalled; }
bool _eulaSigned; bool _eulaSigned;
/// Has the EULA been signed? /// Has the EULA been signed?
bool get eulaSigned => _eulaSigned; bool get eulaSigned {
if (_eulaSigned == null) {
try {
final ProcessResult result = processManager.runSync(<String>['/usr/bin/xcrun', 'clang']);
if (result.stdout != null && result.stdout.contains('license'))
_eulaSigned = false;
else if (result.stderr != null && result.stderr.contains('license'))
_eulaSigned = false;
else
_eulaSigned = true;
} on ProcessException {
_eulaSigned = false;
}
}
return _eulaSigned;
}
String _xcodeVersionText; String _xcodeVersionText;
String get xcodeVersionText => _xcodeVersionText; String get xcodeVersionText {
if (_xcodeVersionText == null) {
try {
_xcodeVersionText = runSync(<String>['/usr/bin/xcodebuild', '-version']).replaceAll('\n', ', ');
} on ProcessException {
// Ignore: return null below.
}
}
return _xcodeVersionText;
}
int _xcodeMajorVersion; int _xcodeMajorVersion;
int get xcodeMajorVersion => _xcodeMajorVersion; int get xcodeMajorVersion => _xcodeMajorVersion;
...@@ -138,7 +142,7 @@ class Xcode { ...@@ -138,7 +142,7 @@ class Xcode {
final RegExp xcodeVersionRegex = new RegExp(r'Xcode ([0-9.]+)'); final RegExp xcodeVersionRegex = new RegExp(r'Xcode ([0-9.]+)');
bool get xcodeVersionSatisfactory { bool get xcodeVersionSatisfactory {
if (!xcodeVersionRegex.hasMatch(xcodeVersionText)) if (xcodeVersionText == null || !xcodeVersionRegex.hasMatch(xcodeVersionText))
return false; return false;
final String version = xcodeVersionRegex.firstMatch(xcodeVersionText).group(1); final String version = xcodeVersionRegex.firstMatch(xcodeVersionText).group(1);
......
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