Skip to content

Add support for compile jobs that read source input from standard input. #595

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 1 commit into from
Apr 15, 2021

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Apr 12, 2021

This change uses Process input handling added in swiftlang/swift-tools-support-core#208 to allow forwarding of the driver's Standard Input to a frontend job.

Resolves rdar://76454784

@artemcm
Copy link
Contributor Author

artemcm commented Apr 14, 2021

@swift-ci please test

1 similar comment
@artemcm
Copy link
Contributor Author

artemcm commented Apr 14, 2021

@swift-ci please test

@artemcm
Copy link
Contributor Author

artemcm commented Apr 14, 2021

@shahmishal, the CI log says it is pulling swift-tools-support-core from main, but the build fails as if swiftlang/swift-tools-support-core#208 hasn't been merged. Am I missing something in how this works?

@shahmishal
Copy link
Member

@swift-ci please test

@shahmishal
Copy link
Member

Its running on clean workspace, let me log in and check the sha.

@shahmishal
Copy link
Member

  {
        "package": "swift-tools-support-core",
        "repositoryURL": "https://github.com/apple/swift-tools-support-core.git",
        "state": {
          "branch": "main",
          "revision": "207f9c6bc5e33eb8383266c0cd3129c1276f534b",
          "version": null
        }
      }

@artemcm
Copy link
Contributor Author

artemcm commented Apr 14, 2021

"revision": "207f9c6bc5e33eb8383266c0cd3129c1276f534b",

Right, this is quite old so if it's resolved to this commit then it makes sense why it doesn't work.

This change builds on swiftlang/swift-tools-support-core#208 to allow forwarding of the driver's Standard Input to a compile job.

Resolves rdar://76454784
@artemcm artemcm force-pushed the ProcessInputSergio branch from 6d5f95b to cc35dca Compare April 14, 2021 17:37
@artemcm
Copy link
Contributor Author

artemcm commented Apr 14, 2021

@swift-ci please test

@artemcm artemcm merged commit cadd652 into swiftlang:main Apr 15, 2021
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