On Mon, Apr 10, 2023 at 03:48:59PM -0500, Andrew Davis via 
lists.yoctoproject.org wrote:
> We do not need nor want a custom libc as the base for our packages.
> Using our own Linux UAPI includes creates this situation and will
> cause issues for packages that make use of the custom kernel headers
> when moving from kernel to kernel.
> 
> If we do have some package that does need some custom header then
> it should be listed in the package itself so this dependency can
> be tracked correct. This is all described in the base recipe here[0].
> 
> As recommended, let's not create a custom linux-libc-headers. Remove
> this recipe and references to specific versions.
> 
> [0] 
> https://cgit.openembedded.org/openembedded-core/tree/meta/recipes-kernel/linux-libc-headers/linux-libc-headers.inc#n7

Yes, the above recommendation from OE-Core was always there and has been 
enforced for a long time, until there were packages added to Arago and 
TI-SDK requiring TI custom Linux UAPI headers that weren't upstreamed...

Not sure what's the latest list of such packages in Arago, but I see at 
least "switch-config" is still there. I'd recommend disabling those custom 
packages from the build first, before removing linux-libc-headers - you'll 
get broken builds for either 5.10 or 6.1, which isn't very nice.

But yes, this is the right move going forward - I was hesitant adding this 
hack back in the day and always wanted to remove it, as it would cause 
extra pain during every LTS migration...


> Signed-off-by: Andrew Davis <[email protected]>

Acked-by: Denys Dmytriyenko <[email protected]>


> ---
>  .../conf/distro/include/toolchain-gcc.inc     |  4 ----
>  .../linux-libc-headers_5.10.bb                | 20 -------------------
>  2 files changed, 24 deletions(-)
>  delete mode 100644 
> meta-arago-distro/recipes-kernel/linux-libc-headers/linux-libc-headers_5.10.bb
> 
> diff --git a/meta-arago-distro/conf/distro/include/toolchain-gcc.inc 
> b/meta-arago-distro/conf/distro/include/toolchain-gcc.inc
> index 5570594d..1c059a6f 100644
> --- a/meta-arago-distro/conf/distro/include/toolchain-gcc.inc
> +++ b/meta-arago-distro/conf/distro/include/toolchain-gcc.inc
> @@ -16,7 +16,6 @@ 
> PREFERRED_PROVIDER_gdb-cross-canadian-${TRANSLATED_TARGET_ARCH} = 
> "gdb-cross-can
>  SDKGCCVERSION ?= "11%"
>  SDKBINUVERSION ?= "2.38%"
>  SDKGLIBCVERSION ?= "2.35%"
> -SDKLINUXLIBCVERSION ?= "5.10%"
>  
>  PREFERRED_VERSION_gcc-crosssdk-${SDK_SYS} ?= "${SDKGCCVERSION}"
>  PREFERRED_VERSION_gcc-crosssdk-initial-${SDK_SYS} ?= "${SDKGCCVERSION}"
> @@ -25,7 +24,6 @@ PREFERRED_VERSION_binutils-crosssdk-${SDK_SYS} ?= 
> "${SDKBINUVERSION}"
>  PREFERRED_VERSION_nativesdk-gcc-runtime ?= "${SDKGCCVERSION}"
>  PREFERRED_VERSION_nativesdk-libgcc ?= "${SDKGCCVERSION}"
>  PREFERRED_VERSION_nativesdk-libgcc-initial ?= "${SDKGCCVERSION}"
> -PREFERRED_VERSION_nativesdk-linux-libc-headers ?= "${SDKLINUXLIBCVERSION}"
>  PREFERRED_VERSION_nativesdk-glibc ?= "${SDKGLIBCVERSION}"
>  PREFERRED_VERSION_nativesdk-glibc-locale ?= "${SDKGLIBCVERSION}"
>  PREFERRED_VERSION_nativesdk-glibc-initial ?= "${SDKGLIBCVERSION}"
> @@ -35,11 +33,9 @@ PREFERRED_VERSION_nativesdk-glibc-initial ?= 
> "${SDKGLIBCVERSION}"
>  GCCVERSION ?= "11%"
>  BINUVERSION ?= "2.38%"
>  GLIBCVERSION ?= "2.35%"
> -LINUXLIBCVERSION ?= "5.10%"
>  
>  PREFERRED_VERSION_gcc ?= "${GCCVERSION}"
>  PREFERRED_VERSION_binutils ?= "${BINUVERSION}"
> -PREFERRED_VERSION_linux-libc-headers ?= "${LINUXLIBCVERSION}"
>  PREFERRED_VERSION_glibc ?= "${GLIBCVERSION}"
>  PREFERRED_VERSION_glibc-locale ?= "${GLIBCVERSION}"
>  PREFERRED_VERSION_glibc-initial ?= "${GLIBCVERSION}"
> diff --git 
> a/meta-arago-distro/recipes-kernel/linux-libc-headers/linux-libc-headers_5.10.bb
>  
> b/meta-arago-distro/recipes-kernel/linux-libc-headers/linux-libc-headers_5.10.bb
> deleted file mode 100644
> index 43493570..00000000
> --- 
> a/meta-arago-distro/recipes-kernel/linux-libc-headers/linux-libc-headers_5.10.bb
> +++ /dev/null
> @@ -1,20 +0,0 @@
> -require recipes-kernel/linux-libc-headers/linux-libc-headers.inc
> -
> -PR:append = ".arago0"
> -
> -LIC_FILES_CHKSUM = "file://COPYING;md5=6bc538ed5bd9a7fc9398086aedcd7e46"
> -
> -BRANCH = "ti-linux-5.10.y"
> -SRCREV = "73aa709ca10103b61fba3a07471dbb4dcb56db45"
> -
> -KERNEL_GIT_URI = "git://git.ti.com/git/ti-linux-kernel/ti-linux-kernel.git"
> -KERNEL_GIT_PROTOCOL = "https"
> -SRC_URI = 
> "${KERNEL_GIT_URI};protocol=${KERNEL_GIT_PROTOCOL};branch=${BRANCH}"
> -
> -S = "${WORKDIR}/git"
> -
> -# below overrides the multilib list - can be dropped for the next LTS
> -do_install_armmultilib () {
> -     oe_multilib_header asm/auxvec.h asm/bitsperlong.h asm/byteorder.h 
> asm/fcntl.h asm/hwcap.h asm/ioctls.h asm/kvm_para.h asm/mman.h asm/param.h 
> asm/perf_regs.h asm/bpf_perf_event.h
> -     oe_multilib_header asm/posix_types.h asm/ptrace.h  asm/setup.h  
> asm/sigcontext.h asm/siginfo.h asm/signal.h asm/stat.h  asm/statfs.h 
> asm/swab.h  asm/types.h asm/unistd.h
> -}
> -- 
> 2.39.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#14307): 
https://lists.yoctoproject.org/g/meta-arago/message/14307
Mute This Topic: https://lists.yoctoproject.org/mt/98184756/21656
Group Owner: [email protected]
Unsubscribe: 
https://lists.yoctoproject.org/g/meta-arago/leave/10763299/21656/89520264/xyzzy 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to