# Re: \[PATCH v3 12/15\] replay: disallow revision specific options and pathspecs

- From: Christian Couder \<hidden\>
- Date: 2023-09-07 17:51:33 
- List: git
- Page: https://ratatoskr.run/git/2023/09/7974517
- Thread: https://ratatoskr.run/git/2023/07/7972889/t.md

```text
On Tue, Jul 25, 2023 at 11:16 PM Junio C Hamano [ref] wrote:
>
> Christian Couder [ref] writes:
>
> > +     /*
> > +      * 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);

> > +             usage_with_options(replay_usage, replay_options);
> > +     }
```