build: remove -lc++fs from linkopts#7534
Conversation
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
|
Nvm this doesn't work. |
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
|
@asraa @htuch more context of this PR:
This PR changes #7507 a bit to only use |
htuch
left a comment
There was a problem hiding this comment.
This is fine to unblock oss-fuzz. It's a bit mind boggling how complicated getting a few filesystem libraries is in the C++ ecosystem. To me, this seems a failure in how Clang is handling the rollout of these new libraries, it should be much simpler to just condition on version without also needing to juggle the link complexity.
Agreed, actually #7507 is needlessly complicated and I can clean it up more. |
|
Thank you so much for the context as well, @lizan |
Signed-off-by: Lizan Zhou lizan@tetrate.io
Description:
Cherry-picking latest commit of #7329 before release. To unbreak fuzz build. Address #7507 that bring real c++fs dependency.
Risk Level: Low
Testing: local fuzz, CI
Docs Changes:
Release Notes: