can you send me your Makefile for cross compiling this version of ekiga? On Mon, Feb 1, 2010 at 11:10 AM, e schmidbauer <[email protected]>wrote:
> just tested the celt codec (0.7.1) with freeswitch and it works! great job > everyone. > > > On Mon, Feb 1, 2010 at 8:50 AM, Thierry Simonnet <[email protected]>wrote: > >> Hi again, >> >> I checked my package and saw that I made a mistake for plib and opal (due >> to name change from 2/3.7beta1 to 2/3.9beta0. >> I made change and the ekiga-setup-3.3.1-git-496_gb9026d8-celt071.exe >> program is now with the right libraries. >> >> Sorry for the trouble >> >> Best regards >> >> Thierry >> >> >> On 02/01/2010 09:59 AM, Thierry Simonnet wrote: >> >>> Hi, >>> >>> I integrate celt 0.7.1 on my Makefile and use your last celtcodec.c file. >>> As Eugen wrote, there is a trouble using HAVE_CELT_0_7_0_OR_LATER. >>> For test purposes, I defined it at 1 in your celtcodec.c file, waiting >>> for a final patch. >>> You can download the last ekiga-setup.exe at >>> http://www.pateam.org/archive/tmp/ekiga-win32/trunk/ekiga-setup-3.3.1-git-496_gb9026d8-celt071.exe. >>> >>> >>> Unfortunately, I can't test celt codec because my asterisk PBX doesn't >>> support it. >>> >>> Thierry >>> >>> On 02/01/2010 02:01 AM, e schmidbauer wrote: >>> >>>> Could you compile it and post the install? I'll work on the >>>> configure.ac <http://configure.ac> >>>> >>>> On Sun, Jan 31, 2010 at 5:33 PM, Eugen Dedu < >>>> [email protected] <mailto:[email protected]>> >>>> wrote: >>>> >>>> Great! >>>> >>>> Note however that replacing >>>> #ifdef HAVE_CELT_0_5_0_OR_LATER >>>> with >>>> #ifdef HAVE_CELT_0_7_0_OR_LATER >>>> is not good. >>>> >>>> There should be something like this: >>>> #ifdef HAVE_CELT_0_7_0_OR_LATER >>>> your new line >>>> #elifdef HAVE_CELT_0_5_0_OR_0_6_0 >>>> existing line >>>> #else >>>> existing line >>>> #endif >>>> >>>> Could you do the correct #define checking in celtcodec.c and >>>> modifying >>>> configure.ac <http://configure.ac> for the new version too >>>> (replacing 050orlater with 050or060 >>>> and adding a similar text to 0.5 version)? >>>> >>>> Eugen >>>> >>>> e schmidbauer wrote: >>>> > This should work. >>>> > >>>> > On Sun, Jan 31, 2010 at 2:48 PM, Eugen Dedu >>>> <[email protected] >>>> <mailto:[email protected]> >>>> >> wrote: >>>> > >>>> >> I have two warnings: >>>> >> snoopy:~/softs/ekiga/opal/plugins/audio/celt$ make >>>> >> [CC] celtcodec.c >>>> >> celtcodec.c: In function ‘valid_for_sip’: >>>> >> celtcodec.c:196: warning: pointer targets in passing argument 1 of >>>> >> ‘strcasecmp’ differ in signedness >>>> >> /usr/include/string.h:534: note: expected ‘const char *’ but >>>> argument is >>>> >> of type ‘const unsigned char *’ >>>> >> celtcodec.c: At top level: >>>> >> celtcodec.c:202: warning: initialization from incompatible >>>> pointer type >>>> >> [LD] ./celtcodec_pwplugin.so >>>> >> >>>> >> I removed the first one by removing "unsigned". Do you know >>>> how to fix >>>> >> the 2nd warning? >>>> >> >>>> >> e schmidbauer wrote: >>>> >>> Please try compiling with this file. I have not been able to >>>> test this >>>> >>> version of celt because i do not have a linux machine >>>> available to cross >>>> >>> compile. If you are successful at compiling, i can test it >>>> out. I will >>>> >> get >>>> >>> to the plugins/configure.ac <http://configure.ac> file later. >>>> Please tell me if you get any >>>> >> errors >>>> >>> or warnings. Thank you. >>>> >>> >>>> >>> On Sun, Jan 31, 2010 at 8:15 AM, Eugen Dedu < >>>> >> [email protected] >>>> <mailto:[email protected]> >>>> >>>> wrote: >>>> >>>> e schmidbauer wrote: >>>> >>>>> I've attached what i think will work for compiling opal with >>>> celt 0.7 >>>> >>>>> Can you please test it to see if it compiles? >>>> >>>> Thanks a lot for the patch. >>>> >>>> >>>> >>>> 1. When compiling, I have a few warnings, could you remove >>>> them too? >>>> >>>> >>>> >>>> snoopy:~/softs/ekiga/opal/plugins/audio/celt$ make >>>> >>>> [CC] celtcodec.c >>>> >>>> celtcodec.c: In function ‘celt_codec_encoder’: >>>> >>>> celtcodec.c:146: warning: pointer targets in passing argument >>>> 4 of >>>> >>>> ‘celt_encode’ differ in signedness >>>> >>>> /usr/include/celt/celt.h:214: note: expected ‘unsigned char >>>> *’ but >>>> >>>> argument is of type ‘char *’ >>>> >>>> celtcodec.c: In function ‘celt_codec_decoder’: >>>> >>>> celtcodec.c:176: warning: pointer targets in passing argument >>>> 2 of >>>> >>>> ‘celt_decode’ differ in signedness >>>> >>>> /usr/include/celt/celt.h:262: note: expected ‘const unsigned >>>> char *’ but >>>> >>>> argument is of type ‘char *’ >>>> >>>> [LD] ./celtcodec_pwplugin.so >>>> >>>> >>>> >>>> 2. Also, have you successfully made a communication with celt >>>> (0.7) >>>> >>>> using your file? >>>> >>>> >>>> >>>> 3. Do you have the will to modify also plugins/configure.ac >>>> <http://configure.ac> so that it >>>> >>>> defines HAVE_CELT_0_7_0 and use that #define in celtcodec.c >>>> so that it >>>> >>>> compiles with previous versions of celt too? If not, I will >>>> send your >>>> >>>> patch to be committed anyway, but it is better with it... >>>> >>>> >>>> >>>> Thanks, >>>> >>>> -- >>>> >>>> Eugen >>>> >>>> _______________________________________________ >>>> >>>> Ekiga-devel-list mailing list >>>> >>>> [email protected] <mailto:[email protected]> >>>> >>>> http://mail.gnome.org/mailman/listinfo/ekiga-devel-list >>>> >>>> >>>> >>> >>>> >>> >>>> >>>> ------------------------------------------------------------------------ >>>> >>> >>>> >>> _______________________________________________ >>>> >>> Ekiga-devel-list mailing list >>>> >>> [email protected] <mailto:[email protected]> >>>> >>> http://mail.gnome.org/mailman/listinfo/ekiga-devel-list >>>> >> _______________________________________________ >>>> >> Ekiga-devel-list mailing list >>>> >> [email protected] <mailto:[email protected]> >>>> >> http://mail.gnome.org/mailman/listinfo/ekiga-devel-list >>>> >> >>>> > >>>> > >>>> > >>>> >>>> ------------------------------------------------------------------------ >>>> > >>>> > _______________________________________________ >>>> > Ekiga-devel-list mailing list >>>> > [email protected] <mailto:[email protected]> >>>> > http://mail.gnome.org/mailman/listinfo/ekiga-devel-list >>>> >>>> _______________________________________________ >>>> Ekiga-devel-list mailing list >>>> [email protected] <mailto:[email protected]> >>>> http://mail.gnome.org/mailman/listinfo/ekiga-devel-list >>>> >>>> >>>> >>>> _______________________________________________ >>>> Ekiga-devel-list mailing list >>>> [email protected] >>>> http://mail.gnome.org/mailman/listinfo/ekiga-devel-list >>>> >>> >>> >>> >> >> -- >> >> Thierry Simonnet >> >> ESIEE-Paris >> >> Par respect pour l’environnement, n’imprimez ce mail que si nécessaire >> >> >> >
_______________________________________________ Ekiga-devel-list mailing list [email protected] http://mail.gnome.org/mailman/listinfo/ekiga-devel-list
