app/test: remove meson dependency on file in /sys
authorBruce Richardson <bruce.richardson@intel.com>
Mon, 20 Jan 2020 12:22:18 +0000 (12:22 +0000)
committerThomas Monjalon <thomas@monjalon.net>
Mon, 20 Jan 2020 15:37:27 +0000 (16:37 +0100)
commit599d67b6a4bf96096352cc5fbc8fc28e54a1ca62
tree06cb12c128af71a4acf473ba297e86ff31f4f99c
parent30512af820feff1627120b0e18f51ce8a5efac3e
app/test: remove meson dependency on file in /sys

Meson versions 0.52 and 0.53 are being overly smart and detecting the path
"/sys/devices/system/cpu/present" in the call to cat in
app/test/meson.build and then adding it as a dependency to the build
configuration. This causes issues on systems where the timestamp of that
file always returns the current time, since it means that the build.ninja
file is always out of date, and therefore needs to be rebuilt.

We can fix this by just using a simple shell script to return the coremask
appropriately for BSD and Linux, and removing that code logic from meson -
thereby hiding the use of the /sys file.

Fixes: c70622ac6f72 ("test: detect number of cores with meson")
Cc: stable@dpdk.org
Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
MAINTAINERS
app/test/get-coremask.sh [new file with mode: 0755]
app/test/meson.build