[asterisk-dev] [Code Review] Remove chan_usbradio and app_rpt.

Steve Totaro stotaro at asteriskhelpdesk.com
Tue Feb 28 14:35:14 CST 2012


Please allow me a few days to get this sorted out.

I know why it is not being maintained in the respository, part is
political, part is business motivated.

It goes against the grain of all that is Asterisk and I am in a position to
make a real issue of it in this "divergent" community.

I ask for a week to convince the maintainers that letting this happen is a
really bad idea, and it is, so the community will back me.

Thanks,
Steve Totaro
KB3OPB

On Tue, Feb 28, 2012 at 9:42 AM, Leif Madsen <reviewboard at asterisk.org>wrote:

>    This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1764/
>
> This patch set needs UPGRADE.txt and CHANGES updated to reflect that these modules have been removed, and ideally, a pointer to where the maintained code lives. For 1.8 and 10 of course you'll need to add a header about changes between 1.8.x and 1.8.x+1 (obviously same for 10.x).
>
>
> - Leif
>
> On February 22nd, 2012, 10:01 a.m., Russell Bryant wrote:
>   Review request for Asterisk Developers.
> By Russell Bryant.
>
> *Updated Feb. 22, 2012, 10:01 a.m.*
> Description
>
> This patch removes app_rpt and chan_usbradio from trunk.  I don't have a problem at all with these modules being in trunk, but they do not appear to be maintained in this repository.  One of the primary radio guys said in #asterisk-dev the other day that plenty of people use this stuff, but that fixes haven't been sent back up to this repo in years.  It seems harmful to have an out-of-date copy of this code here.
>
>   Testing
>
> ran configure and make.
>
>   Diffs
>
>    - /trunk/apps/app_rpt.c (356258)
>    - /trunk/build_tools/menuselect-deps.in (356258)
>    - /trunk/channels/chan_usbradio.c (356258)
>    - /trunk/channels/xpmr/sinetabx.h (356258)
>    - /trunk/channels/xpmr/xpmr.h (356258)
>    - /trunk/channels/xpmr/xpmr.c (356258)
>    - /trunk/channels/xpmr/xpmr_coef.h (356258)
>    - /trunk/configure (UNKNOWN)
>    - /trunk/configure.ac (356258)
>    - /trunk/include/asterisk/autoconfig.h.in (356258)
>    - /trunk/makeopts.in (356258)
>
> View Diff <https://reviewboard.asterisk.org/r/1764/diff/>
>
> --
> _____________________________________________________________________
> -- 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120228/66796d91/attachment.htm>


More information about the asterisk-dev mailing list