asterisk-1.6.1.12

[stefan.van.der.eijk@…]

Update to the asterisk port, to 1.6.1.12.

[jann@…]

You seem to be hardcoding x86_64 as platform. This is bad. Can you please fix this. ${build_arch} will tell you if you need to build for x86_64 or i386 or ppc. But you can also use platform to distinguish between ppc and intel.

[stefan.van.der.eijk@…]

OK, I'll disable building the gsm codec, as this is the part causing the issue. Please review new diff.

[jann@…]

Looks good now, but it shouldn't be too hard to give the correct argument depending on the build platform. Or does the GSM codec only work on x86_64 ?

[stefan.van.der.eijk@…]

Replying to jann@…:

Looks good now, but it shouldn't be too hard to give the correct argument depending on the build platform.

I'd rather not do that. It should just build on all platforms.

Or does the GSM codec only work on x86_64 ?

No, I think there is something wrong with the way the platform selection is done for the GSM codec. When nothing is specified I get the following error on my mac mini (10.6.2):

src/add.c:1: error: CPU you selected does not support x86-64 instruction set

See #20973

I guess the best place to address this is upstream with the Asterisk team at Digium.

[jann@…]

I committed the patch in r62817. Can you talk to the Digium guys? Another thing that I find stange is teh dependency on wget and curl. Is this really required?