From owner-svn-src-all@FreeBSD.ORG Thu Dec 16 14:25:08 2010 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 372B3106575B; Thu, 16 Dec 2010 14:25:08 +0000 (UTC) (envelope-from bz@FreeBSD.org) Received: from mail.cksoft.de (mail.cksoft.de [IPv6:2001:4068:10::3]) by mx1.freebsd.org (Postfix) with ESMTP id 3FB7E8FC0A; Thu, 16 Dec 2010 14:25:07 +0000 (UTC) Received: from localhost (amavis.fra.cksoft.de [192.168.74.71]) by mail.cksoft.de (Postfix) with ESMTP id 0B26D41C6B4; Thu, 16 Dec 2010 15:25:06 +0100 (CET) X-Virus-Scanned: amavisd-new at cksoft.de Received: from mail.cksoft.de ([192.168.74.103]) by localhost (amavis.fra.cksoft.de [192.168.74.71]) (amavisd-new, port 10024) with ESMTP id ypKUzdRJkd9W; Thu, 16 Dec 2010 15:25:05 +0100 (CET) Received: by mail.cksoft.de (Postfix, from userid 66) id 8AC0C41C6A7; Thu, 16 Dec 2010 15:25:05 +0100 (CET) Received: from maildrop.int.zabbadoz.net (maildrop.int.zabbadoz.net [10.111.66.10]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.int.zabbadoz.net (Postfix) with ESMTP id 232304448F3; Thu, 16 Dec 2010 14:22:30 +0000 (UTC) Date: Thu, 16 Dec 2010 14:22:30 +0000 (UTC) From: "Bjoern A. Zeeb" X-X-Sender: bz@maildrop.int.zabbadoz.net To: src-committers@freebsd.org In-Reply-To: <201012161158.oBGBwoep051709@svn.freebsd.org> Message-ID: <20101216135547.B6126@maildrop.int.zabbadoz.net> References: <201012161158.oBGBwoep051709@svn.freebsd.org> X-OpenPGP-Key: 0x14003F198FEFA3E77207EE8D2B58B8F83CCF1842 MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org Subject: Re: svn commit: r216483 - head X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 16 Dec 2010 14:25:08 -0000 On Thu, 16 Dec 2010, Shteryana Shopova wrote: Hi, this is not directly related to this commit but I admit I looked at it before the commit and hadn't spotted it from just the diff and am hell of confused now that I look at the final result basically because I hadn;t realy looked at the file in a while. The file says: # The file is partitioned: OLD_FILES first, then OLD_LIBS and OLD_DIRS last. but it seems since 2007 only few people actually followed this (and I guess I didn't either allways) and the best we try is: 1. order by date 2. if lucky go by the above rules for the entry under a given date. This probably is because everyone stopped reading after the 2nd sentence which says "Recently removed entries first (with the date as a comment).". You can check http://svn.freebsd.org/viewvc/base/head/ObsoleteFiles.inc?annotate=148330&pathrev=216483 to see how it was originally designed to be. So this commit actually tried to do it right, kind of and was just trapped on the intermediate breakage, not making it better or worse. I am happy people have lately started to care about duplicates etc. but can we fix the description or enforce a proper ordering as well to avoid further confusion like this. I think grouping by date and then by file/lib/dir is actually better as it'll keep things logically together rather than possibly splitting it over 3 sections? I would suggest if we'd formerlly switch, that we will create a bar line for everything below the current one with a comment along the lines of: ######################################################################## # # Everything below was supposed to be in 3 different sections with all # OLD_FILES first, followed by all OLD_LIBS and all OLD_DIRS last (which # didn't work out well). Do not touch it unless you really need to. # And adopting the comment at the top to say something along the lines: # The file is chronologically sorted with newest entries at the top. # For a certain date one may add multiple different entries in which # case they should be sorted in order of commits with the latest one # first, e.g. # # 20101216 remove baz # ... # # 20101216 remove bar # ... # # 20101216 remove foo # ... # Each such entry should list OLD_FILES first, OLD_LIBS second and # OLD_DIRS last, e.g. # # 20101216 remove baz # OLD_LIBS+= /lib/libkvm.so.5 # # 20101216 remove bar # OLD_FILES+= /bin/sh # OLD_DIRS+= /dev # # 20101216 remove foo # OLD_FILES+= /boot/kernel/kernel # OLD_FIELS+= /boot/loader # OLD_FILES+= /sbin/init # OLD_FILES+= /usr/lib/libc.a # OLD_LIBS+= /lib/libc.so.7 # OLD_DIRS+= /root You get the idea. Any takers? > Author: syrinx > Date: Thu Dec 16 11:58:50 2010 > New Revision: 216483 > URL: http://svn.freebsd.org/changeset/base/216483 > > Log: > libbsnmp was moved to usr/lib > > Modified: > head/ObsoleteFiles.inc > > Modified: head/ObsoleteFiles.inc > ============================================================================== > --- head/ObsoleteFiles.inc Thu Dec 16 11:20:37 2010 (r216482) > +++ head/ObsoleteFiles.inc Thu Dec 16 11:58:50 2010 (r216483) > @@ -284,6 +284,8 @@ OLD_FILES+=usr/share/man/man9/ieee80211_ > OLD_FILES+=usr/share/man/man9/ieee80211_wep_crypt.9.gz > # 20090801: vimage.h removed in favour of vnet.h > OLD_FILES+=usr/include/sys/vimage.h > +# 20101208: libbsnmp was moved to usr/lib > +OLD_LIBS+=lib/libbsnmp.so.5 > # 20090719: library version bump for 8.0 > OLD_LIBS+=lib/libalias.so.6 > OLD_LIBS+=lib/libavl.so.1 > -- Bjoern A. Zeeb Welcome a new stage of life. Going to jail sucks -- All my daemons like it! http://www.freebsd.org/doc/en_US.ISO8859-1/books/handbook/jails.html