From: Darrick J. Wong <djwong@xxxxxxxxxx> Geert Hendrickx reported an inconsistency between the xfs_admin manpage and its behavior -- the documentation says that users must provide the status explicitly, but the script injects '=1' anyway. While this seems to work with the glibc getsubopt, it's a bit ugly and isn't consistent with the docs. So, get rid of that extra two bytes. Reported-by: Geert Hendrickx <geert@xxxxxxxxxxxx> Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> --- db/xfs_admin.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/db/xfs_admin.sh b/db/xfs_admin.sh index 7a467dbe..02f34b73 100755 --- a/db/xfs_admin.sh +++ b/db/xfs_admin.sh @@ -20,7 +20,7 @@ do j) DB_OPTS=$DB_OPTS" -c 'version log2'";; l) DB_OPTS=$DB_OPTS" -r -c label";; L) DB_OPTS=$DB_OPTS" -c 'label "$OPTARG"'";; - O) REPAIR_OPTS=$REPAIR_OPTS" -c $OPTARG=1";; + O) REPAIR_OPTS=$REPAIR_OPTS" -c $OPTARG";; p) DB_OPTS=$DB_OPTS" -c 'version projid32bit'";; r) REPAIR_DEV_OPTS=" -r '$OPTARG'";; u) DB_OPTS=$DB_OPTS" -r -c uuid";;
