Skip to content

[Backtracing] Make the backtracer work for fully-static Linux. #71847

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 4 commits into from
May 7, 2024

Conversation

al45tair
Copy link
Contributor

The backtracer and its build system need a couple of changes to make them work with the new fully-static Linux support.

rdar://123507656

@al45tair
Copy link
Contributor Author

This sits on top of #71844.

Copy link
Contributor

@mikeash mikeash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I'm assuming I should just be looking at the last commit here.

@al45tair
Copy link
Contributor Author

LGTM, I'm assuming I should just be looking at the last commit here.

Yes, that's right. It'd be nice if Github had better support for this kind of pattern (a stack of PRs).

al45tair added 4 commits May 7, 2024 16:45
Use the new `SWIFT_SDK_<sdk>_STATIC_LINKING_SUPPORTED` and
`_STATIC_ONLY` flags instead of hardcoding support for Linux
and WASI.

Also, use the `_LIB_SUBDIR` variable rather than lowercasing
the SDK.

rdar://123504757
This is really just about setting appropriate defaults (such as
making sure that the static Linux triple causes us to use lld).

rdar://123506306
We only need this for WASI and Glibc.

rdar://123506306
The backtracer and its build system need a couple of changes to make
them work with the new fully-static Linux support.

rdar://123507656
@al45tair al45tair force-pushed the eng/PR-123507656 branch from eb3eb09 to f4bf278 Compare May 7, 2024 15:52
@al45tair
Copy link
Contributor Author

al45tair commented May 7, 2024

@swift-ci Please smoke test

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.

2 participants