Summary: | ASTERISK-09429: [patch] extra spaces in configure script | ||
Reporter: | Tzafrir Cohen (tzafrir) | Labels: | |
Date Opened: | 2007-05-12 06:56:00 | Date Closed: | 2007-07-11 19:58:53 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | 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! |