Re: [PATCH v4 2/2] attr: add flag `--revision` to work with revisions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Dec 21, 2022 at 10:13 PM Eric Sunshine <sunshine@xxxxxxxxxxxxxx> wrote:
>
> On Wed, Dec 21, 2022 at 3:58 PM Phillip Wood <phillip.wood123@xxxxxxxxx> wrote:
> > On 21/12/2022 13:47, Karthik Nayak wrote:
> > > Since we use a tree-ish object, the user can pass "--revision
> > > HEAD:subdirectory" and all the attributes will be looked up as if
> > > subdirectory was the root directory of the repository.
> >
> > We should be clear in the documentation and option help that --revision
> > takes a tree-ish (i.e. --revision=<tree-ish>). Maybe calling the option
> > --tree would be clearer.
>
> If that's the case, shouldn't this be called --source (or some
> variation thereof) for consistency[1] with git-restore?
>
> [1]: https://lore.kernel.org/git/CAPig+cT_xz3m-3kkL_Scf0opNXkikU94kSF46VO8KHkNMWbGzg@xxxxxxxxxxxxxx/
>

Let me rename it to source, and we can discuss on the next version if
the name suits or not.

-- 
- Karthik



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux