Closed GoogleCodeExporter closed 9 years ago
In examining the config.log and makefile some more I see that libwebrtc_aecm.a
got missed somehow.
configure:18554: checking for WebRtcAecm_Create in -lwebrtc_aecm
configure:18579:
/Users/anand/opensource/doubango/branches/2.0/doubango/my-android-toolchain-armv
5te/bin/arm-linux-androideabi-gcc -o conftest -DANDROID=1 -fPIC -Os -s
-I/Users/anand/opensource/doubango/branches/2.0/doubango/thirdparties/android/ar
mv5te/include
-I/Users/anand/opensource/doubango/branches/2.0/doubango/thirdparties/android/co
mmon/include
-I/Users/anand/opensource/doubango/branches/2.0/doubango/thirdparties/common/inc
lude/libyuv -DANDROID=1 -fPIC -Os -s
-I/Users/anand/opensource/doubango/branches/2.0/doubango/thirdparties/android/ar
mv5te/include
-I/Users/anand/opensource/doubango/branches/2.0/doubango/thirdparties/android/co
mmon/include
-I/Users/anand/opensource/doubango/branches/2.0/doubango/thirdparties/common/inc
lude/libyuv -lc -lm -lgcc
-L/Users/anand/opensource/doubango/branches/2.0/doubango/thirdparties/android/ar
mv5te/lib/dist
-L/Users/anand/opensource/doubango/branches/2.0/doubango/thirdparties/android/ar
mv5te/lib/dist/gpl conftest.c -lwebrtc_aecm -lwebrtc_apm_utility
-lwebrtc_system_wrappers -lwebrtc_spl >&5
configure:18579: $? = 0
configure:18588: result: yes
ac_cv_lib_webrtc_aecm_WebRtcAecm_Create=yes
Missing in the auto-generated doubango/makefile
LIBWEBRTC_LIBADD = -lwebrtc_nsx -lwebrtc_apm_utility -lwebrtc_system_wrappers
-lwebrtc_spl
So I suspect, if I can link with -lwebrtc_aecm, the problem will go away
Anyone else seeing this issue?
Original comment by saket...@gmail.com
on 7 Mar 2014 at 1:39
Works fine on CentOS but it's strange because AECM is missing.
Patch:
Index: bindings/Makefile.am
===================================================================
--- bindings/Makefile.am (revision 1037)
+++ bindings/Makefile.am (working copy)
@@ -85,6 +85,10 @@
libtinyWRAP_la_LIBADD += ${LIBILBC_LIBADD}
endif
+if USE_WEBRTC
+libtinyWRAP_la_LIBADD += ${LIBWEBRTC_LIBADD}
+endif
+
libtinyWRAP_la_SOURCES = \
_common/ActionConfig.cxx \
_common/AudioResampler.cxx \
Index: configure.ac
===================================================================
--- configure.ac (revision 1037)
+++ configure.ac (working copy)
@@ -801,11 +801,11 @@
have_webrtc_aec=no
if test "$target_os" = "android"; then
AC_CHECK_HEADERS(webrtc/echo_control_mobile.h,
- AC_CHECK_LIB(webrtc_aecm, WebRtcAecm_Create, [have_webrtc_aec=yes],
[], [-lwebrtc_apm_utility -lwebrtc_system_wrappers -lwebrtc_spl])
+ AC_CHECK_LIB(webrtc_aecm, WebRtcAecm_Create, [have_webrtc_aec=yes],
[LIBWEBRTC_LIBADD="-lwebrtc_aecm ${LIBWEBRTC_LIBADD}"], [-lwebrtc_apm_utility
-lwebrtc_system_wrappers -lwebrtc_spl])
)
else
AC_CHECK_HEADERS(webrtc/echo_cancellation.h,
- AC_CHECK_LIB(webrtc_aec, WebRtcAec_Create, [have_webrtc_aec=yes], [],
[-lwebrtc_apm_utility -lwebrtc_system_wrappers -lwebrtc_spl])
+ AC_CHECK_LIB(webrtc_aec, WebRtcAec_Create, [have_webrtc_aec=yes],
[LIBWEBRTC_LIBADD="-lwebrtc_aec ${LIBWEBRTC_LIBADD}"], [-lwebrtc_apm_utility
-lwebrtc_system_wrappers -lwebrtc_spl])
)
fi
# WebRTC NS
Original comment by boss...@yahoo.fr
on 8 Mar 2014 at 2:56
r1040
Original comment by boss...@yahoo.fr
on 10 Mar 2014 at 1:28
checkout version 1043, building is ok(centos 6.5), but test on device, Echo is
still very serious, AECM does not seem to work. more detail:
https://groups.google.com/forum/#!topic/doubango/OPh9m3QePsQ
Original comment by taolesss...@gmail.com
on 10 Mar 2014 at 2:48
i am also have this problem,do you solve it?
Original comment by zhangxia...@gmail.com
on 19 Apr 2014 at 11:41
Original issue reported on code.google.com by
saket...@gmail.com
on 7 Mar 2014 at 12:50