From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1KaTo2-00073N-3l for mharc-grub-devel@gnu.org; Tue, 02 Sep 2008 07:12:18 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KaTo0-00073E-Cj for grub-devel@gnu.org; Tue, 02 Sep 2008 07:12:16 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KaTnv-00072D-Qf for grub-devel@gnu.org; Tue, 02 Sep 2008 07:12:16 -0400 Received: from [199.232.76.173] (port=51562 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KaTnv-000729-L5 for grub-devel@gnu.org; Tue, 02 Sep 2008 07:12:11 -0400 Received: from moutng.kundenserver.de ([212.227.126.177]:61828) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1KaTnv-0003sZ-2e for grub-devel@gnu.org; Tue, 02 Sep 2008 07:12:11 -0400 Received: from [85.180.13.152] (e180013152.adsl.alicedsl.de [85.180.13.152]) by mrelayeu.kundenserver.de (node=mrelayeu8) with ESMTP (Nemesis) id 0ML31I-1KaTnt0eYL-00026s; Tue, 02 Sep 2008 13:12:09 +0200 From: Felix Zielcke To: The development of GRUB 2 In-Reply-To: <1220148681.7450.14.camel@dv> References: <1219403324.4450.37.camel@fz.local> <20080822125823.GC20330@thorin> <1219410801.4450.56.camel@fz.local> <1220050444.11557.11.camel@fz.local> <1220051014.7979.6.camel@dv> <1220088538.10913.9.camel@fz.local> <1220148681.7450.14.camel@dv> Content-Type: multipart/mixed; boundary="=-g/76LwvFe2O0udEvtMSg" Date: Tue, 02 Sep 2008 13:12:08 +0200 Message-Id: <1220353928.4101.26.camel@fz.local> Mime-Version: 1.0 X-Mailer: Evolution 2.22.3.1 X-Provags-ID: V01U2FsdGVkX1+Zit3b/zq3dBzJUVv0O7GNn6VAQ5Qpk4FxFzW /zZx78eyjDGKyTzxX14pUFAF63BuvZFjI2dx0K+DilZwrZ3XbC qwO7STpi9fCpuhmcyDulnWFq4Z4g6dg X-detected-kernel: by monty-python.gnu.org: Linux 2.6? (barebone, rare!) Subject: Re: [RFC] DISTLIST and gendistlist.sh X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: The development of GRUB 2 List-Id: The development of GRUB 2 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 02 Sep 2008 11:12:16 -0000 --=-g/76LwvFe2O0udEvtMSg Content-Type: text/plain Content-Transfer-Encoding: 7bit Am Samstag, den 30.08.2008, 22:11 -0400 schrieb Pavel Roskin: > > svn ls -R | sed '/\/$/d' | sort > DISTLIST2 Thanks for the regexp. The sort is actual not needed, `svn' does this and it places even the files with a capital letter above everything else just like gendistlist.sh. > > This seems to be the best then nobody needs to remember to > > update ./gendistlist.sh and current DISTLIST seems again already a bit > > out of date ;) > Thus, we don't need any lists for included or excluded files. Things > will change if we stop keeping dependent files in Subversion. This was just a little hint from me that some files were recently again added to SVN without adding them to DISTLIST by e.g. calling gendistlist.sh But that's the point why I do this whole thing now :) Attached is now a patch which adds ./gendistlist.sh call to autogen.sh and uses SVN in case it is avaible to generate it. I think this now needs the decision of Marco and Okuji because the DISTLIST file is actual used by them i.e. `make dist' I think using `svn ls' is really great. But 2 questions come now in my mind: `make dist' already creates DISTLIST if it does not exist, so should ./autogen.sh generate it or directly `make dist' should autogen.sh/`make dist' use directly SVN or should it be in gendistlist.sh? Especially should ony SVN be used to generate it or should `make dist' be actual supported for everyone? For me it seems that DISTLIST thing is actually only useful for Okuji and Marco or whoever in the future will maintain GRUB 2 -- Felix Zielcke --=-g/76LwvFe2O0udEvtMSg Content-Disposition: attachment; filename=distlist.diff.2 Content-Type: text/plain; name=distlist.diff.2; charset=UTF-8 Content-Transfer-Encoding: 7bit 2008-08-02 Felix Zielcke * autogen.sh: Call `gendistlist.sh'. * gendistlist.sh: Update to use `svn' in case it is avaible. Index: autogen.sh =================================================================== --- autogen.sh (Revision 1845) +++ autogen.sh (Arbeitskopie) @@ -8,5 +8,6 @@ echo timestamp > stamp-h.in for rmk in conf/*.rmk; do ruby genmk.rb < $rmk > `echo $rmk | sed 's/\.rmk$/.mk/'` done +./gendistlist.sh > DISTLIST exit 0 Index: gendistlist.sh =================================================================== --- gendistlist.sh (Revision 1845) +++ gendistlist.sh (Arbeitskopie) @@ -13,6 +13,9 @@ # Generate a list of distributed files. +if test -d .svn ; then + svn ls -R | sed '/\/$/d' +fi EXTRA_DISTFILES="AUTHORS COPYING ChangeLog DISTLIST INSTALL NEWS README \ THANKS TODO Makefile.in aclocal.m4 autogen.sh config.guess \ config.h.in config.sub configure configure.ac gencmdlist.sh \ --=-g/76LwvFe2O0udEvtMSg--