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

[Pádraig Brady]

On 27/12/2023 16:45, Jim Meyering wrote:
> 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.

Oops. Updated patch attached.

thanks,
Pádraig

chown-chgrp-merge.patch
Description: Text Data