]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
test: correctly handle installation of newly introduced binaries
authorFrantisek Sumsal <frantisek@sumsal.cz>
Mon, 8 Jul 2019 19:05:51 +0000 (21:05 +0200)
committerFrantisek Sumsal <frantisek@sumsal.cz>
Mon, 8 Jul 2019 19:05:51 +0000 (21:05 +0200)
In certain cases we might attempt to install a binary which is already
present in the test image, yet it's missing from the host system.
In such cases, let's check if the binary indeed exists in the image
before doing any other chcecks. If it does, immediately return with
success.

This was discovered during installation of
/usr/lib/systemd/systemd-bless-boot, which was not present in Ubuntu CI
(as the installed systemd was from the Ubuntu repositories), and the
binary itself was already in the image thanks to `ninja install`.
However, during extraction of binaries from the systemd service files,
another attempt to install this binary was made, which failed due to
`find_binary` being unable to find it.

test/test-functions

index 4a76dd70ea10cb97190bc9dbd36884d172cd86fb..c4ac25fbc59ce3adf5c65023f694ba0e3e1f1e17 100644 (file)
@@ -1238,6 +1238,14 @@ find_binary() {
 # Install binary executable, and all shared library dependencies, if any.
 inst_binary() {
     local _bin _target
+
+    # In certain cases we might attempt to install a binary which is already
+    # present in the test image, yet it's missing from the host system.
+    # In such cases, let's check if the binary indeed exists in the image
+    # before doing any other chcecks. If it does, immediately return with
+    # success.
+    [[ $# -eq 1 && -e $initdir/$1 ]] && return 0
+
     _bin=$(find_binary "$1") || return 1
     _target=${2:-$_bin}
     [[ -e $initdir/$_target ]] && return 0