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

[CHANGE] Remove FreeSWITCH translator #201

Merged
merged 2 commits into from Jun 16, 2015

Conversation

@benlangfeld
Copy link
Member

benlangfeld commented Jan 8, 2014

Since mod_rayo was initially created, it's been said that we have deprecated support for the FreeSWITCH translator in Punchblock, that fixes to it would be lowest priority and that people should use mod_rayo. In fact, all of our documentation pushes mod_rayo and hides the IES capability of Punchblock.

Arguments in favour of removing this functionality:

  • mod_rayo is more capable and performant
  • Lower maintenance overhead and greater focus in Punchblock
  • Adhearsion is moving in the direction of being Rayo-only (a Rayo server implementation on Asterisk has been on wish-lists for a long time, and Asterisk 13 is an opportunity for a potential res_rayo), and Punchblock is very much a Rayo client

Arguments against:

  • Backward compatibility
  • Potential inability to move to mod_rayo
  • mod_rayo features / stability?

This would necessarily bump the Punchblock version number to 3.0.0. I am of the opinion that the following should be candidates for inclusion in this major version bump:

This will go to the mailing list to gather input from users of Adhearsion and Punchblock, particularly on the above arguments against this change. A decision will be made after sufficient time is allowed for comment.

@coveralls
Copy link

coveralls commented Jan 8, 2014

Coverage Status

Coverage decreased (-0.0%) when pulling 136d3a8 on feature/remove_freeswitch_translator into cad18ed on develop.

@benlangfeld benlangfeld force-pushed the feature/remove_freeswitch_translator branch from 3d039fa to 910a125 Jun 15, 2015
benlangfeld added a commit that referenced this pull request Jun 16, 2015
…nslator

[CHANGE] Remove FreeSWITCH translator
@benlangfeld benlangfeld merged commit 53a27a9 into develop Jun 16, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@benlangfeld benlangfeld deleted the feature/remove_freeswitch_translator branch Jun 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.