On 04/05/2017 04:51 AM, Karel Zak wrote: > On Tue, Apr 04, 2017 at 09:03:08PM -0400, J William Piggott wrote: >> >> I don't understand this: >> >> 8< --- >> >> -o, --login-options "/path/command --option argument -- \u" >> Options that are passed to the login program. \u is replaced >> by the login name. The default is "/bin/login -- \u". >> 8< --- >> >> Is this saying the argument must include the login command? Is that what >> "/path/command" represents? If yes, then the second line is incorrect. >> If no, then the last sentence is incorrect. > > from code: > > login_argv[login_argc++] = options.login; > ... > > login_options_to_argv(login_argv, &login_argc, > options.logopt, username); > ... > execv(options.login, login_argv); > > > where: > options.login is --login-program > login-options is --login-options > > so the man page is wrong, --login-options should be without the path. > > I'll fix it. The man-page quote is from Sami's working branch (which I'm reviewing). PS It looks like your email didn't make it to the list for some reason. > > Karel > -- To unsubscribe from this list: send the line "unsubscribe util-linux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
