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

SR-11411: Update GettingStarted.md #2657

Merged
merged 1 commit into from Feb 13, 2020
Merged

Conversation

spevans
Copy link
Collaborator

@spevans spevans commented Feb 11, 2020

Update the document to reflect current cmake usage.

@compnerd @millenomi This is based on the commands I use, do you have anything to add/alter?

@compnerd
Copy link
Collaborator

Thanks for adding the blurb on running individual tests, thats extremely valuable. LGTM (might be nice to wait a day or so to see if @millenomi chimes in)

@spevans
Copy link
Collaborator Author

spevans commented Feb 13, 2020

@swift-ci test and merge

@compnerd
Copy link
Collaborator

macOS builds are known to be broken due to the dysmutil issue. Im going to merge this as is. If you know how to convert the build from dwarf-with-dsym to dwarf, would you mind putting up a PR to switch modes?

@compnerd compnerd merged commit c326918 into apple:master Feb 13, 2020
@spevans
Copy link
Collaborator Author

spevans commented Feb 13, 2020

The macOS build is actually pointless in this repo as according to build-scrip-impl:

if [[ ${host} == "macosx"* ]]; then
    echo "Skipping Foundation on OS X -- use the Xcode project instead"
    continue
fi

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