On May 26, 2010, at 12:20 PM, Jeff Eastman wrote: > On 5/26/10 7:55 AM, Grant Ingersoll wrote: >> On May 25, 2010, at 11:51 PM, Jake Mannix wrote: >> >> >>> >>>> 2) In getting a feel for Mahout, I've been running a few of the examples on >>>> my own, and have noticed that if I supply the "-h" argument by itself to >>>> some of the available programs, I get an exception, followed by the list of >>>> available options for that program. For instance, in running "./bin/mahout >>>> seq2sparse -h": >>>> >>>> >>> That is almost definitely a bug in either AbstractJob, or the Driver class >>> which is using it. File a JIRA ticket for it, and earn brownie points from >>> all of us (find out where in the code it's barfing, get even Bigger points, >>> and if you post too many patches fixing said problems, and you'll wind >>> up inadvertently becoming a committer!). >>> >> It's kind of a bug, kind of just laziness about creating the right kind of >> Option structure. It occurs out of the fact that we have required options, >> but we don't put them in a group, so the CLI parser pukes before it >> recognizes that there is a --help request. The fix, I suppose, is to figure >> out CLI2 handles saying something to the effect of either --help is required >> (and can be standalone) or if not a help request, then some other stuff may >> be required. >> > Even better than my crude hack idea, adding these 2 lines seems to resolve > the problem: > > Parser parser = new Parser(); > parser.setGroup(group); > + parser.setHelpFormatter(new HelpFormatter()); > + parser.setHelpOption(helpOpt); > CommandLine cmdLine = parser.parse(args); >
Sounds like we have a winner.
