On Tue, Jul 25, 2023 at 11:16 PM Junio C Hamano [off-list ref] wrote:
Christian Couder [off-list ref] writes:
quoted
+ /*
+ * Reject any pathspec. (They are allowed and eaten by
+ * setup_revisions() above.) In the future we might accept
+ * them, after adding related tests and doc though.
+ */
+ if (revs.prune_data.nr) {
+ error(_("invalid pathspec: %s"), revs.prune_data.items[0].match);
This made me waste a few minutes wondering if and how I misspelt my
pathspec elements. If we mean "no pathspec is allowed", we should
say so instead.
Yeah, right. I have changed this to:
error(_("no pathspec is allowed: '%s'"),
revs.prune_data.items[0].match);
quoted
+ usage_with_options(replay_usage, replay_options);
+ }