dnbaker / dashing2

Dashing 2 is a fast toolkit for k-mer and minimizer encoding, sketching, comparison, and indexing.
MIT License
62 stars 7 forks source link

`sketch -L` doesn't work #59

Closed BenLangmead closed 1 year ago

BenLangmead commented 2 years ago

The -L parameter for the sketch subcommand does not work as advertised. I get:

sketch: invalid option -- 'L'

Followed by usage message. If I replace -L with the long version of the parameter, --sketch-size-l2, it works fine. I took a brief look at the code and found the getopt struct for this option but didn't immediately see what might be wrong with it.

dnbaker commented 2 years ago

Found the problem - missing "L" in the getopt string. I've prepared its fix with a CLI sanity check (making sure there are no unexpected flags)