sonyxperiadev / bug_tracker

Empty repository that is used as a bugtracker for Open Devices project
52 stars 13 forks source link

[XQ-AU52] AOSP build for Android 13 fails due to missing constant definition #844

Closed arnixrobotix closed 2 months ago

arnixrobotix commented 2 months ago

Platform: Seine Device: Xperia X10 II (XQ-AU52) Kernel version: 4.19 Android version: 13 Software binaries version: v4a

Previously working on Build never succeeded

Description While following this guide (but building inside a Docker container as my build server is running Void Linux), I get the following error when building the images for aosp_xqau52-eng:

arnix@4c8458713082:~/src/android$ make -j$(nproc)
14:13:25 Build sandboxing disabled due to nsjail error.
build/make/core/soong_config.mk:209: warning: BOARD_PLAT_PUBLIC_SEPOLICY_DIR has been deprecated. Use SYSTEM_EXT_PUBLIC_SEPOLICY_DIRS instead.
build/make/core/soong_config.mk:210: warning: BOARD_PLAT_PRIVATE_SEPOLICY_DIR has been deprecated. Use SYSTEM_EXT_PRIVATE_SEPOLICY_DIRS instead.
============================================
PLATFORM_VERSION_CODENAME=REL
PLATFORM_VERSION=13
TARGET_PRODUCT=aosp_xqau52
TARGET_BUILD_VARIANT=eng
TARGET_BUILD_TYPE=release
TARGET_ARCH=arm64
TARGET_ARCH_VARIANT=armv8-a
TARGET_CPU_VARIANT=generic
TARGET_2ND_ARCH=arm
TARGET_2ND_ARCH_VARIANT=armv8-a
TARGET_2ND_CPU_VARIANT=generic
HOST_ARCH=x86_64
HOST_2ND_ARCH=x86
HOST_OS=linux
HOST_OS_EXTRA=Linux-6.6.48_1-x86_64-Ubuntu-20.04.6-LTS
HOST_CROSS_OS=windows
HOST_CROSS_ARCH=x86
HOST_CROSS_2ND_ARCH=x86_64
HOST_BUILD_TYPE=release
BUILD_ID=TQ3A.230901.001
OUT_DIR=out
PRODUCT_SOONG_NAMESPACES=device/sony/common device/sony/seine vendor/qcom/opensource/core-utils vendor/qcom/opensource/audio-hal/primary-hal vendor/qcom/opensource/display/sm8250 vendor/qcom
/opensource/display-commonsys-intf/sm8250
============================================

(...)

[  6% 11764/170243] target thumb C: libqcompostprocbundle_32 <= vendor/qcom/opensource/audio-hal/primary-hal/post_proc/effect_api.c
FAILED: out/target/product/pdx201/obj_arm/SHARED_LIBRARIES/libqcompostprocbundle_intermediates/effect_api.o
/bin/bash -c "PWD=/proc/self/cwd  prebuilts/clang/host/linux-x86/clang-r450784d/bin/clang -I external/tinyalsa/include -I vendor/qcom/opensource/audio-hal/primary-hal/hal -I system/media/aud
io_effects/include -I vendor/qcom/opensource/audio-hal/primary-hal/hal/audio_extn -I vendor/qcom/opensource/audio-hal/primary-hal/post_proc -I out/target/product/pdx201/obj_arm/SHARED_LIBRAR
IES/libqcompostprocbundle_intermediates -I out/target/product/pdx201/gen/SHARED_LIBRARIES/libqcompostprocbundle_intermediates -isystem kernel/sony/msm-4.19/common-headers/kernel-headers -isy
stem kernel/sony/msm-4.19/common-headers/kernel-headers -Isystem/core/libcutils/include_outside_system -Isystem/core/libprocessgroup/include -Isystem/core/libcutils/include -Isystem/logging/
liblog/include_vndk -Iexternal/tinyalsa/include -Iexternal/tinyalsa/include -Iexternal/libcxx/include -Iexternal/libcxxabi/include -isystem out/soong/.intermediates/bionic/libc/libc/android_
vendor.33_arm_armv8-a_shared/gen/include -isystem bionic/libc/kernel/uapi/asm-arm -isystem bionic/libc/kernel/uapi -isystem bionic/libc/kernel/android/scsi -isystem bionic/libc/kernel/androi
d/uapi   -Iexternal/libcxxabi/include  -Ihardware/libhardware/include -Isystem/media/audio/include -Isystem/core/libcutils/include_outside_system -Isystem/core/libsystem/include -Ipackages/m
odules/Bluetooth/system/types -Isystem/core/libsystem/include -Isystem/core/libutils/include -Isystem/unwinding/libbacktrace/include -Isystem/libbase/include -Iexternal/fmtlib/include -Isyst
em/core/libcutils/include_outside_system -Isystem/logging/liblog/include_vndk -Isystem/core/libprocessgroup/include -Isystem/core/libsystem/include   -isystem out/target/product/pdx201/obj/i
nclude -c  -fomit-frame-pointer -DANDROID -fmessage-length=0 -W -Wall -Wno-unused -Winit-self -Wpointer-arith -Wunreachable-code-loop-increment -no-canonical-prefixes -DNDEBUG -UDEBUG -fno-e
xceptions -Wno-multichar -O2 -g -fdebug-default-version=5 -fno-strict-aliasing -Werror=date-time -Werror=pragma-pack -Werror=pragma-pack-suspicious-include -Werror=string-plus-int -Werror=un
reachable-code-loop-increment -D__compiler_offsetof=__builtin_offsetof -faddrsig -fcommon -Werror=int-conversion -fexperimental-new-pass-manager -Wno-reserved-id-macro -fcolor-diagnostics -W
no-sign-compare -Wno-defaulted-function-deleted -Wno-inconsistent-missing-override -Wno-c99-designator -Wno-gnu-designator -Wno-gnu-folding-constant -Wunguarded-availability -D__ANDROID_UNAV
AILABLE_SYMBOLS_ARE_WEAK__ -ffp-contract=off -fdebug-prefix-map=/proc/self/cwd= -ftrivial-auto-var-init=zero -enable-trivial-auto-var-init-zero-knowing-it-will-be-removed-from-clang -ffuncti
on-sections -fdata-sections -fno-short-enums -funwind-tables -fstack-protector-strong -Wa,--noexecstack -D_FORTIFY_SOURCE=2 -Wstrict-aliasing=2 -Werror=return-type -Werror=non-virtual-dtor -
Werror=address -Werror=sequence-point -Werror=format-security -nostdlibinc -fdebug-info-for-profiling -msoft-float -march=armv8-a -mfloat-abi=softfp -mfpu=neon-fp-armv8    -std=gnu99 -mthumb
 -Os   -DPRIMARY_HAL_PLATFORM_NAME=sm6125 -Wno-unused-variable -Wno-sign-compare -Wno-unused-parameter -Wno-unused-label -Wno-gnu-designator -Wno-typedef-redefinition -Wno-shorten-64-to-32 -
