devtools/test-meson-builds.sh: add support for FreeBSD

Message ID 20190225154001.62842-1-bruce.richardson@intel.com
State Accepted, archived
Delegated to: Thomas Monjalon
Headers show
Series
  • devtools/test-meson-builds.sh: add support for FreeBSD
Related show

Checks

Context Check Description
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/intel-Performance-Testing success Performance Testing PASS
ci/Intel-compilation success Compilation OK
ci/checkpatch success coding style OK

Commit Message

Bruce Richardson Feb. 25, 2019, 3:40 p.m.
readlink option "-m" is not supported on FreeBSD (checked on BSD 11),
so change to the largely-equivalent "-f" flag.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 devtools/test-meson-builds.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Monjalon Feb. 26, 2019, 5:04 p.m. | #1
25/02/2019 16:40, Bruce Richardson:
> readlink option "-m" is not supported on FreeBSD (checked on BSD 11),
> so change to the largely-equivalent "-f" flag.
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>

    Fixes: a55277a788df ("devtools: add test script for meson builds")
    Cc: stable@dpdk.org

Applied, thanks

Patch

diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index 3edc805f6..e59474d53 100755
--- a/devtools/test-meson-builds.sh
+++ b/devtools/test-meson-builds.sh
@@ -7,7 +7,7 @@ 
 # * if a build-directory already exists we assume it was properly configured
 # Run ninja after configuration is done.
 
-srcdir=$(dirname $(readlink -m $0))/..
+srcdir=$(dirname $(readlink -f $0))/..
 MESON=${MESON:-meson}
 use_shared="--default-library=shared"