Skip to content

Migrate building SwiftSyntax to swift_build_support #27871

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

Merged
merged 2 commits into from
Oct 29, 2019

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Oct 24, 2019

This migrates the build infrastructure of SwiftSyntax to swift_build_support. It goes hand-in-hand with swiftlang/swift-syntax#160.

@ahoppen ahoppen force-pushed the swiftsyntax_build_support branch 2 times, most recently from 9f38bf5 to aff7a5c Compare October 25, 2019 19:21
@ahoppen
Copy link
Member Author

ahoppen commented Oct 25, 2019

swiftlang/swift-syntax#160

@swift-ci Please smoke test

@ahoppen
Copy link
Member Author

ahoppen commented Oct 25, 2019

swiftlang/swift-syntax#160

preset=buildbot_swiftsyntax_macos

@swift-ci Please test with preset macOS Platform

@ahoppen
Copy link
Member Author

ahoppen commented Oct 25, 2019

Great! The preset caught the generated files in SwiftSyntax being out-of-date. Let’s see if it passes now that the files have been updated.

swiftlang/swift-syntax#160

preset=buildbot_swiftsyntax_macos

@swift-ci Please test with preset macOS Platform

@ahoppen ahoppen force-pushed the swiftsyntax_build_support branch from aff7a5c to 46501b8 Compare October 25, 2019 22:58
@ahoppen
Copy link
Member Author

ahoppen commented Oct 25, 2019

swiftlang/swift-syntax#160
preset=buildbot_swiftsyntax_macos
@swift-ci Please test with preset macOS Platform

@ahoppen
Copy link
Member Author

ahoppen commented Oct 25, 2019

swiftlang/swift-syntax#160
@swift-ci Please test

@swiftlang swiftlang deleted a comment from swift-ci Oct 25, 2019
@swiftlang swiftlang deleted a comment from swift-ci Oct 25, 2019
@ahoppen
Copy link
Member Author

ahoppen commented Oct 26, 2019

SwiftSyntax preset passed.

Let’s try if SouceKit-LSP still builds since I touched its build script as well.

swiftlang/swift-syntax#160
preset=buildbot_sourcekitlsp_macos
@swift-ci Please test with preset macOS Platform

@ahoppen
Copy link
Member Author

ahoppen commented Oct 26, 2019

Building SourceKit-LSP also passed. https://ci.swift.org/job/swift-PR-osx-preset/60/

Let’s finally test the stress tester.

swiftlang/swift-syntax#160
preset= buildbot_skstresstester_macos
@swift-ci Please test with preset macOS Platform

@ahoppen
Copy link
Member Author

ahoppen commented Oct 26, 2019

Mistakingly added a space after preset= in the above comment. Let’s run again without the space.

swiftlang/swift-syntax#160
preset=buildbot_skstresstester_macos
@swift-ci Please test with preset macOS Platform

@ahoppen
Copy link
Member Author

ahoppen commented Oct 27, 2019

swiftlang/swift-syntax#160
preset=buildbot_skstresstester_macos
@swift-ci Please test with preset macOS Platform

@@ -101,8 +97,6 @@ KNOWN_SETTINGS=(
installable-package "" "the path to the archive of the installation directory"
test-installable-package "" "whether to run post-packaging tests on the produced package"
reconfigure "" "force a CMake configuration run even if CMakeCache.txt already exists"
build-libparser-only "" "only build libSwiftSyntaxParser"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are we removing --build-libparser-only? Isn't is still useful to be able to build the parser library without building the rest of the compiler? CC @akyrtzi

Copy link
Contributor

Choose a reason for hiding this comment

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

This has been superseded by running utils/build-parser-lib

@aciidgh
Copy link
Contributor

aciidgh commented Oct 29, 2019

Cool! Is this ready to land?

@ahoppen ahoppen merged commit 282d0de into swiftlang:master Oct 29, 2019
@ahoppen ahoppen deleted the swiftsyntax_build_support branch October 30, 2019 18:51
dan-zheng added a commit to dan-zheng/swift that referenced this pull request Nov 5, 2019
marcrasi pushed a commit that referenced this pull request Nov 5, 2019
keith added a commit to keith/swift that referenced this pull request Mar 17, 2022
It seems that build-libparser-only hasn't existed since 2019 and this
change swiftlang#27871
keith added a commit that referenced this pull request Mar 18, 2022
It seems that build-libparser-only hasn't existed since 2019 and this
change #27871
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.

4 participants