Wno-tautological-compare -Wno-unused-function -Wno-unused-local-typedef -Wno-format -Wno-unused-value -Wall -Werror -DAFE_PROXY_ENABLED -DAUDIO_GKI_ENABLED -DINSTANCE_ID_ENABLED -DINSTANCE_I
D_ENABLED -O2 -fvisibility=hidden -DDTS_EAGLE -DDO_NOT_CHECK_MANUAL_BINDER_INTERFACES -D__ANDROID_VNDK__ -D__ANDROID_VENDOR__ -fPIC -DANDROID_STRICT -target armv7a-linux-androideabi33   -Wer
ror=bool-operation -Werror=implicit-int-float-conversion -Werror=int-in-bool-context -Werror=int-to-pointer-cast -Werror=pointer-to-int-cast -Werror=string-compare -Werror=xor-used-as-pow -W
no-void-pointer-to-enum-cast -Wno-void-pointer-to-int-cast -Wno-pointer-to-int-cast -Werror=fortify-source -Werror=address-of-temporary -Werror=return-type -Wno-tautological-constant-compare
 -Wno-tautological-type-limit-compare -Wno-reorder-init-list -Wno-implicit-int-float-conversion -Wno-sizeof-array-div -Wno-tautological-overlap-compare -Wno-deprecated-copy -Wno-range-loop-c
onstruct -Wno-misleading-indentation -Wno-zero-as-null-pointer-constant -Wno-deprecated-anon-enum-enum-conversion -Wno-string-compare -Wno-pessimizing-move -Wno-non-c-typedef-for-linkage -Wn
o-align-mismatch -Wno-error=unused-but-set-variable -Wno-error=unused-but-set-parameter -Wno-unused-but-set-variable -Wno-unused-but-set-parameter -Wno-bitwise-instead-of-logical -MD -MF out
/target/product/pdx201/obj_arm/SHARED_LIBRARIES/libqcompostprocbundle_intermediates/effect_api.d -o out/target/product/pdx201/obj_arm/SHARED_LIBRARIES/libqcompostprocbundle_intermediates/eff
ect_api.o vendor/qcom/opensource/audio-hal/primary-hal/post_proc/effect_api.c"
vendor/qcom/opensource/audio-hal/primary-hal/post_proc/effect_api.c:183:20: error: use of undeclared identifier 'AUDIO_EFFECTS_CONFIG_SET'
    config_param = AUDIO_EFFECTS_CONFIG_SET;
                   ^
1 error generated.
[  6% 11773/170243] //bionic/libc:libc versioner preprocess include
warning: attempted to generate guard with empty availability: obsoleted = 21
warning: attempted to generate guard with empty availability: obsoleted = 23
14:20:25 ninja failed with: exit status 1

#### failed to build some targets (07:01 (mm:ss)) ####

How to reproduce Follow the instructions from the guide.

Additional context I am building inside a Docker container running Ubuntu 20.04.

arnixrobotix commented 2 months ago

Here's the full log, including the previous commands to initialize the repo: failed_build_aosp_xq-au52.txt

Edit: I noticed that the update of the repo had a warning (about the rename limit), I fixed it but I am still facing the same issue regarding AUDIO_EFFECTS_CONFIG_SET not being defined...

bartcubbins commented 2 months ago

I see the reason. I will send a patch tomorrow https://github.com/sonyxperiadev/kernel-techpack-audio/pull/62

arnixrobotix commented 2 months ago

Thanks for the patch, I'll close this issue once it's merged!

arnixrobotix commented 2 months ago

Fixed by this PR