From owner-cvs-src-old@FreeBSD.ORG Mon Jul 6 02:03:00 2009 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 10424106574D for ; Mon, 6 Jul 2009 02:03:00 +0000 (UTC) (envelope-from kientzle@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id EFAA58FC14 for ; Mon, 6 Jul 2009 02:02:59 +0000 (UTC) (envelope-from kientzle@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id n6622xl1068145 for ; Mon, 6 Jul 2009 02:02:59 GMT (envelope-from kientzle@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id n6622xul068144 for cvs-src-old@freebsd.org; Mon, 6 Jul 2009 02:02:59 GMT (envelope-from kientzle@repoman.freebsd.org) Message-Id: <200907060202.n6622xul068144@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to kientzle@repoman.freebsd.org using -f From: Tim Kientzle Date: Mon, 6 Jul 2009 02:02:45 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/usr.bin/cpio cmdline.c src/usr.bin/cpio/test test_owner_parse.c X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 06 Jul 2009 02:03:00 -0000 kientzle 2009-07-06 02:02:45 UTC FreeBSD src repository Modified files: usr.bin/cpio cmdline.c usr.bin/cpio/test test_owner_parse.c Log: SVN rev 195389 on 2009-07-06 02:02:45Z by kientzle This addresses some issues with my earlier -R fix that were pointed out by Brooks Davis and Alexey Dokuchaev: * It now tries to lookup arguments as names first, then tries to parse them as numbers. In particular, this makes the behavior consistent with POSIX conventions when usernames consist entirely of digits. * It now uses strtoul() for the numeric parsing. Finally, I've included an update to the test harness to exercise the new numeric cases for -R. Approved by: re (kib) Revision Changes Path 1.8 +24 -34 src/usr.bin/cpio/cmdline.c 1.3 +50 -32 src/usr.bin/cpio/test/test_owner_parse.c