Skip to content

[Dependency Scanning] Build libSwiftScan with Swift Modules required for parsing regex #65372

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 26, 2023

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Apr 21, 2023

Otherwise scanning actions which encounter regex literals will fail.

Resolves rdar://108321860

@artemcm
Copy link
Contributor Author

artemcm commented Apr 21, 2023

@swift-ci smoke test

@artemcm artemcm force-pushed the DepScanNowWithRegex branch from 6b0933e to 83be391 Compare April 24, 2023 21:08
@artemcm
Copy link
Contributor Author

artemcm commented Apr 24, 2023

@swift-ci smoke test

3 similar comments
@artemcm
Copy link
Contributor Author

artemcm commented Apr 24, 2023

@swift-ci smoke test

@artemcm
Copy link
Contributor Author

artemcm commented Apr 24, 2023

@swift-ci smoke test

@artemcm
Copy link
Contributor Author

artemcm commented Apr 24, 2023

@swift-ci smoke test

@artemcm
Copy link
Contributor Author

artemcm commented Apr 24, 2023

@swift-ci smoke test Windows platform

@artemcm
Copy link
Contributor Author

artemcm commented Apr 25, 2023

@swift-ci smoke test

…for parsing regex

Otherwise scanning actions which encounter regex literals will fail.

Resolves rdar://108321860
@artemcm artemcm force-pushed the DepScanNowWithRegex branch from 83be391 to 6c71d80 Compare April 25, 2023 17:17
@artemcm
Copy link
Contributor Author

artemcm commented Apr 25, 2023

@swift-ci smoke test

@artemcm
Copy link
Contributor Author

artemcm commented Apr 25, 2023

@swift-ci smoke test macOS platform

@artemcm artemcm merged commit aae2fe2 into swiftlang:main Apr 26, 2023
@artemcm artemcm deleted the DepScanNowWithRegex branch April 26, 2023 15:23
@finagolfin
Copy link
Member

@artemcm, since you added this, I see an incorrect runpath added to this library on linux:

> readelf -d swift-DEVELOPMENT-SNAPSHOT-2023-05-06-a-ubuntu20.04/usr/lib/swift/host/lib_InternalSwiftScan.so | ag runpath
 0x000000000000001d (RUNPATH)            Library runpath: [$ORIGIN:$ORIGIN/./swift/linux]

The issue is that you've triggered this CMake function to automatically add the Swift runpath, which fails because it expects all host libraries to be placed in usr/lib/.

You'll want to fix this, else this library won't always work on linux.

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