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

Make fails (for 0.9.4 version) #5

Closed
geeksha opened this issue Apr 29, 2020 · 2 comments
Closed

Make fails (for 0.9.4 version) #5

geeksha opened this issue Apr 29, 2020 · 2 comments

Comments

@geeksha
Copy link

geeksha commented Apr 29, 2020

I was trying to compile your plugin for janus-gateway 0.9.4 and gives me an erro in make step. Here the log:

https://gist.github.com/geeksha/be22ed493ea863c238c82977da6ea320#file-gistfile1-txt

I hope it's usefel for you

One question regarding your approach: why not using directly the rtp_forwarding feature in videoroom plugin? What is the improvement added for your approach? I am researching different ways to do rtp_forwarding from WebRTC source to plain RTP.

Thanks in advance!

@michaelfranzl
Copy link
Owner

Your error log contains:

janus_rtpforward.c:581:5: error: too many arguments to function ‘gateway->relay_rtcp’
  581 |     gateway->relay_rtcp(session->handle, 1, buf, remblen);
      |     ^~~~~~~
make: *** [Makefile:454: janus_rtpforward.lo] Error 1

It seems you are trying to compile janus-rtpforward-plugin version 0.4 (which was made for janus-gateway version 0.4) against janus-gateway 0.9. This will not work because janus-gateway version 0.9 introduced breaking changes the plugin API.

Please re-try an up-to-date master branch which recently received support for janus-gateway version 0.9.

I addressed your question about the videoroom plugin in an update to the README. See ed8e87f

@geeksha
Copy link
Author

geeksha commented May 3, 2020

Many thanks for updating the README. This was a really clear explanation regarding your approach.
I was retrying compilation and it was compiling without any problem. I will check a little the behavior and the differences with videoroom testing both :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants