[Dependency Scanning] Query scanner whether discovered binary Swift modules are frameworks

[artemcm]

This info is required so that it can be specified as input to compilation that uses such module as a dependency, in order to decide whether or not to insert an auto-linking directive for it.

Companion change to swiftlang/swift#62540

Part of rdar://102824777

[artemcm]

@swift-ci test

[nkcsgexi]

LGTM! Shouldn't linking be explicit too given that we have all the dependency info?

[artemcm]

LGTM! Shouldn't linking be explicit too given that we have all the dependency info?

It absolutely should. But I'd like to get the existing flow working first, to avoid changing too many systems at-once, and then followup with the explicit linking work.

[artemcm]

@swift-ci test Windows platform

[artemcm]

@swift-ci test Windows platform

[artemcm]

C:\Users\swift-ci\jenkins\workspace\swift-driver-PR-windows\sourcekit-lsp\Sources\LanguageServerProtocol\Messages\.swift:59:3: error: cannot find 'OpenInterfaceRequest' in scope

This is a new one and is not related to the contents of this PR. @compnerd have we seen this before?

[compnerd]

Don't think we have. Could be a recent change for LSP? Should check that, but at the very least, we should check the driver build doesn't fail.

[artemcm]

Seems really weird. The reference to this type was added in the same commit as the definition. 🤔

[artemcm]

@swift-ci test Windows platform

[artemcm]

Ohh, I think the CMake build is broken, but not the package build.

[compnerd]

That doesn't make sense: swiftlang/sourcekit-lsp@c31dff2#diff-9e9107c03ddddab7789c4b0e4591aa6a0c569955404bda08039056526263391dR59 seems that it does update the CMake build.

[artemcm]

That doesn't make sense: apple/sourcekit-lsp@c31dff2#diff-9e9107c03ddddab7789c4b0e4591aa6a0c569955404bda08039056526263391dR59 seems that it does update the CMake build.

swiftlang/sourcekit-lsp#678

[artemcm]

@swift-ci test Windows platform

[artemcm]

Now Windows CI is broken for a new reason. 😥
Waiting on swiftlang/llvm-project#5792 to merge to unblock.

[artemcm]

@swift-ci test Windows platform