The short option for --list is -l, not -1. The code that defines the
option is src/XrdApps/XrdMapCluster.cc line 481:
{OPT_TYPE "list", 1, 0, (int)'l'},
$ xrdmapc -1
xrdmapc: Invalid option, '-1'.
$ xrdmapc -l
xrdmapc: '-l' argument missing.
Add a zero-width space (\&) before 'all' at the start of the line, so
that the apostrophe at the start of the line is not interpreted as a
macro.
Fixes rpmlint warning:
$ rpmlint xrootd-client
xrootd-client.x86_64: W: manual-page-warning /usr/share/man/man1/xrdmapc.1.gz 21: warning: macro `all'' not defined
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
Also fixes broken rendering of the man page where the line with the
unknown macro is ignored.
Original:
OPTIONS
-1 | --list [all|m|s]
-q | --quiet
Does not print error messages to cerr; errors appear inline.
Updated:
OPTIONS
-l | --list [all|m|s]
'all' lists managers and servers (default), 'm' lists only man‐
agers and 's' lists only servers.
-q | --quiet
Does not print error messages to cerr; errors appear inline.
You can view, comment on, or merge this pull request online at:
https://github.com/xrootd/xrootd/pull/1201
-- Commit Summary --
* Make the xrdmapc help text and manpage match the code.
-- File Changes --
M docs/man/xrdmapc.1 (6)
M src/XrdApps/XrdMapCluster.cc (2)
-- Patch Links --
https://github.com/xrootd/xrootd/pull/1201.patch
https://github.com/xrootd/xrootd/pull/1201.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/pull/1201
########################################################################
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1
