MIRRORS/poky
1
0
Fork 0
mirror of git://git.yoctoproject.org/poky.git synced 2025-07-19 21:09:03 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

rust-common.bbclass: rewrite toolchain wrappers in (native) python

Browse Source 
librsvg on centos 7 and friends exhibits the same libtinfo leakage
problem, this time coming from the compiler and not the linker.
Simply covering the compiler by the existing C wrapper-of-wrapper
does not work, as rust-native builds put Important Stuff into
LD_LIBRARY_PATH and unsetting it breaks things badly.

Rather than try to figure out which combination of wrappers and
LD_LIBRARY_PATH settings works for which situation, or provide
some kind of sh-native, let's simply use python3-native for the
wrappers, which should insulate builds from the the host shell.
rust-native already depends on python3-native, so this does not
lengthen the builds.

This also reverts:
rust-common: Hack around LD_LIBRARY_PATH issues on centos7
(commit 63b1fd2226).

I'd also like to say boo to Red Hat (or GNU?) for breaking ABI
compatibility for stat() in glibc 2.33, we ended up sorting
this mess because of it.

(From OE-Core rev: 997d54363a3cb3a0e949b3626855f2fa41afeb2b)

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Alexander Kanavin 2021-10-10 21:10:06 +02:00 committed by Richard Purdie
parent bcab6dd34f
commit 4abd6ee9d4
2 changed files with 8 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
meta/classes/rust-common.bbclass
View File

@ -1,3 +1,5 @@
inherit python3native
# Common variables used by all Rust builds
export rustlibdir = "${libdir}/rust"
FILES:${PN} += "${rustlibdir}/*.so"
@ -133,8 +135,12 @@ create_wrapper () {
shift
cat <<- EOF > "${file}"
#!/bin/sh
exec $@ "\$@"
#!/usr/bin/env python3
import os, sys
orig_binary = "$@"
binary = orig_binary.split()[0]
args = orig_binary.split() + sys.argv[1:]
os.execvp(binary, args)
EOF
chmod +x "${file}"
}
@ -169,11 +175,6 @@ do_rust_create_wrappers () {
# Yocto Target / Rust Target archiver
create_wrapper "${RUST_TARGET_AR}" "${WRAPPER_TARGET_AR}"
# Need to filter out LD_LIBRARY_PATH from the linker without using shell
mv ${RUST_BUILD_CCLD} ${RUST_BUILD_CCLD}.real
${BUILD_CC} ${COREBASE}/meta/files/rust-ccld-wrapper.c -o ${RUST_BUILD_CCLD}
mv ${RUST_TARGET_CCLD} ${RUST_TARGET_CCLD}.real
${BUILD_CC} ${COREBASE}/meta/files/rust-ccld-wrapper.c -o ${RUST_TARGET_CCLD}
}
addtask rust_create_wrappers before do_configure after do_patch do_prepare_recipe_sysroot

29
meta/files/rust-ccld-wrapper.c
View File

@ -1,29 +0,0 @@
/*
* Copyright 2021 Richard Purdie
*
* SPDX-License-Identifier: GPL-2.0-only
*/
#include <errno.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
/*
* Run the original script (argv[0] + ".real") with LD_LIBRARY_PATH unset
* This avoids issues where cargo is running a wrapper script using /bin/sh from the host
* which links to something which has an incompatible version in in recipe-sysroot-native
* such as libtinfo on centos 7.
*/
int main(int argc, char* argv[]) {
char *real = malloc(strlen(argv[0] + 5));
strcpy(real, argv[0]);
strcpy(real + strlen(argv[0]), ".real");
putenv("LD_LIBRARY_PATH=");
if(execv(real, argv) == -1) {
printf("Wrapper failed to execute, error: %s\n", strerror(errno));
return -1;
}
}