Skip to content

Conversation

@slavapestov
Copy link
Contributor

No description provided.

@slavapestov
Copy link
Contributor Author

@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][]:
Copy link
Contributor

Choose a reason for hiding this comment

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

out of curiosity, why is the Ninja stuff being removed here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

And neither sccache nor homebrew is a requirement either. You just need Xcode and CMake.

Copy link
Collaborator

Choose a reason for hiding this comment

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

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.

Copy link
Contributor

Choose a reason for hiding this comment

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

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 --clean failed to rebuild CMake (as far as I remember).

  • utils/run-test couldn't find CMake (I had to manually add it to the 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
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: the link might still be useful to include somewhere even if the tool is installed automatically

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.

5 participants