From a9b4362fc52886c0d68a9af0956e8f21d8ddd25a Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Tue, 23 Apr 2019 11:22:57 +0200 Subject: tools: cleanup path variable in "tools/check-docs.sh" - don't append the path separator to the SOURCEDIR variable. Instead, use the path separator when we need it. --- tools/check-docs.sh | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/tools/check-docs.sh b/tools/check-docs.sh index b6fca6e0d9..04f03e76d3 100755 --- a/tools/check-docs.sh +++ b/tools/check-docs.sh @@ -23,9 +23,9 @@ same_lines() { libnm_headers() { ( - ls -1 "$1"libnm/*.h "$1"libnm-core/*.h + ls -1 "$1/libnm"/*.h "$1/libnm-core"/*.h if [ -n "$2" ]; then - ls -1 "$2"/libnm/*.h "$2"/libnm-core/*.h + ls -1 "$2/libnm"/*.h "$2/libnm-core"/*.h fi ) | sort | uniq } @@ -36,28 +36,28 @@ BUILDDIR="$2" if test "$SOURCEDIR" == "$BUILDDIR"; then BUILDDIR= fi -[ -n "$SOURCEDIR" ] && SOURCEDIR="$SOURCEDIR/" +[ -z "$SOURCEDIR" ] && SOURCEDIR='.' # Check that the D-Bus API docs contain all known interfaces -F1="$(sed -n 's,^ $,\1,p' "$SOURCEDIR"docs/api/network-manager-docs.xml)" +F1="$(sed -n 's,^ $,\1,p' "$SOURCEDIR/docs/api/network-manager-docs.xml")" F1_EXTRA=" org.freedesktop.NetworkManager.Device.WiMax.xml org.freedesktop.NetworkManager.WiMax.Nsp.xml " -F2="$(cd "$SOURCEDIR"introspection; ls -1 *.xml)" +F2="$(cd "$SOURCEDIR/introspection"; ls -1 *.xml)" if ! same_lines "$F1"$'\n'"$F1_EXTRA" "$F2" ; then die "*** Error: D-Bus interfaces not included in docs/api/network-manager-docs.xml ***" fi # Check that files that define types that are in public libnm API are included in libnm documentation. -F1="$(sed -n 's/.*