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

Generating bindings for Cricket - missing headers #22

Closed
nravic opened this issue Jul 17, 2023 · 2 comments
Closed

Generating bindings for Cricket - missing headers #22

nravic opened this issue Jul 17, 2023 · 2 comments
Labels
question Further information is requested

Comments

@nravic
Copy link

nravic commented Jul 17, 2023

Hello,

I'm working on generating bindings in Go for Cricket, but I can't do anything without access to some headers that are listed in .gitignore - namely cpu_rpc_prot.h.

Out of curiosity why are these excluded? Also can I work around this by using cpu_rpc_prot.x? Not really sure what the .x file extension is or what the file itself is doing.

Thanks!

@nravic nravic changed the title Generating bindings for Cricket Generating bindings for Cricket - missing headers Jul 17, 2023
@n-eiling
Copy link
Member

Just do make cpu_rpc_prot.h. This should generate the header from the .x file. You need rpcgen for that. Have you had a look at https://github.com/gavrie/oncrpc? I am sure there are other projects.

@RWTH-ACS RWTH-ACS deleted a comment from Cattacker Jul 19, 2023
@n-eiling n-eiling added the question Further information is requested label Jul 19, 2023
@nravic
Copy link
Author

nravic commented Jul 19, 2023

Ah got it, I'll take a look at oncrpc too. Thanks!

Out of curiosity, not sure if I'm doing anything wrong here:

nravichandra@ubuntu:/Users/nravichandra/go/src/github.com/nravic/cricket-go/cricket/cpu$ rpcgen -C cpu_rpc_prot.x
        ptr_result rpc_register_function(ptr, ptr, string, string, int) = 50;
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
cpu_rpc_prot.x, line 216: only one argument is allowed

EDIT: nvm - got it, had to run make in the cpu directory. Thanks again

@nravic nravic closed this as completed Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants