Summary:ASTERISK-05424: [patch] Recent Cygwin portability patches broke cross-compilation
Reporter:Wojtek Kaniewski (wojtekka)Labels:
Date Opened:2005-11-02 05:59:07.000-0600Date Closed:2008-01-15 15:54:46.000-0600
Versions:Frequency of
Environment:Attachments:( 0) asterisk-head-cross-gsm.patch
( 1) CrossCompileFix.diff.txt
Description:Patches for bug ASTERISK-4558 broke cross-compilation of codecs/gsm/libgsm with the line:

PROC=$(shell uname -m)

in codecs/gsm/libgsm/Makefile which shadows user-defined $(PROC). In my case (cross-compilation on x86 for PowerPC), ppc-linux-gcc complained about unknown -march=i686.

Removing the line fixes the problem, at least for PowerPC. I don't have Cygwin to test, so someone should test the fix before commiting to CVS.


No disclaimer on file, the fix is trivial.
Comments:By: Michael Jerris (mikej) 2005-11-02 07:41:01.000-0600

When I do that change on cygwin, I get the following:  

make[1]: Entering directory `/usr/src/asterisk/codecs'
make -C gsm lib/libgsm.a
make[2]: Entering directory `/usr/src/asterisk/codecs/gsm'
gcc  -pipe  -Wall -Wstrict-prototypes -Wmissing-prototypes -Wmissing-declaration
s -g   -Iinclude -I../include -D_REENTRANT -D_GNU_SOURCE  -O6          -fomit-fr
ame-pointer   -c -DNeedFunctionPrototypes=1 -funroll-loops -O6 -march= -DSASR -D
NDEBUG    -DWAV49   -I./inc src/add.c
src/add.c:1: error: bad value () for -march= switch
src/add.c:1: error: bad value () for -mtune= switch
make[2]: *** [src/add.o] Error 1
make[2]: Leaving directory `/usr/src/asterisk/codecs/gsm'
make[1]: *** [gsm/lib/libgsm.a] Error 2
make[1]: Leaving directory `/usr/src/asterisk/codecs'
make: *** [subdirs] Error 1

Which was the original reason for this change.  It makes no sense as that var should be set from the main makefile, but for some reason, it just isn't there.  Anyone have any ideas why?

By: Michael Jerris (mikej) 2005-11-02 07:51:42.000-0600

Uploaded the complete fix that doesn't break cygwin... sorry about that.

By: Kevin P. Fleming (kpfleming) 2005-11-07 22:40:51.000-0600

Committed to CVS HEAD, thanks!

By: Digium Subversion (svnbot) 2008-01-15 15:54:46.000-0600

Repository: asterisk
Revision: 7018

U   trunk/ChangeLog
U   trunk/Makefile
U   trunk/codecs/gsm/Makefile

r7018 | kpfleming | 2008-01-15 15:54:46 -0600 (Tue, 15 Jan 2008) | 2 lines

issue ASTERISK-5424