From owner-freebsd-hackers@FreeBSD.ORG Mon May 1 19:29:25 2006 Return-Path: X-Original-To: hackers@freebsd.org Delivered-To: freebsd-hackers@FreeBSD.ORG Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 2DE4A16A407 for ; Mon, 1 May 2006 19:29:25 +0000 (UTC) (envelope-from brdavis@odin.ac.hmc.edu) Received: from odin.ac.hmc.edu (Odin.AC.HMC.Edu [134.173.32.75]) by mx1.FreeBSD.org (Postfix) with ESMTP id D156443D55 for ; Mon, 1 May 2006 19:29:23 +0000 (GMT) (envelope-from brdavis@odin.ac.hmc.edu) Received: from odin.ac.hmc.edu (localhost.localdomain [127.0.0.1]) by odin.ac.hmc.edu (8.13.0/8.13.0) with ESMTP id k41JTLwF017370; Mon, 1 May 2006 12:29:21 -0700 Received: (from brdavis@localhost) by odin.ac.hmc.edu (8.13.0/8.13.0/Submit) id k41JTLDs017369; Mon, 1 May 2006 12:29:21 -0700 Date: Mon, 1 May 2006 12:29:20 -0700 From: Brooks Davis To: Eric Anderson Message-ID: <20060501192920.GE4315@odin.ac.hmc.edu> References: <444C51BA.3020905@centtech.com> <20060424131508.GB23163@pint.candc.home> <444CD48A.4060501@centtech.com> <444CE475.30104@centtech.com> <20060430231621.GA551@pint.candc.home> <44557F34.3020906@centtech.com> <20060501190645.GB4315@odin.ac.hmc.edu> <44565DD2.1020604@centtech.com> <20060501191447.GD4315@odin.ac.hmc.edu> <44565E74.3060801@centtech.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="+SfteS7bOf3dGlBC" Content-Disposition: inline In-Reply-To: <44565E74.3060801@centtech.com> User-Agent: Mutt/1.4.1i X-Virus-Scanned: by amavisd-new X-Spam-Status: No, hits=0.0 required=8.0 tests=none autolearn=no version=2.63 X-Spam-Checker-Version: SpamAssassin 2.63 (2004-01-11) on odin.ac.hmc.edu Cc: hackers@freebsd.org, Coleman Kane Subject: Re: [PATCH] Fancy rc startup style RFC X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 May 2006 19:29:25 -0000 --+SfteS7bOf3dGlBC Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, May 01, 2006 at 02:16:04PM -0500, Eric Anderson wrote: > Brooks Davis wrote: > >On Mon, May 01, 2006 at 02:13:22PM -0500, Eric Anderson wrote: > >>Brooks Davis wrote: > >>>On Sun, Apr 30, 2006 at 10:23:32PM -0500, Eric Anderson wrote: > >>>>Coleman Kane wrote: > >>>>>On Mon, Apr 24, 2006 at 09:45:09AM -0500, Eric Anderson wrote: > >>>>>>Eric Anderson wrote: > >>>>>> > >>>>>>Actually, some other things got changed somewhere in the history,= =20 > >>>>>>that broke some things and assumptions I was making. This patch ha= s=20 > >>>>>>them fixed, and I've tested it with all the different options: > >>>>>> > >>>>>>http://www.googlebit.com/freebsd/patches/rc_fancy.patch-9 > >>>>>> > >>>>>>It's missing the defaults/rc.conf diffs, but you should already kno= w=20 > >>>>>>those. > >>>>>> > >>>>>> > >>>>>>Eric > >>>>>> > >>>>>I have a new patch (to 7-CURRENT) of the "fancy_rc" updates. > >>>>> > >>>>>This allows the use of: > >>>>>rc_fancy=3D"YES" ---> Turns on fancy reporting (w/o color) > >>>>>rc_fancy_color=3D"YES" ---> Turns on fancy reporting (w/ color), n= eeds > >>>>> rc_fancy=3D"YES" > >>>>>rc_fancy_colour=3D"YES" ---> Same as above for you on the other sid= e of > >>>>> the pond. > >>>>>rc_fancy_verbose=3D"YES" --> Turn on more verbose activity messages. > >>>>> This will cause what appear to be "false > >>>>> positives", where an unused service is > >>>>> "OK" instead of "SKIP". > >>>>> > >>>>>You can also customize the colors, the widths of the message > >>>>>brackets (e.g. [ OK ] vs. [ OK ]), the screen width, and > >>>>>the contents of the message (OK versus GOOD versus BUENO). > >>>>> > >>>>>Also, we have the following message combinations: > >>>>>OK ---> Universal good message > >>>>>SKIP,SKIPPED ---> Two methods for conveying the same idea? > >>>>>ERROR,FAILED ---> Ditto above, for failure cases > >>>>> > >>>>>Should we just have 3 different messages, rather than 5 messages > >>>>>in 3 categories? > >>>>Yes, that's something that started with my first patch, and never got= =20 > >>>>ironed out. I think it should be: > >>>>OK > >>>>SKIPPED > >>>>FAILED > >>>>and possibly also: > >>>>ERROR > >>>> > >>>>The difference between FAILED and ERROR would be that FAILED means th= e=20 > >>>>service did not start at all, and ERROR means it started but had some= =20 > >>>>kind of error response. > >>>FAILED vs ERROR seems confusing. I'd be inclined toward WARNING vs > >>>FAILED or ERROR. > >>True, however I still see a difference between FAILED and WARNING. For= =20 > >>instance, as an example: a FAILED RAID is different than a RAID with a= =20 > >>WARNING. > > > >For that level of detail, the ability to provide additional output seems > >like the appropriate solution. >=20 > Yes, true, but you'd still want to show something (I would think) in the= =20 > [ ]'s to keep it consistent. My feeling is that anything short of complete success should report WARNING and a message unless it actually totally failed in which case FAILED or ERROR (I slightly perfer ERROR) should be used. -- Brooks --=20 Any statement of the form "X is the one, true Y" is FALSE. PGP fingerprint 655D 519C 26A7 82E7 2529 9BF0 5D8E 8BE9 F238 1AD4 --+SfteS7bOf3dGlBC Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (GNU/Linux) iD8DBQFEVmGPXY6L6fI4GtQRAoRRAKDe3Jz202bcirmwS8CRRBAVyDavggCcDzrx IivIEKdZACW9zfQ+BvVUjbU= =wT+/ -----END PGP SIGNATURE----- --+SfteS7bOf3dGlBC--