On Mon, Mar 29, 2021 at 07:42:18AM -0400, Jeff King wrote: > I.e., I'd suggest changing this: > > > else > > - die(_("unrecognized subcommand: %s"), argv[0]); > > +usage: > > + usage_with_options(builtin_multi_pack_index_usage, > > + builtin_multi_pack_index_options); > > to: > > error(_("unrecognized subcommand: %s"), argv[0]); > usage_with_options(...); Thanks, that's a helpful suggestion (and pretty easy to change, too). Thanks, Taylor