Skip to content

Commit

Permalink
Add a comment if file has no bulid settings
Browse files Browse the repository at this point in the history
Change 'common' to 'shared' in ReadMe
  • Loading branch information
dempseyatgithub committed Jan 31, 2015
1 parent 6821e39 commit 6e9c573
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
9 changes: 9 additions & 0 deletions BuildSettingExtractor/BuildSettingExtractor.m
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,15 @@ - (void)writeConfigFilesToDestinationFolder:(NSURL *)destinationURL {
configFileString = [configFileString stringByAppendingString:includeDirective];
}

// If there are no settings at all, add a comment that the lack of settings is on purpose
if ([settings isEqualToString:@""]) {
settings = [settings stringByAppendingString:@"//********************************************//\n"];
settings = [settings stringByAppendingString:@"//* Currently no build settings in this file *//\n"];
settings = [settings stringByAppendingString:@"//********************************************//"];

;
}

configFileString = [configFileString stringByAppendingString:settings];

// Trim whitespace and newlines
Expand Down
2 changes: 1 addition & 1 deletion ReadMe.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ Using the app:

- BuildSettingExtractor does not alter the original Xcode project file.
- BuildSettingExtractor does not update existing xcconfig files, it does a one-time extraction.
- BuildSettingExtractor does not hoist common target build settings to the project level.
- BuildSettingExtractor does not hoist shared target build settings to the project level.
- Do not taunt BuildSettingExtractor.

**Version History**
Expand Down

0 comments on commit 6e9c573

Please sign in to comment.