From owner-freebsd-current Sat Nov 16 13: 6:10 2002 Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id D31FB37B401 for ; Sat, 16 Nov 2002 13:06:07 -0800 (PST) Received: from lurza.secnetix.de (lurza.secnetix.de [212.66.1.130]) by mx1.FreeBSD.org (Postfix) with ESMTP id EF6C243E42 for ; Sat, 16 Nov 2002 13:06:06 -0800 (PST) (envelope-from olli@lurza.secnetix.de) Received: from lurza.secnetix.de (localhost [IPv6:::1]) by lurza.secnetix.de (8.12.6/8.12.5) with ESMTP id gAGL66dK050252 for ; Sat, 16 Nov 2002 22:06:06 +0100 (CET) (envelope-from oliver.fromme@secnetix.de) Received: (from olli@localhost) by lurza.secnetix.de (8.12.6/8.12.5/Submit) id gAGL65oo050251; Sat, 16 Nov 2002 22:06:05 +0100 (CET) Date: Sat, 16 Nov 2002 22:06:05 +0100 (CET) Message-Id: <200211162106.gAGL65oo050251@lurza.secnetix.de> From: Oliver Fromme To: freebsd-current@FreeBSD.ORG Reply-To: freebsd-current@FreeBSD.ORG Subject: Re: Asking for tester (small patch to chown(8)/chgrp(1)) In-Reply-To: <20021116200605.GA67266@bunrab.catwhisker.org> X-Newsgroups: list.freebsd-current User-Agent: tin/1.5.4-20000523 ("1959") (UNIX) (FreeBSD/4.7-RELEASE (i386)) MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG David Wolfskill wrote: > On Sat, Nov 16, 2002 at 12:29:20PM +0100, Oliver Fromme wrote: > > I've submitted a small patch (bin/45333) for both -stable > > and -current, but I haven't been able to test it under > > -current (due to lack of a spare machine). Would someone > > please give it a try and let me know if it compiles and > > works? > > Seems to for me (today's -CURRENT). Thanks! > I did take the liberty of removing the extraneous space on (the new) line > 151 (so its "if" lines up with those on lines 153 & 155), Oops, you're right, somehow there are spaces instead of tabs in that line (probably left over from copy&paste). > and of > replacing the 'usage:' on (the new) line 305 (chgrp's usage() message, > showing the second alternative invocation) with whitespace (to match the > pattern for chown). Right again, I missed that. > (If someone wants my diff, please let me know. > It's not sufficiently different from what Oliver put in his PR to > warrant spamming -current, as far as I'm concerned.) I've sent a small follow-up to the PR. > > The patch adds an option -r to chown(8) and chgrp(1), which > > does pretty much the same as the -r option of touch(1) and > > truncate(1). Basically, it let's you "copy" ownerships and > > group memberships from one file to another, which is useful > > in scripts. > > Makes sense. Only reservation I'd have -- and this may well be > unfounded -- is to be sure we're not introducing gratutitous > differences with respect to other implementations or to applicable > standards. I have no problem with functional differences that don't > break standards conformance (to the extent we claim it, anyway). I have checked a number of other operating systems including our brother and sister BSDs (Net- and Open-), Solaris and Tru64. There are no collisions nor similar functionality with different syntax. I also checked SUSv3 / POSIX, same result. On the other hand, GNU chown/chgrp (used under Linux) does have such functionality (what a surprise), _but_ only with a GNU-style long option: --reference=. There is no short single-letter option for it. Clearly, I decided not to introduce GNU-style long option into our chown/chgrp. ;-) > Hope this is useful. It is indeed, thanks! Regards Oliver -- Oliver Fromme, secnetix GmbH & Co KG, Oettingenstr. 2, 80538 München Any opinions expressed in this message may be personal to the author and may not necessarily reflect the opinions of secnetix in any way. "All that we see or seem is just a dream within a dream" (E. A. Poe) To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message