[Home]

Summary:ASTERISK-06952: [patch] aelparse will not build with astmm enabled
Reporter:Andrey S Pankov (casper)Labels:
Date Opened:2006-05-10 21:22:21Date Closed:2006-05-19 14:46:36
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) aelparse.diff
( 1) cflags.xml.diff
( 2) make_buildopts_h
( 3) Makefile.diff
Description:I attach a simple script to do the job. It could be easily integrated into
existing build process.

The rationale for this is to provide an easy way to determine how Asterisk
was built... and for external 3rd-party modules as well of course.

It would be nice to have somewhere the API version. Can we assume that it is
equal to major release number (1), minor (4), cleancount?
Comments:By: Russell Bryant (russell) 2006-05-11 04:51:48

I have made all of the necessary changes to do this in the trunk in revision 26808, thanks

By: Andrey S Pankov (casper) 2006-05-11 13:25:47

Sorry about re-opening this one...

Why do you think this is better like this?
-     echo "#define ${x} 1"
+     echo "#define ${x}"

Buildopts are define to 1 intentionnaly, and I think that way this is
safer. Both #if and #ifdef will work.

By: Andrey S Pankov (casper) 2006-05-11 14:51:47

Makefile.diff and cflags.xml.diff are to restore astmm-enabled build.
This is a workaround... maybe you'll come up with better solution ;)

By: Andrey S Pankov (casper) 2006-05-11 15:12:53

And aelparse build is still broken with astmm enabled.

By: Russell Bryant (russell) 2006-05-11 16:35:08

I fixed the build of astmm, except for aelparse ... I'll have to think about that one when I'm not so tired

By: Andrey S Pankov (casper) 2006-05-15 15:51:53

Please see aelparse.diff for some ideas... :)

By: Russell Bryant (russell) 2006-05-19 14:46:36

fixed in the trunk in revision 28796