JP Simard

Results 706 comments of JP Simard

I'll take `swift-ide-test` entirely out of the equation since [it doesn't use SourceKit under the hood](https://twitter.com/UINT_MIN/status/887700362475356160), and instead use `SOURCEKIT_LOGGING=3` while requesting the interface for a module in Xcode. When...

The exact same approach works with 3rd party Swift frameworks: ```bash $ pwd /Users/jp/Downloads $ curl -O -L https://github.com/jpsim/SourceKitten/releases/download/0.18.0/SourceKittenFramework.framework.zip $ unzip SourceKittenFramework.framework.zip $ cat req.yml key.request: source.request.editor.open.interface key.name: "5F63C5B8-6D92-44FF-8012-DCA7D787D243" key.compilerargs:...

> Out of curiosity - do you know why is it necessary to generate interfaces? The compiler can't resolve all the dependencies if they're not available, or if it doesn't...

Feel free to close this issue if you feel your questions have been adequately addressed.

Step by step instructions are in this comment: https://github.com/jpsim/SourceKitten/issues/405#issuecomment-316473679

Here it is, reposted. Contents of `req.yml` ```yaml key.request: source.request.editor.open.interface key.name: "F6DA8E2C-95B6-40FB-ABC3-FE66F519B552" key.compilerargs: - "-target" - "x86_64-apple-macosx10.12" - "-sdk" - "/Applications/Xcode-8.3.3.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.12.sdk" - "-I" - "/Applications/Xcode-8.3.3.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.12.sdk/usr/local/include" - "-F" - "/Applications/Xcode-8.3.3.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.12.sdk/System/Library/PrivateFrameworks" -...

Agreed. Jazzy and `sourcekitten docs` already basically does this. It'd be great to know specifically what more folks want.

Thanks for investigating this. I'd be willing to bet this is a SourceKit bug, not a SourceKitten one, but confirming that will need further reduction of the repro case.

> While I'm doing this — would you be willing to consider changing this dependency from `upToNextMinor` to `upToNextMajor`? Yes, let's do that but revert back to 1.2 so that...

Don't worry about the Bazel test failures, there's been some changes to Bazel that I should adjust for.