Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 20 Dec 2014 14:51:25 +0100
From:      Baptiste Daroussin <bapt@FreeBSD.org>
To:        Dag-Erling =?iso-8859-1?Q?Sm=F8rgrav?= <des@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r275768 - in head/usr.sbin: . vigr
Message-ID:  <20141220135125.GE21894@ivaldir.etoilebsd.net>
In-Reply-To: <201412141640.sBEGelm0072509@svn.freebsd.org>

index | next in thread | previous in thread | raw e-mail

[-- Attachment #1 --]
On Sun, Dec 14, 2014 at 04:40:47PM +0000, Dag-Erling Smørgrav wrote:
> Author: des
> Date: Sun Dec 14 16:40:46 2014
> New Revision: 275768
> URL: https://svnweb.freebsd.org/changeset/base/275768
> 
> Log:
>   Add a vigr(8) utility which does for /etc/group what vipw(8) does for
>   /etc/master.passwd.
> 
> Added:
>   head/usr.sbin/vigr/
>   head/usr.sbin/vigr/Makefile   (contents, props changed)
>   head/usr.sbin/vigr/vigr.8   (contents, props changed)
>   head/usr.sbin/vigr/vigr.sh   (contents, props changed)

I wonder why you chose making a script instead of using libutil? gr_* have now
the same level of features as of pw_* the benifit would have been to be able to
share the code with vipw(8) (maybe just have vigr(8) a hardlink so vipw(8)) and
to allow transparent change if one day he have a group db (aka gr_mkdb() is now
a simple rename(2) but could be changed to generate a db like pw_mkdb() does).

Best regards,
Bapt

[-- Attachment #2 --]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iEYEABECAAYFAlSVftcACgkQ8kTtMUmk6Eyj+QCgh3S3FfLYKHSmEGOaL+auvNkR
KM0AoJpN+4xm8Hto360Y6hC9ExGjTJT0
=BDIl
-----END PGP SIGNATURE-----
help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20141220135125.GE21894>