-
Notifications
You must be signed in to change notification settings - Fork 193
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
[SwiftScan] Add APIs to load/check libSwiftScan directly #1529
[SwiftScan] Add APIs to load/check libSwiftScan directly #1529
Conversation
@swift-ci please test |
ping |
Add APIs for SwiftDriver library users to directly load/check the libSwiftScan directly before calling into its function. * Add APIs to query Driver for libSwiftScan location * Make `verifyOrCreateScannerInstance` to be public APIs to load or check the libSwiftScan. * Update `verifyOrCreateScannerInstance` to be more ergonomic to use. The function only returns true if the given path to libSwiftScan does not exist while most of the time caller actually errors out in those cases. Make throwing error the default behavior and caller can check for file existance before calling the function to perserve old behavior.
e796558
to
d5d64d1
Compare
@swift-ci please test |
@swift-ci please test windows platform |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks ok to me, but I think @artemcm should review too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Add APIs for SwiftDriver library users to directly load/check the libSwiftScan directly before calling into its function.
verifyOrCreateScannerInstance
to be public APIs to load or check the libSwiftScan.verifyOrCreateScannerInstance
to be more ergonomic to use. The function only returns true if the given path to libSwiftScan does not exist while most of the time caller actually errors out in those cases. Make throwing error the default behavior and caller can check for file existance before calling the function to perserve old behavior.