diff options
author | Ying Wang <wangying@google.com> | 2010-11-23 15:46:40 -0800 |
---|---|---|
committer | Ying Wang <wangying@google.com> | 2010-11-23 15:46:40 -0800 |
commit | 0268060cbc8999b52b5dbcffed7ac7cfbace53db (patch) | |
tree | 2d379cb5cfac906725f2cb87a1f141b910a418a3 /sec_mm/sec_omx/sec_omx_component/video/dec/h264dec/Android.mk | |
parent | e37bd8e50e8c1181a0a2783d908e0e58ffba547e (diff) | |
download | device_samsung_crespo-0268060cbc8999b52b5dbcffed7ac7cfbace53db.zip device_samsung_crespo-0268060cbc8999b52b5dbcffed7ac7cfbace53db.tar.gz device_samsung_crespo-0268060cbc8999b52b5dbcffed7ac7cfbace53db.tar.bz2 |
Fix build race condition.
Before this change, referencing source in parent dir, such as
"../SEC_OMX_Vdec.c", results in obj files spilling into the top obj dir.
What's worse, the same source is included in more than 1 module, so the
obj file may be written concurrently by multiple build jobs.
Because gmake use the path string to detect if 2 targets point to the
same file, make could not even issue a warning for duplicate rules.
I believe this is the root cause of the following transient build error:
1)
out/target/product/stingray/obj/SHARED_LIBRARIES/libOMX.SEC.AVC.Encoder_intermediates/../SEC_OMX_Venc.P:54:
*** missing separator. Stop.
2)
prebuilt/darwin-x86/toolchain/arm-linux-androideabi-4.4.x/bin/../lib/gcc/arm-linux-androideabi/4.4.3/../../../../arm-linux-androideabi/bin/ld:
error:
out/target/product/stingray/obj/SHARED_LIBRARIES/libOMX.SEC.M4V.Encoder_intermediates/../SEC_OMX_Venc.o:
unknown mandatory EABI object attribute 46
Change-Id: I07721771d3afa86bd96703960c3a13305c424d60
Diffstat (limited to 'sec_mm/sec_omx/sec_omx_component/video/dec/h264dec/Android.mk')
-rw-r--r-- | sec_mm/sec_omx/sec_omx_component/video/dec/h264dec/Android.mk | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sec_mm/sec_omx/sec_omx_component/video/dec/h264dec/Android.mk b/sec_mm/sec_omx/sec_omx_component/video/dec/h264dec/Android.mk index baabbad..f6e6802 100644 --- a/sec_mm/sec_omx/sec_omx_component/video/dec/h264dec/Android.mk +++ b/sec_mm/sec_omx/sec_omx_component/video/dec/h264dec/Android.mk @@ -4,7 +4,6 @@ include $(CLEAR_VARS) LOCAL_MODULE_TAGS := optional LOCAL_SRC_FILES := \ - ../SEC_OMX_Vdec.c \ SEC_OMX_H264dec.c \ library_register.c @@ -15,7 +14,7 @@ LOCAL_CFLAGS := LOCAL_ARM_MODE := arm -LOCAL_STATIC_LIBRARIES := libsecosal libsecbasecomponent libsecmfcdecapi +LOCAL_STATIC_LIBRARIES := libSEC_OMX_Vdec libsecosal libsecbasecomponent libsecmfcdecapi LOCAL_SHARED_LIBRARIES := libc libdl libcutils libutils LOCAL_C_INCLUDES := $(SEC_OMX_INC)/khronos \ |