Main parameter doesn't support Converter

The converter isn’t called and I get an Exception: Exception in thread “main” com.beust.jcommander.ParameterException: Could not invoke null Reason: Can not set java.nio.file.Path field mypackage.MyCommand.file to java.lang.String

@Parameter(description = "File", converter = PathConverter.class) private Path file;

(The PathConverter is from JCommander)

If I add a name to the Parameter (like -f) then the converter works like a charm.

Whole class: MyCommand.txt

[cbeust]

1.74 was released about a month ago:

https://bintray.com/cbeust/maven/jcommander/1.74

[antonio-rodriges]

Thanks for adding converter capability to parse the main param: https://github.com/cbeust/jcommander/blob/05254453c0a824f719bd72dac66fa686525752a5/src/main/java/com/beust/jcommander/JCommander.java#L766-L771

However, the code does not work if we provide IStringConverterFactory. A workaround is to specify a dummy converter class for the main parameter that cannot be instantiated.

I noticed the solution above Added a check in JCommander.java testing if Converter is specified and is not a NoConverter.

But the condition https://github.com/cbeust/jcommander/blob/05254453c0a824f719bd72dac66fa686525752a5/src/main/java/com/beust/jcommander/JCommander.java#L769

prevents using the IStringConverterFactory. It would be good to call convertValue anyway. All in all, if there is no appropriate converter, it would just return the same string https://github.com/cbeust/jcommander/blob/05254453c0a824f719bd72dac66fa686525752a5/src/main/java/com/beust/jcommander/JCommander.java#L1373

However, to avoid breaking tests without the condition, should the if clause be extended to account for IStringConverterFactory?