Clarify GettingStarted.md for macOS #86008
Conversation
|
@swift-ci Please smoke test |
| 1. Install Xcode. The minimum required version is specified in the node | ||
| information on <https://ci.swift.org>, may change frequently, and is often | ||
| a beta release. | ||
| 1. Install [Ninja][] and [Sccache][]: |
There was a problem hiding this comment.
out of curiosity, why is the Ninja stuff being removed here?
There was a problem hiding this comment.
Because the build process checks out and builds ninja for you, there's no need to do it yourself. You can if you want, though.
There was a problem hiding this comment.
And neither sccache nor homebrew is a requirement either. You just need Xcode and CMake.
There was a problem hiding this comment.
Because the build process checks out and builds ninja for you
Does it by default? I don’t have a ninja clone in any of my checkouts.
There was a problem hiding this comment.
You just need Xcode and CMake.
CMake was previously removed from this document (in #83063 by @MaxDesiatov).
As with ninja, CMake will be built from source, but I did encounter two issues:
-
utils/build-script --cleanfailed to rebuild CMake (as far as I remember). -
utils/run-testcouldn't find CMake (I had to manually add it to the PATH).
There was a problem hiding this comment.
Also, ninja is the preferred tool for incremental builds AFAIC. I don’t think the build script installs it your PATH.
| This step is not required, but it speeds up rebuilds by caching build artifacts. | ||
| [Xcode]: https://developer.apple.com/xcode/resources/ | ||
| [Ninja]: https://ninja-build.org |
There was a problem hiding this comment.
nit: the link might still be useful to include somewhere even if the tool is installed automatically
There was a problem hiding this comment.
We had certain ninja compatibility problem's too, so I'd prefer we don't mislead users telling them to install a wrong version.
MaxDesiatov
left a comment
MaxDesiatov
left a comment
There was a problem hiding this comment.
The CMake installation step needs to be removed as Homebrew doesn't supply CMake 3.x and CMake 4.x is not compatible with our build system. Additionally, build-script will build and cache a compatible version for you already.
6 participants
No description provided.