[asterisk-dev] gratuitous change in rev.48416 ?

Russell Bryant russell at digium.com
Tue Dec 12 17:45:12 MST 2006


----- Kevin P. Fleming <kpfleming at digium.com> wrote:
> Luigi Rizzo wrote:
> > I'd suggest to revert this and, if necessary, make the
> > change to codecs/codec_zap.c

If that's true, then I appologize.  I when I saw the compiler errors had something
to do with "zap", I just assumed it was chan_zap, I guess.  Sorry ...

> I'm not even sure why this is needed, when we can already test for
> the proper things in the configure script.

Right, but as I stated in this commit message, checking for this type of thing
is the right thing to do, but it doesn't help existing checkouts unless we
force re-running the configure script after any code change.  So, something is
better than nothing, and in this case, a proper message with the failure at
build time is better than the standing compilation failure leaving the person
to wonder what the problem is.

-- 
Russell Bryant
Software Engineer
Digium, Inc.



More information about the asterisk-dev mailing list