search

jambonz / jambonz-infrastructure

packer and cloudformation templates for creating EC2-based jambonz deployments
23 stars 29 forks source link

packer/jambonz-mini/scripts/install_drachtio.sh uses -DNDEBUG #35

Open asarubbo opened 1 year ago

asarubbo commented 1 year ago

Hello Dave,

packer/jambonz-mini/scripts/install_drachtio.sh uses -DNDEBUG see https://github.com/jambonz/jambonz-infrastructure/blob/b1f3bdc1c1529ef545f6cdbc1c44e39d58b931bc/packer/jambonz-mini/scripts/install_drachtio.sh#L11 while by grepping in the drachtio sources I see:

sip-transports.cpp:#ifdef DEBUG

So to trigger the DEBUG in sip-transports.cpp the macro should be -DDEBUG

So since -DNDEBUG has it own scope, see https://en.cppreference.com/w/c/error/assert I'm wondering if it was just misconfused or the correct configuration is -DNDEBUG -DDEBUG