[Freeswitch-dev] Maybe bug in switch_rtp.c?

[Lawrence Conroy]

Hi Ken, folks,
 I had a feeling you were going to say that :).
I'll try to do the Jira thing the other side of night.

OS X 10.6.8 -- just using the command line: ".bootstrap.sh, ./configure, make, make install, ..."
PATH is tweaked to favour /usr/local over /bin /usr/bin ...
(I have Xcode 3.2.6 installed so the tools are what came with that, and I used SourceTree 1.5.1 for git). 
Old habits & all that.
Thanks for the confirmation -- I was beginning to wonder.

all the best,
  Lawrence

On 31 Jul 2012, at 00:57, Ken Rice wrote:
> Yeah this is a build error, open a Jira with a complete build log so we can
> get some attention on this.
> 
> Also how are you building this?
> 
> Homebrew, just Xcode tools or something else?
> 
> 
> On 7/30/12 6:54 PM, "Lawrence Conroy" <lconroy at insensate.co.uk> wrote:
> 
>> Hi folks,
>> at the risk of irritating everyone by not *starting* with a JIRA ...
>> 
>> I've just been building on MAC OS X 10.6 from git (version as of 120730 at
>> 22:36 GMT) -- commit 0f4ecab5e9d3ad765ee266f1af41560df5f434ae[0f4ecab].
>> 
>> The make reports an error when compiling switch_rtp.c (line 3314).
>> 
>> Is this me, my whacko MBP, or is this (as would seem) a bug?
>> 
>> ..............................................................................
>> cc1: warnings being treated as errors
>> src/switch_rtp.c: In function ‘rtp_common_read’:
>> src/switch_rtp.c:3314: warning: format ‘%s’ expects type ‘char *’, but
>> argument 9 has type ‘int’
>> make[1]: *** [libfreeswitch_la-switch_rtp.lo] Error 1
>> 
>> Ugly Quick hack ... changed it to show the int result of strerror_r and then
>> the string
>> -> add an %i before the [%s] in the format, and then put ,tmp after the
>> strerror_r call.
>> Is now:
>> switch_log_printf(SWITCH_CHANNEL_SESSION_LOG(session), SWITCH_LOG_ERROR, "Poll
>> failed with error: %d %i [%s]\n",
>> poll_status, strerror_r(poll_status, tmp, sizeof(tmp)), tmp);
>> 
>> builds OK (but maaaan, someone else needs to write something that doesn't make
>> the eyes bleed).
>> ..............................................................................
>> 
>> I guess this may be because of a switch from using strerror to the thread-safe
>> strerror_r function.
>> 
>> 
>> all the best,
>>  Lawrence
>> _________________________________________________________________________
>> Professional FreeSWITCH Consulting Services:
>> consulting at freeswitch.org
>> http://www.freeswitchsolutions.com
>> 
>> 
>> 
>> 
>> Official FreeSWITCH Sites
>> http://www.freeswitch.org
>> http://wiki.freeswitch.org
>> http://www.cluecon.com
>> 
>> Join Us At ClueCon - Aug 7-9, 2012
>> 
>> FreeSWITCH-dev mailing list
>> FreeSWITCH-dev at lists.freeswitch.org
>> http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev
>> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-dev
>> http://www.freeswitch.org
> 
> 
> 
> _________________________________________________________________________
> Professional FreeSWITCH Consulting Services:
> consulting at freeswitch.org
> http://www.freeswitchsolutions.com
> 
> 
> 
> 
> Official FreeSWITCH Sites
> http://www.freeswitch.org
> http://wiki.freeswitch.org
> http://www.cluecon.com
> 
> Join Us At ClueCon - Aug 7-9, 2012
> 
> FreeSWITCH-dev mailing list
> FreeSWITCH-dev at lists.freeswitch.org
> http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev
> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-dev
> http://www.freeswitch.org