Re: [PATCH] chgrp: --from parameter similar to chown

[Jim Meyering]

On Wed, Dec 27, 2023 at 8:20 AM Pádraig Brady <P@draigbrady.com> wrote:
> On 17/12/2023 14:17, Pádraig Brady wrote:
> > On 16/12/2023 22:01, Bernhard Voelker wrote:
> >> On 12/15/23 14:22, Pádraig Brady wrote:
> >>> On 31/10/2023 19:49, Pádraig Brady wrote:
> >>>> On 31/10/2023 17:51, Ed Neville wrote:
> >>>>> Hi there,
> >>>>>
> >>>>> Please could one of the maintainers take a look at this patch? Is it
> >>>>> ready for merge? It should be a straight forward change that brings the
> >>>>> same functionality from chown.
> >>>>>
> >>>>> Best regards,
> >>>>> Ed
> >>>>
> >>>> Yes I'll get to this one soon.
> >>>
> >>> I'll merge the attached later.
> >>> It's a complete patch adding docs, tests, some refactoring.
> >>
> >> The patch looks good ... yet didn't
> >>      chown --from=:SOMEGROUP :OTHERGROUP ...
> >> suffice?
> >
> > Heh I was thinking that myself.
> > But then what's the point of chgrp at all :)
> > We could (and perhaps should) treat chown/chgrp like we do for ls/dir.
>
> I'll push the attached later to merge chown/chgrp sources,
> which will avoid divergence in future.

Hi Pádraig! Happy holidays :-)
An important new file, chown.h, is not included in that diff.