xfs/279: use MKFS_XFS_PROG, not hardcoded path

PATCH In review Eric Sandeen 2 participants 2 messages Jun 13, 2014 – Jun 16, 2014

lore

Share & tools

lore

Eric Sandeen

author

2014-06-13 21:10 UTC #0 lore raw

PATCH 1/1 xfs/279: use MKFS_XFS_PROG, not hardcoded path

I'm really not sure why this wasn't this way from the start. On some systems it's in /usr/sbin, and the test fails.

Signed-off-by Eric Sandeen <[email protected]>

---

tests/xfs/279 +1 -1

index 872f8be..cde7d21 100755
@@ -62,7 +62,7 @@ _check_mkfs()
 {
 	echo "===================" 
 	echo "mkfs with opts: $@" | sed -e "s,/dev/sd.,DEVICE,"
-	/sbin/mkfs.xfs $@ 2>/dev/null > $tmp.mkfs.full
+	$MKFS_XFS_PROG $@ 2>/dev/null > $tmp.mkfs.full
 	if [ $? -ne 0 ]; then
 		echo "Failed."
 		return

Christoph Hellwig

Reviewed-by on v1 1/1

2014-06-16 09:34 UTC #1 lore raw

Quoted 4 lines · Eric Sandeen

I'm really not sure why this wasn't this way from the start. On some systems it's in /usr/sbin, and the test fails.
Signed-off-by Eric Sandeen <[email protected]>

Looks good,

Reviewed-by Christoph Hellwig <[email protected]>