linux/poky.git/meta/classes/devshell.bbclass, branch yocto-3.1.23

install/devshell: Introduce git intercept script due to fakeroot issues

2022-05-09T10:51:59+00:00

In a devshell, recent versions of git will complain if the repo is owned by someone other than the current UID - consider this example: ------ bitbake -c devshell linux-yocto [...] kernel-source#git branch fatal: unsafe repository ('/home/paul/poky/build-qemuarm64/tmp/work-shared/qemuarm64/kernel-source' is owned by someone else) To add an exception for this directory, call: git config --global --add safe.directory /home/paul/poky/build-qemuarm64/tmp/work-shared/qemuarm64/kernel-source kernel-source# ------ Of course the devshell has UID zero and the "real" UID is for "paul" in this case. And so recent git versions complain. As the whole purpose of the devshell is to invoke a shell where development can take place, having a non-functional git is clearly unacceptable. Richard suggested we could use PSEUDO_UNLOAD=1 to evade this issue, and I suggested we probably will see other similar instances like this and should make use of PATH to intercept via devshell wrappers - conveniently we already have examples of this. Here, we copy the existing "ar" example and tune it to the needs of git to combine Richard's suggestion and mine. As such we now also can store commit logs and use send-email with our user specific settings, instead of "root", so in additon to fixing basic commands like "git branch" it should also increase general usefulness. RP: Tweaked the patch so the PATH change only applies to the devshell task and is a generic git intercept rather than devshell specific. RP: Also apply the PATH change to do_install tasks since that also runs under fakeroot and several software projects inject "git describe" output into their binaries (systemd, iputils, llvm, ipt-gpu-tools at least) causing reproducibility issues from systems with different git versions. (From OE-Core rev: 0ff2cff2c1eac3fe6304644db0fc27ba3130c237) Signed-off-by: Paul Gortmaker Signed-off-by: Richard Purdie (cherry picked from commit 3266c327dfa186791e0f1e2ad63c6f5d39714814) Signed-off-by: Steve Sakoman Signed-off-by: Richard Purdie

devshell.bbclass: Exceptions displayed within devpyshell

2021-03-28T21:31:47+00:00

If you type something into devpyshell, an exception should be triggered, but that exception isn't printed out because it is not being flushed out of the buffer. This patch fixes that issue. [YOCTO #12156] (From OE-Core rev: 92b2da5824c11063d0bb8e39bc221530137e29b4) Signed-off-by: Dorinda Bassey Signed-off-by: Richard Purdie (cherry picked from commit 9f95e82a3b27bfd5177a855326dcad8a5bdf5b2d) Signed-off-by: Steve Sakoman Signed-off-by: Richard Purdie

devshell.bbclass: clean up trailing whitespaces

2017-06-06T18:02:43+00:00

(From OE-Core rev: fac6c0fcb30d7cb49036cb32247569d65d1c5e7f) Signed-off-by: Chang Rebecca Swee Fun Signed-off-by: Richard Purdie

devshell.bbclass: ensure toolchains available for do_devshell

2017-06-06T18:02:43+00:00

Toolchains are required to be prepopulated in sysroot before entering devshell. [YOCTO #11464] (From OE-Core rev: eb8a1cdf5106d119f77db76f9a690826af9fcda3) Signed-off-by: Chang Rebecca Swee Fun Signed-off-by: Richard Purdie

meta: remove True option to getVarFlag calls

2016-12-16T10:23:23+00:00

getVarFlag() now defaults to expanding by default, thus remove the True option from getVarFlag() calls with a regex search and replace. Search made with the following regex: getVarFlag ?$( ?[^,()]*, ?[^,()]*), True$ (From OE-Core rev: 2dea9e490a98377010b3d4118d054814c317a735) Signed-off-by: Joshua Lock Signed-off-by: Ross Burton Signed-off-by: Richard Purdie

meta: remove True option to getVar calls

2016-12-16T10:23:23+00:00

getVar() now defaults to expanding by default, thus remove the True option from getVar() calls with a regex search and replace. Search made with the following regex: getVar ?$( ?[^,()]*), True$ (From OE-Core rev: 7c552996597faaee2fbee185b250c0ee30ea3b5f) Signed-off-by: Joshua Lock Signed-off-by: Ross Burton Signed-off-by: Richard Purdie

devpyshell: python3: flush stdout explicitly

2016-07-08T08:57:23+00:00

Opening text stream in unbuffered mode raises the following exception In Python 3: ValueError: can't have unbuffered text I/O Fixed by leaving std* streams in text mode and flushing stdout explicitly. (From OE-Core rev: 732001cb268683f5b56e251e2964ec5b694a2147) Signed-off-by: Ed Bartosh Signed-off-by: Richard Purdie

devshell.bbclass: fix double unbuffering

2016-07-08T08:57:23+00:00

stdout is already unbuffered in bitbake code. Attempt to do it again in devshell.bbclass causes this crash when running devpyshell: File "scripts/oepydevshell-internal.py", line 29, in pty = open(sys.argv[1], "w+b", 0) IOError: [Errno 13] Permission denied: '/dev/pts/6' (From OE-Core rev: 875910451e1ce97d0c42b41b1140c8160ed1f40a) Signed-off-by: Ed Bartosh Signed-off-by: Richard Purdie

classes/lib: Add expand parameter to getVarFlag

2016-02-04T23:39:00+00:00

This sets the scene for removing the default False for expansion from getVarFlag. This would later allow True to become the default. On the most part this is an automatic translation with: sed -e 's:$\.getVarFlag([^,()]*, [^,()]*$):\1, True):g' -i `grep -ril getVar *` In this case, the default was False, but True was used since in most cases here expansion would be expected. (From OE-Core rev: 42a10788e89b07b14a150ced07113566cf99fcdd) Signed-off-by: Richard Purdie

devshell: allow the starting directory to be overridden

2015-03-16T17:41:55+00:00

Some people like to have the devshell start in ${B} instead of ${S}, so allow this to be overridden. (From OE-Core rev: 217af639d13258429949b14c67110650288cba7a) Signed-off-by: Ross Burton Signed-off-by: Richard Purdie