11453 2017-05-03 21:55:13 +0000 misconfigured "diff" utility 2017-06-30 08:26:36 +0000 1 1 7 Build System, Metadata & Runtime OE-Core core 2.3 x86 Multiple RESOLVED FIXED Medium normal 2.4 M2 1 juro.bystricky jku jku meta.mr.watcher meta.watcher randy.macleod --- --- Don't know oldest_to_newest 73021 0 juro.bystricky 2017-05-03 21:55:13 +0000 When building "core-image-minimal-initramfs-qemux86.cpio.gz", the image contains the utility "diff". (due to update alternatives it is called diff.diffutils). This utility relies on another utility "pr" to be present (for paginated output?). The location of this utility is hardcoded in the "diff" utility as "/data/master-master/poky/build-repro-again/tmp/hosttools/pr", in other words the utility was detected from my local "hosttools" folder. 73038 1 jku 2017-05-04 10:58:45 +0000 Is this known to break something (other than manually running "diff -l")? Fixing the path is easy (set ac_cv_path_PR_PROGRAM) but then we'd have to RDEPEND on coreutils to actually have /usr/bin/pr ... 73039 2 ross.burton 2017-05-04 11:08:31 +0000 Assuming that its only diff -l that is broken then hardcoding the path and not adding a dependency seems right. Juro, did you discover this by using diff -l, or some other way? 73043 3 juro.bystricky 2017-05-04 14:18:59 +0000 I found out by using diffoscope, binary comparison of two different builds, showing the build directory path. Another possible way (I think) to fix this would be to remove "pr" from HOSTTOOLS in the recipe. 73046 4 ross.burton 2017-05-04 14:41:41 +0000 Removing pr from HOSTTOOLS causes libav's configure to moan. Lets tell diff what pr to use. Jussi, mind taking this? 74678 5 jku 2017-06-30 08:26:36 +0000 commit f1a5c7c53be36bbee3b7476b336bab5a04c2d46d Author: Jussi Kukkonen <jussi.kukkonen@intel.com> Date: Fri Jun 9 10:36:18 2017 +0300 diffutils: prevent host path leaks, set pr path Note that this just removes a host path leakage: diffutils still doesn't depend on coreutils so 'pr' might not actually be there on image (this only breaks "diff -l" so no big deal). (From OE-Core rev: 3f7fd738981752a8c34c6cf4a1f2d4c035b081ca)