From owner-cvs-all Sun Jun 4 18:25:42 2000 Delivered-To: cvs-all@freebsd.org Received: from peach.ocn.ne.jp (peach.ocn.ne.jp [210.145.254.87]) by hub.freebsd.org (Postfix) with ESMTP id BF0AB37B6B9; Sun, 4 Jun 2000 18:25:36 -0700 (PDT) (envelope-from dcs@newsguy.com) Received: from newsguy.com (p53-dn01kiryunisiki.gunma.ocn.ne.jp [211.0.245.54]) by peach.ocn.ne.jp (8.9.1a/OCN/) with ESMTP id KAA03074; Mon, 5 Jun 2000 10:25:14 +0900 (JST) Message-ID: <393B01BF.E106AFD2@newsguy.com> Date: Mon, 05 Jun 2000 10:26:23 +0900 From: "Daniel C. Sobral" X-Mailer: Mozilla 4.7 [en] (Win98; I) X-Accept-Language: en,pt-BR,ja MIME-Version: 1.0 To: Doug Barton Cc: Warner Losh , "Matthew N. Dodd" , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org, Josef Karthauser Subject: Re: cvs commit: src/bin/ls extern.h ls.1 ls.c ls.h print.c References: <200006042215.QAA53873@harmony.village.org> <393ADBA7.7A2B5BF9@gorean.org> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG Doug Barton wrote: > > As far as I can see, the consensus at this point is as follows: > > 1. We can (sometimes grudgingly) live with color as an _option_ in > /bin/ls. > 2. The color options should be #ifdef'ed so that people who need a > slimmer ls can easily build one. (Thus anti-bloatists, picobsd, fixit > floppy people, etc.; can define "NOCOLORINLSNOWAYNOHOW" and be done with > it. :) > 3. First choice for 'ls -G' should be tgetstr(), with the ANSI codes > left in as a fallback if that returns NULL. (Thus handling the case of > no mounted /usr as well as possible.) Good summary, but there should be no ANSI fallback. Do not hardcode ANSI codes, period. I'll personally kill the first person who complains about not having colors on ls when booting single. -- Daniel C. Sobral (8-DCS) dcs@newsguy.com dcs@freebsd.org capo@yet.another.bsdconspiracy.org Hmmm - I have to go check this. My reality assumptions are shattered. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message