Skip to content
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

[Xcodeproj] Add support for automatic project generation #1604

Merged
merged 1 commit into from
Jun 16, 2018

Conversation

aciidgh
Copy link
Member

@aciidgh aciidgh commented Jun 15, 2018

This adds a --watch option to generate-xcodeproj subcommand. SwiftPM
will do the necessary configuration and exec to the watchman-make tool.
If the tool can't be found on the system, it'll emit help information to
install it.

rdar://problem/32390792 Add support for automatically maintaining the generated Xcode project

This adds a `--watch` option to generate-xcodeproj subcommand. SwiftPM
will do the necessary configuration and exec to the watchman-make tool.
If the tool can't be found on the system, it'll emit help information to
install it.

<rdar://problem/32390792> Add support for automatically maintaining the generated Xcode project
@aciidgh aciidgh requested a review from hartbit June 15, 2018 23:48
@aciidgh
Copy link
Member Author

aciidgh commented Jun 15, 2018

@swift-ci please smoke test

@aciidgh aciidgh merged commit b0df140 into apple:master Jun 16, 2018
@aciidgh aciidgh deleted the watchman branch June 16, 2018 00:41
@@ -195,7 +195,7 @@ public final class Process: ObjectIdentifierProtocol {
/// Returns true if the given program is present and executable in search path.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably want to update to documentation to say a path is returned.

option: "--legacy-scheme-generator", kind: Bool.self,
usage: "Use the legacy scheme generator"),
generateXcodeParser.add(
option: "--watch", kind: Bool.self,
usage: "Watch the filesystem and autogenerate the Xcode project if needed"),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be worth being more specific?

Watch for changes to the Package manifest to regenerate the Xcode project

name: "org.swift.diags.watchman-missing",
description: {
$0 <<< "this feature requires 'watchman' to work"
$0 <<< "\n\n installation instructions for 'watchman' are available at https://facebook.github.io/watchman/docs/install.html#buildinstall"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the double-newlines?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks a bit "pretty".


set -eu

swift package generate-xcodeproj
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we also want a test that makes such that we pass --xcconfig-overrides when necessary?

aciidgh added a commit to aciidgh/swift-package-manager that referenced this pull request Jun 19, 2018
aciidgh added a commit that referenced this pull request Jun 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants