From owner-cvs-src@FreeBSD.ORG Wed Aug 9 21:59:12 2006 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id B1D5416A4E0; Wed, 9 Aug 2006 21:59:12 +0000 (UTC) (envelope-from rik@inse.ru) Received: from mail.inse.ru (inse.ru [144.206.128.1]) by mx1.FreeBSD.org (Postfix) with ESMTP id 2D63B43D49; Wed, 9 Aug 2006 21:59:11 +0000 (GMT) (envelope-from rik@inse.ru) Received: from [127.0.0.1] (www.inse.ru [144.206.128.1]) by mail.inse.ru (Postfix) with ESMTP id 47FD633C66; Thu, 10 Aug 2006 01:59:10 +0400 (MSD) Message-ID: <44DA5C5A.70404@inse.ru> Date: Thu, 10 Aug 2006 02:06:18 +0400 From: Roman Kurakin User-Agent: Mozilla/5.0 (X11; U; Linux i686; ru-RU; rv:1.7.12) Gecko/20060103 ASPLinux/1.7.12-1.5.1.1asp X-Accept-Language: ru-ru, ru MIME-Version: 1.0 To: Robert Watson References: <200608061106.k76B6ahO042778@repoman.freebsd.org> <44D5F2C1.7080606@inse.ru> <20060808124221.GO54416@comp.chem.msu.su> <20060808142616.S45647@fledge.watson.org> <44D89ED3.3000805@inse.ru> <20060808152429.W45647@fledge.watson.org> In-Reply-To: <20060808152429.W45647@fledge.watson.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Roman Kurakin , src-committers@FreeBSD.org, Yar Tikhiy , cvs-all@FreeBSD.org, Ruslan Ermilov , cvs-src@FreeBSD.org Subject: Re: cvs commit: src/share/examples/drivers make_device_driver.sh X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 09 Aug 2006 21:59:12 -0000 Robert Watson: > > On Tue, 8 Aug 2006, Roman Kurakin wrote: > >> We could add a "foo" driver to the LINT generated by this script with >> notification that it was generated and all changes should be placed >> to script. > > I guess I can't help but wonder if a script is necessarily better than > a well-commented template. Today I've talked with ru@. He suggests to put an example to tree and change script to produce a copy of driver using example from tree as a template via sed or smth like that. It sounds like a good idea. We could have a single dir in subtree with various examples and a set of scripts that could produce from one of the examples a driver to start with. rik > > Robert N M Watson > Computer Laboratory > University of Cambridge > >> >> rik >> >> Robert Watson wrote: >> >>> >>> On Tue, 8 Aug 2006, Yar Tikhiy wrote: >>> >>>> On Sun, Aug 06, 2006 at 05:46:41PM +0400, Roman Kurakin wrote: >>>> >>>>> Example is for developers mostly, so I in doubts about MFC >>>>> compilation >>>>> fixup and this >>>>> one commit. >>>> >>>> >>>> I still can remeber that, when the world and I were younger, I used >>>> to study FreeBSD examples and sources for any release I had on hand >>>> at the moment. There still are plenty of places on the globe where >>>> off-line is the normal state of being, so we should provide good >>>> reference material for developers on our release CD's, to my mind. >>>> This also warrants MFC of fixes to source comments as long as they >>>> help to understand the code. Of course, all this is just my >>>> private opinion. >>> >>> >>> In general, I think we're better served by providing skeleton or >>> stub implementations of code that are part of the base source tree >>> (and compiled as part of it) than providing scripts and detached >>> sample source code. The reason is that any code not attached to the >>> tree almost instantly rots into uselessness. >>> >>> Robert N M Watson >>> Computer Laboratory >>> University of Cambridge >>> >>>> >>>>> Roman Kurakin: >>>>> >>>>>> rik 2006-08-06 11:06:36 UTC >>>>>> >>>>>> FreeBSD src repository >>>>>> >>>>>> Modified files: >>>>>> share/examples/drivers make_device_driver.sh >>>>>> Log: >>>>>> - Added ability to provide (optional) path to the kernel sources >>>>>> - Fixed path for ${s}/conf/files.FOO and a note that it should be >>>>>> merged >>>>>> into >>>>>> corresponding file to be able to compile the kernel >>>>>> - Changed kernel configuration example with the driver: it is not >>>>>> produced by >>>>>> copying Generic but by including it >>>>>> - Changed from automatic module building to asking an user if it >>>>>> whants to >>>>>> >>>>>> Reviewed by: julian@ >>>>>> >>>>>> Revision Changes Path >>>>>> 1.23 +32 -19 >>>>>> src/share/examples/drivers/make_device_driver.sh >>>>>> >>>>>> >>>> >>>> -- >>>> Yar >>>> >> >>