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

Support cc_embed_data #41

Open
silvasean opened this issue Oct 11, 2019 · 7 comments
Open

Support cc_embed_data #41

silvasean opened this issue Oct 11, 2019 · 7 comments
Labels
P3 Accepted issue. Team is likely to fix this issue in the future, but no short-term promises are made. type: feature request Request for new, generally useful functionality that is missing

Comments

@silvasean
Copy link

Description of the problem / feature request:

Bazel doesn't support cc_embed_data

Feature requests: what underlying problem are you trying to solve with this feature?

I would like to embed data into C++ code.

We ended up having to hand-roll our own: iree-org/iree@4be1097

Feel free to take code from there. I don't think we have time to upstream this properly.

@stellaraccident
Copy link

Here is the implementation (hand written): https://github.com/google/iree/tree/master/build_tools/embed_data

@chandlerc
Copy link

FWIW, LLVM needs this as well, and it is challenging to use this implementation. (Abseil and other dependencies that we shouldn't need to pull down and put in the host build just for this...)

Is there any chance of progress here, or should we too roll our own?

@stellaraccident
Copy link

stellaraccident commented Oct 27, 2020

It shouldn't be hard to remove the abseil dep, and in hindsight I shouldn't have taken it. I don't have any plans to evolve this further unless if a need emerges for us. Feel free to grab anything we have to bootstrap rolling a new one, and if you get a dep-cleaner version, we can grab that back when it is ready.

@chandlerc
Copy link

For now, trying to avoid the need, but will do if it ends up necessary.

@schoppmp
Copy link

I'd also +1 this, as I'd like to use cc_embed_data in https://github.com/google/distributed_point_functions/.

@jiawen
Copy link

jiawen commented Nov 20, 2021

Xoogler here. I'm happy to take on this task of upstreaming this into Bazel as my first contribution.

@chandlerc @stellaraccident It's hard for me to unwind the threads but it seems your implementation doesn't actually have a dependency on abseil?

@stellaraccident
Copy link

Yeah, it should be dependency clean. Looks like its grown a couple of define checks in the generated code but should be easily removed. Feel free to take it if it is helpful.

@comius comius added P3 Accepted issue. Team is likely to fix this issue in the future, but no short-term promises are made. type: feature request Request for new, generally useful functionality that is missing labels Dec 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3 Accepted issue. Team is likely to fix this issue in the future, but no short-term promises are made. type: feature request Request for new, generally useful functionality that is missing
Projects
None yet
Development

No branches or pull requests

6 participants