Skip to content

Conversation

@briancroom
Copy link
Contributor

What's in this pull request?

Significant progress is being made towards getting SourceKit buildable on Linux. This adds a temporary switch to build-script-impl for enabling SourceKit in the CMake build, to facilitate others who are interested in following this work.

cc @akyrtzi @benlangmuir @gribozavr

Resolved bug number: (SR-1676)


Before merging this pull request to apple/swift repository:

  • Test pull request on Swift continuous integration.

Triggering Swift CI

The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:

Smoke Testing

Platform Comment
All supported platforms @swift-ci Please smoke test
All supported platforms @swift-ci Please smoke test and merge
OS X platform @swift-ci Please smoke test OS X platform
Linux platform @swift-ci Please smoke test Linux platform

Validation Testing

Platform Comment
All supported platforms @swift-ci Please test
All supported platforms @swift-ci Please test and merge
OS X platform @swift-ci Please test OS X platform
OS X platform @swift-ci Please benchmark
Linux platform @swift-ci Please test Linux platform

Lint Testing

Language Comment
Python @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@akyrtzi
Copy link
Contributor

akyrtzi commented Jun 23, 2016

Note that one can use --extra-cmake-options="-DSWIFT_BUILD_SOURCEKIT:BOOL=TRUE"

build-swift-examples "1" "set to 1 to build examples"
build-serialized-stdlib-unittest "0" "set to 1 to build the StdlibUnittest module with -sil-serialize-all"
build-sil-debugging-stdlib "0" "set to 1 to build the Swift standard library with -gsil to enable debugging and profiling on SIL level"
build-sourcekit "" "set to build the SourceKit library and tools (enabled by default on Darwin hosts)"
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you lift the computation of the default value into the Python script? Maybe as a separate patch.

@briancroom
Copy link
Contributor Author

Sorry for never following up on this. Shortly after opening this I was unable to continue working on SourceKit for Linux.

I'm going to close this as I don't think it's very relevant at this point. For reference, this comment describes what is currently required to get a SourceKit build on Linux: #3594 (comment)

@briancroom briancroom closed this Aug 25, 2016
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.

3 participants