Re: [asterisk-dev] [Code Review] 4106: configure: Add autoconf check for libopus

2014-10-27 Thread Matt Jordan

---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4106/#review13600
---

Ship it!


I think this is fine to go into 13.

- Matt Jordan


On Oct. 22, 2014, 2:35 p.m., Sean Bright wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviewboard.asterisk.org/r/4106/
 ---
 
 (Updated Oct. 22, 2014, 2:35 p.m.)
 
 
 Review request for Asterisk Developers.
 
 
 Repository: Asterisk
 
 
 Description
 ---
 
 Because opus transcoding support cannot be included in the standard Asterisk 
 distribution, a few codec_opus implementations have popped up.  To make it 
 easier for people to drop in opus support in their own installations, this 
 patch adds configure checks for libopus.
 
 I don't see why this wouldn't be safe for 13, but I will defer to the 
 reviewers on that decision.
 
 
 Diffs
 -
 
   /trunk/makeopts.in 426095 
   /trunk/include/asterisk/autoconfig.h.in 426095 
   /trunk/configure.ac 426095 
   /trunk/configure UNKNOWN 
   /trunk/build_tools/menuselect-deps.in 426095 
 
 Diff: https://reviewboard.asterisk.org/r/4106/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Sean Bright
 


-- 
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

Re: [asterisk-dev] [Code Review] 4106: configure: Add autoconf check for libopus

2014-10-27 Thread Sean Bright

---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4106/
---

(Updated Oct. 27, 2014, 12:54 p.m.)


Status
--

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
---

Committed in revision 426234


Repository: Asterisk


Description
---

Because opus transcoding support cannot be included in the standard Asterisk 
distribution, a few codec_opus implementations have popped up.  To make it 
easier for people to drop in opus support in their own installations, this 
patch adds configure checks for libopus.

I don't see why this wouldn't be safe for 13, but I will defer to the reviewers 
on that decision.


Diffs
-

  /trunk/makeopts.in 426095 
  /trunk/include/asterisk/autoconfig.h.in 426095 
  /trunk/configure.ac 426095 
  /trunk/configure UNKNOWN 
  /trunk/build_tools/menuselect-deps.in 426095 

Diff: https://reviewboard.asterisk.org/r/4106/diff/


Testing
---


Thanks,

Sean Bright

-- 
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev