Skip to content
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

llvmPackages: add overrideable withLibunwind #207325

Closed
wants to merge 1 commit into from

Conversation

ehmry
Copy link
Contributor

@ehmry ehmry commented Dec 22, 2022

See #170850

This makes it possible to disable bundling libunwind with Clang
for out-of-tree platforms.
@AndersonTorres
Copy link
Member

Closing as dead, unmergeable shit

@rrbutani
Copy link
Contributor

@AndersonTorres why do you think this is dead/unmergable?

@ehmry
Copy link
Contributor Author

ehmry commented May 14, 2023

Whatever, llvmPackages is messy but if it works don't break it.

@AndersonTorres
Copy link
Member

AndersonTorres commented May 14, 2023

@AndersonTorres why do you think this is dead/unmergable?

I don't think, it is: Github interface says this can't be merged because conflicts with current codebase.

Further, this PR is 5 months old.

@zimbatm
Copy link
Member

zimbatm commented Aug 4, 2023

@AndersonTorres please be more respectful of other people's work. This is the second time that one of your messages got reported to the moderation team. Please DM me if you have any questions.

@rrbutani rrbutani added the 6.topic: llvm/clang Issues related to llvmPackages, clangStdenv and related label May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: llvm/clang Issues related to llvmPackages, clangStdenv and related 10.rebuild-darwin: 0 10.rebuild-linux: 0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants