search

richardlehane / siegfried

signature-based file format identification
http://www.itforarchivists.com/siegfried
Apache License 2.0
217 stars 30 forks source link

Roy inspect FlagSet doesn't include -nopronom resulting in PRONOM IDs being added to identifiers #182

Closed ross-spencer closed 2 years ago

ross-spencer commented 2 years ago

Minor issue, but -nopronom isn't part of the inspect flag set which means an identifier will still try and add PRONOM ids to it before returning inspect results. As an optional flag, it can speed up inspection of results if the number of formats in an identifier is high.

ross-spencer commented 2 years ago

NB. Happy to submit the fix for this if you like Richard, I have a -inspectnopronom flag in my demo, but it's a little unwieldy. But maybe it's the most logical name?

richardlehane commented 2 years ago

thanks Ross, that'd be great. I'd go with -nopronom for the flag name to keep it symmetrical with the build command