-
Notifications
You must be signed in to change notification settings - Fork 231
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
Minor fixes for XCode 9.1 #207
Conversation
Please don't merge this. This is a placeholder until Xcode 9.1 is released. Tested with Xcode 9.1-beta so far. |
Source/Shared/BPXCTestFile.m
Outdated
testHostPath = [testHostPath stringByAppendingString:appNameWithExtension]; | ||
testBundlePath = [testBundlePath stringByReplacingOccurrencesOfString:TESTROOT withString:temp]; | ||
if ([testHostPath rangeOfString:PLATFORMS].location != NSNotFound) { | ||
|
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.
any handling here?
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.
This is actually from a different PR... I'm not sure how I ended up committing it here. I'll finish it and piggyback my fix on this PR.
remember to update read.me |
Don't suppose this can be merged now? I'm trying to use Bluepill but Xcode 9.1 is the problem I think :( |
I'm just waiting for the tests to finish on this one and I'll merge it and move what's there to the |
@@ -65,13 +65,15 @@ @implementation BPApp | |||
|
|||
+ (NSArray <BPXCTestFile *>*)testsFromXCTestRunDict:(NSDictionary *)xcTestRunDict | |||
andXCTestRunPath:(NSString *)xcTestRunPath | |||
andXcodePath: (NSString *)xcodePath |
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.
extra space
withError:(NSError *__autoreleasing *)error { | ||
|
||
NSMutableArray<BPXCTestFile *> *allTests = [[NSMutableArray alloc] init]; | ||
NSUInteger errorCount = 0; | ||
for (NSString *key in xcTestRunDict) { | ||
BPXCTestFile *xcTestFile = [BPXCTestFile BPXCTestFileFromDictionary:[xcTestRunDict objectForKey:key] | ||
withTestRoot:[xcTestRunPath stringByDeletingLastPathComponent] | ||
andXcodePath:xcodePath | ||
andError:error]; |
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.
indent is wrong
@@ -98,6 +100,7 @@ + (instancetype)appWithConfig:(BPConfiguration *)config | |||
[BPUtils printInfo:INFO withString:@"Using xctestrun configuration"]; | |||
NSArray<BPXCTestFile *> *loadedTests = [BPApp testsFromXCTestRunDict:config.xcTestRunDict | |||
andXCTestRunPath:config.xcTestRunPath | |||
andXcodePath:config.xcodePath |
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.
indent
@@ -73,7 +73,8 @@ int main(int argc, char * argv[]) { | |||
|
|||
//Check if Bluepill compile time Xcode version is matched with Bluepill runtime Xcode version | |||
//Senario to prevent: Bluepill is compiled with Xcode 8, but runs with host installed with Xcode 9 | |||
if ([[BPUtils getXcodeRuntimeVersion] isEqualToString:@XCODE_VERSION]) { | |||
//Only compare major version Exg. 9.1 == 9.1 |
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.
major and minor version (not comparing patch version)
@@ -73,7 +73,8 @@ int main(int argc, char * argv[]) { | |||
|
|||
//Check if Bluepill compile time Xcode version is matched with Bluepill runtime Xcode version | |||
//Senario to prevent: Bluepill is compiled with Xcode 8, but runs with host installed with Xcode 9 | |||
if ([[BPUtils getXcodeRuntimeVersion] isEqualToString:@XCODE_VERSION]) { | |||
//Only compare major version Exg. 9.1 == 9.1 | |||
if ([[[BPUtils getXcodeRuntimeVersion] substringToIndex:3] isEqualToString:[@XCODE_VERSION substringToIndex:3]]) { | |||
printf("Bluepill runtime version and compile time version are matched: %s\n", XCODE_VERSION); | |||
} else { | |||
fprintf(stderr, "ERROR: Bluepill runtime version %s and compile time version %s are mismatched\n", [[BPUtils getXcodeRuntimeVersion] UTF8String], XCODE_VERSION); |
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.
we should also update the error msg to only use substring to index 3.. otherwise people would think you need the exact patch version
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.
Go!
* Minor fixes for XCode 9.1 * Only check runtime major iOS version number in testRunUITest * Bump Xcode version for circleci * Bump runtime versions and Xcode version * only compare major Xcode version for bluepill compile time and bluepill runtime * Wire in the XcodePath to fix the code for replacing __PLATFORMS__ * Update README
Mostly cosmetic and bump to iOS version