[Home]

Summary:ASTERISK-09429: [patch] extra spaces in configure script
Reporter:Tzafrir Cohen (tzafrir)Labels:
Date Opened:2007-05-12 06:56:00Date Closed:2007-07-11 19:58:53
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) nospace_include.diff
Description:The configure script has some syntx errors that cause certain tests to needlessly fail. Specifically: in the shell syntax there should not be an extra space after a '=' sign used for assignment.

Problem is aparantly not present in 1.4 .

After applying the patch, you will need to re-run autoconf (or ./bootstrap.sh)

****** ADDITIONAL INFORMATION ******

To locate those issues:

$ grep '[a-zA-Z0-9]= ' configure
ECHO_C= ECHO_N= ECHO_T=
.) ac_dir_suffix= ac_top_builddir_sub=. ac_top_build_prefix= ;;
 "") ac_top_builddir_sub=. ac_top_build_prefix= ;;
           PTHREAD_RWLOCK_INITIALIZER_INCLUDE= "-I${PTHREAD_RWLOCK_INITIALIZER_DIR}/include"
           PTHREAD_RWLOCK_PREFER_WRITER_NP_INCLUDE= "-I${PTHREAD_RWLOCK_PREFER_WRITER_NP_DIR}/include"
           RTLD_NOLOAD_INCLUDE= "-I${RTLD_NOLOAD_DIR}/include"
           IP_MTU_DISCOVER_INCLUDE= "-I${IP_MTU_DISCOVER_DIR}/include"
                         ac_lo= ac_hi=
                         ac_lo= ac_hi=
       ac_lo= ac_hi=
           ZAPTEL_INCLUDE= "-I${ZAPTEL_DIR}/include"
           ZAPTEL_INCLUDE= "-I${ZAPTEL_DIR}/include"
           ZAPTEL_VLDTMF_INCLUDE= "-I${ZAPTEL_VLDTMF_DIR}/include"
           ZAPTEL_TRANSCODE_INCLUDE= "-I${ZAPTEL_TRANSCODE_DIR}/include"
ECHO_C= ECHO_N= ECHO_T=
.) ac_dir_suffix= ac_top_builddir_sub=. ac_top_build_prefix= ;;
 "") ac_top_builddir_sub=. ac_top_build_prefix= ;;

The attached patch only solves some of them (the INCLUDE= ones)


Output of the same command after applying the patch and running the above command:
ECHO_C= ECHO_N= ECHO_T=
.) ac_dir_suffix= ac_top_builddir_sub=. ac_top_build_prefix= ;;
 "") ac_top_builddir_sub=. ac_top_build_prefix= ;;
                         ac_lo= ac_hi=
                         ac_lo= ac_hi=
       ac_lo= ac_hi=
ECHO_C= ECHO_N= ECHO_T=
.) ac_dir_suffix= ac_top_builddir_sub=. ac_top_build_prefix= ;;
 "") ac_top_builddir_sub=. ac_top_build_prefix= ;;
Comments:By: Russell Bryant (russell) 2007-05-14 11:08:37

This has been fixed in trunk in revisions 64225 and 64226.  Thanks!