-
Notifications
You must be signed in to change notification settings - Fork 10
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 for Asterisk 21 #13
Comments
|
Hello rnseb, have you succeeded in making the installation on Asterisk 21? If so, can you share the details of the steps with the community? Thanks, Eric |
Hi, I tried to add the recommendation from @rnseb and did some changes, I compiled the code with Asterisk 22-current, and run a mobile call to the PBX I was able to also do transcoding from g722 to EVS. |
Hello Abdul,
Thank you very much, yes I will give a try and let you know how it went
Eric
On Dec 27, 2024, at 23:04, Abdulkarim Barbour ***@***.***> wrote:
Hi, I tried to add the recommendation from @rnseb<https://github.com/rnseb> and did some changes, I compiled the code with Asterisk 22-current, and run a mobile call to the PBX I was able to also do transcoding from g722 to EVS.
I add the changes to the my fork, maybe you could give it a try :
https://github.com/NUCLEAR-WAR/asterisk-evs
—
Reply to this email directly, view it on GitHub<#13 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAU6DBMQSF3TCVCI3FGDKPD2HYPLNAVCNFSM6AAAAABNNXMSQOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRUGE3DGMBZGU>.
You are receiving this because you commented.Message ID: ***@***.***>
|
Hi,
Could we please make this work with Asterisk 21?
For example, I think
res |= ast_rtp_engine_load_format(ast_format_evs);
should be added here, after L824: https://github.com/traud/asterisk-evs/blob/master/codecs/codec_evs.c#L824And this should probably be removed: https://github.com/traud/asterisk-evs/blob/master/codec_evs.patch#L52
The text was updated successfully, but these errors were encountered: