Easy enough to accept the syntax documented in the man page, so accept the -I flag with or without a space between -I and the directory argument.
https://bugs.freedesktop.org/show_bug.cgi?id=34117 Signed-off-by: Alan Coopersmith <[email protected]> --- Updated to just warn & ignore as the other options do when no argument is given, as suggested in: https://bugs.freedesktop.org/show_bug.cgi?id=34117#c4 setxkbmap.c | 10 +++++++++- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git a/setxkbmap.c b/setxkbmap.c index e812a9c..511659b 100644 --- a/setxkbmap.c +++ b/setxkbmap.c @@ -414,7 +414,15 @@ parseArgs(int argc, char **argv) usage(argc, argv); exit(0); } - else if (strpfx(argv[i], "-I")) + else if (streq(argv[i], "-I")) /* space between -I and path */ + { + if ( ++i < argc ) + ok = addToList(&szInclPath, &numInclPath, &inclPath, argv[i]); + else + VMSG(0, "No directory specified on the command line\n" + "Trailing -I option ignored\n"); + } + else if (strpfx(argv[i], "-I")) /* no space between -I and path */ ok = addToList(&szInclPath, &numInclPath, &inclPath, &argv[i][2]); else if (streq(argv[i], "-keycodes")) ok = setOptString(&i, argc, argv, KEYCODES_NDX, FROM_CMD_LINE); -- 1.7.3.2 _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
