-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
abseil-cpp: Add package #25565
abseil-cpp: Add package #25565
Conversation
CC @BKPepe |
a3e031f
to
8f89073
Compare
PowerPC is not a supported platform for
EDIT: Addressed with patches from OpenEmbedded / Void Linux |
b8ceb54
to
22349b2
Compare
Signed-off-by: Austin Lane <[email protected]>
22349b2
to
0521dad
Compare
i just feel like randomly ranting here that C++ is a terrible language, mostly because people tend to write weirdly-complicated code (using template magic), and it eventually can need a ton of RAM just to compile things, which makes it's usage in the embedded space all the more weird (or seemingly out of place) this PR however, is fine as-is :) |
Per @vortexilation #25566 (comment) this addition breaks builds of the
Node implements a bundled CC: @nxhack |
@vidplace7 About segmentation fault report in that issue, I don't encountered it anymore, please confirm. You can confirm it with :
If it's ended as node-yarn installed then it doesn't segmentation anymore, before it was happened at the process of compiling node-yarn. |
https://github.com/nodejs/node/blob/v22.11.0/deps/v8/DEPS#L358-L361 @vortexilation NodeJS should probably continue to use it's own bundled Instead of patching to include the shared absl, node should entirely ignore the |
@vidplace7 [EDIT] |
Maintainer: me
Compile tested:
Description:
Adds new package
abseil-cpp
. This is a dependency for many Google C libraries / projects. Notablyprotobuf
>= 22.x, but also: gRPC, tensorflow, and many others.Pre-requisite in order to update the
protobuf
package.https://protobuf.dev/support/migration/#abseil