Skip to content

Add an expectation of a warning diagnostic to tests that hide libSwiftScan. #7995

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
Oct 7, 2024

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Sep 23, 2024

Such tests cause the driver to take a code-path that results in a warning: warning: Unable to locate libSwiftScan. Fallback to swift-frontend dependency scanner invocation.

This needs to be taken into account.

@artemcm
Copy link
Contributor Author

artemcm commented Sep 23, 2024

@swift-ci test

@artemcm artemcm force-pushed the ExpectNoLibSwiftcanWarnOnHide branch 2 times, most recently from 98d6cba to bb9351d Compare September 23, 2024 21:06
@artemcm
Copy link
Contributor Author

artemcm commented Sep 23, 2024

@artemcm
Copy link
Contributor Author

artemcm commented Sep 24, 2024

From Smoke test and cross-repo test:

/Users/ec2-user/jenkins/workspace/swift-package-manager-PR-macos-smoke-test/branch-main/swiftpm/Tests/PackageModelSyntaxTests/ManifestEditTests.swift:410: error: -[PackageModelSyntaxTests.ManifestEditTests testAddLibraryTargetWithDependencies] : failed - Actual output does not match the expected
 import OtherLib
+
 import SwiftSyntax
+
 import TargetLib

I don't think these are related.

@artemcm
Copy link
Contributor Author

artemcm commented Sep 24, 2024

swiftlang/swift-syntax#2858 resolved the above failures. Re-testing.

@artemcm
Copy link
Contributor Author

artemcm commented Sep 24, 2024

@swift-ci test

@artemcm
Copy link
Contributor Author

artemcm commented Sep 24, 2024

1 similar comment
@artemcm
Copy link
Contributor Author

artemcm commented Sep 24, 2024

@artemcm artemcm force-pushed the ExpectNoLibSwiftcanWarnOnHide branch from bb9351d to 7ad725f Compare September 25, 2024 23:37
@artemcm
Copy link
Contributor Author

artemcm commented Sep 25, 2024

@artemcm artemcm force-pushed the ExpectNoLibSwiftcanWarnOnHide branch from 7ad725f to 6aa6d8f Compare September 26, 2024 16:10
@artemcm
Copy link
Contributor Author

artemcm commented Sep 26, 2024

…tScan.

Such tests cause the driver to take a code-path that results in a warning:
'warning: Unable to locate libSwiftScan. Fallback to `swift-frontend` dependency scanner invocation.'

This needs to be taken into account.
@artemcm artemcm force-pushed the ExpectNoLibSwiftcanWarnOnHide branch from 6aa6d8f to 9e2afee Compare September 27, 2024 16:37
@artemcm
Copy link
Contributor Author

artemcm commented Sep 27, 2024

@bnbarham
Copy link
Contributor

swiftlang/swift-driver#1703
@swift-ci please test Windows

@artemcm artemcm merged commit e08708e into main Oct 7, 2024
5 checks passed
@artemcm artemcm deleted the ExpectNoLibSwiftcanWarnOnHide branch October 7, 2024 16:24
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