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.
