"kpm /?" doesn't print out help for kpm

[maumar]

This is a common way for people to ask for help (and works for kvm, for instance). However kpm doesn't accept this format.

This is what gets printed at the moment:

"Calls one batch program from another.
CALL [drive:][path]filename [batch-parameters]
batch-parameters Specifies any command-line information required by the
batch program.
If Command Extensions are enabled CALL changes as follows:
CALL command now accepts labels as the target of the CALL. The syntax
is:
CALL :label arguments
...
"

[davidfowl]

2 things:

1. maybe you have an older version of kpm
2. We don't support / for args, just - and -- because its more *nix friendly. / could be the start of the path as the argument value so we removed it as it causes problems on *nix

[ChengTian]

@davidfowl , I just checked with latest kpm. kpm /? is displaying help information of CALL because we get CALL ... /? after string concatenation in kpm.cmd and /? takes over the control. I can fix it soon.

[davidfowl]

What happens on osx and linux?

[Praburaj]

This needs to be fixed for both k and kpm

[ChengTian]

This change was reverted. Reopen here.

[Eilon]

Running any of the command line tools (k, kpm, kvm) without any arguments prints the help. They also all support -?, -h, --help to show the help. Some also support the format kpm help <command> for detailed command information.

@davidfowl @ChengTian @muratg @glennc do we need anything more here? We're not planning to support / as a command separator so I don't think there's anything left to do...

[ChengTian]

kpm /? should show invalid argument '/?', instead of help information of cmd.exe.

Since kpm.cmd is called with cmd.exe /C kpm.cmd <KPM_ARGS> while /? has a super high priority in the eyes of cmd.exe, /? makes cmd.exe show help information and kpm doesn't get called. There is a PR pending for this bug and I will merge it after adding functional tests.

[Praburaj]

Yes as long as it does not show weird stuff on typing /? it is enough.

[muratg]

We need to get this in for RC

[ChengTian]

This is not true any more after we reduce scripts in this commit 29e6e8a