Re: [PATCH] xfs: use -w instead of --path for man program portability

Zorro Lang <[email protected]>
Newsgroups org.kernel.vger.fstests
Message-ID <anpCryB4LUn6PCib@zlang-mailbox>
On Mon, Aug 03, 2026 at 01:58:19PM +0200, Anthony Iliopoulos wrote:
> Some implementations of man(1) do not support the long option --path to
> display the actual location of a man page, which can cause these tests
> to fail on systems using alternative man packages, such as mandoc.
> 
> Switch to using the more standard -w flag across these tests to improve
> portability while maintaining the same behavior, given that the --path
> option is just an alias to -w in the standard man-db utility.
> 
> Signed-off-by: Anthony Iliopoulos <[email protected]>
> ---

Looks harmless and effective,

Reviewed-by: Zorro Lang <[email protected]>

>  tests/xfs/293 | 2 +-
>  tests/xfs/505 | 2 +-
>  tests/xfs/514 | 2 +-
>  tests/xfs/515 | 2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/xfs/293 b/tests/xfs/293
> index 3e887f1c40cf..9735c441ab32 100755
> --- a/tests/xfs/293
> +++ b/tests/xfs/293
> @@ -19,7 +19,7 @@ _require_command "$MAN_PROG" man
>  
>  echo "Silence is golden"
>  
> -MANPAGE=`$MAN_PROG --path xfs_io`
> +MANPAGE=`$MAN_PROG -w xfs_io`
>  
>  case "$MANPAGE" in
>  *.gz|*.z\|*.Z)	CAT=zcat;;
> diff --git a/tests/xfs/505 b/tests/xfs/505
> index 81f53ffac23d..f0f435241dda 100755
> --- a/tests/xfs/505
> +++ b/tests/xfs/505
> @@ -16,7 +16,7 @@ _require_command "$MAN_PROG" man
>  
>  echo "Silence is golden"
>  
> -MANPAGE=$($MAN_PROG --path xfs_spaceman)
> +MANPAGE=$($MAN_PROG -w xfs_spaceman)
>  
>  case "$MANPAGE" in
>  *.gz|*.z\|*.Z)	CAT=zcat;;
> diff --git a/tests/xfs/514 b/tests/xfs/514
> index d3641c3e1af6..215ae5e7cc5a 100755
> --- a/tests/xfs/514
> +++ b/tests/xfs/514
> @@ -24,7 +24,7 @@ _require_test
>  
>  echo "Silence is golden"
>  
> -MANPAGE=$($MAN_PROG --path xfs_db)
> +MANPAGE=$($MAN_PROG -w xfs_db)
>  
>  case "$MANPAGE" in
>  *.gz|*.z\|*.Z)	CAT=zcat;;
> diff --git a/tests/xfs/515 b/tests/xfs/515
> index 7d6e99405972..0596457e5adc 100755
> --- a/tests/xfs/515
> +++ b/tests/xfs/515
> @@ -24,7 +24,7 @@ _require_test
>  
>  echo "Silence is golden"
>  
> -MANPAGE=$($MAN_PROG --path xfs_quota)
> +MANPAGE=$($MAN_PROG -w xfs_quota)
>  
>  case "$MANPAGE" in
>  *.gz|*.z\|*.Z)	CAT=zcat;;
> -- 
> 2.49.0
> 
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.