From owner-freebsd-current@FreeBSD.ORG Sat Jan 24 01:16:35 2015 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id E767E95F for ; Sat, 24 Jan 2015 01:16:34 +0000 (UTC) Received: from smtpout1.timeweb.ru (smtpout1.timeweb.ru [92.53.117.15]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 977D5F3A for ; Sat, 24 Jan 2015 01:16:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=amdmi3.ru; s=dkim; h=Content-Transfer-Encoding:In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date; bh=EbUO3noukaSaHNj25c0YoyngEH+wVm4HFbPCN4tF+7Y=; b=CydahQbn7hN3WwmE3IRu/nhBUujvOqtzkr9/LeBczoM0WnsnMts2VLrcMNDvdHdMD7XrEVw8wRai9QiWT0Ez/QCilKjPyD5ai0yM8bupQclHGRu9KpeFHEUJz0IQunhr+EOl9OOUvYaFtdxSQx8z/xz5eq55HIPqZdt+jftWJ5U=; Received: from [213.148.20.85] (helo=hive.panopticon) by smtp.timeweb.ru with esmtpsa (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.76) (envelope-from ) id 1YEpKv-00063L-K7; Sat, 24 Jan 2015 04:16:29 +0300 Received: from hades.panopticon (hades.panopticon [192.168.0.32]) by hive.panopticon (Postfix) with ESMTP id E66BD2C5; Sat, 24 Jan 2015 04:15:54 +0300 (MSK) Received: by hades.panopticon (Postfix, from userid 1000) id 52E074D91; Sat, 24 Jan 2015 04:16:10 +0300 (MSK) Date: Sat, 24 Jan 2015 04:16:10 +0300 From: Dmitry Marakasov To: Garrett Cooper Subject: Re: OptionalObsoleteFiles.inc completeness improvement, try 2 Message-ID: <20150124011610.GJ1101@hades.panopticon> References: <20150124002956.GI1101@hades.panopticon> <4F0C24D1-9313-49BC-992E-5C31DAC6DADC@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <4F0C24D1-9313-49BC-992E-5C31DAC6DADC@gmail.com> User-Agent: Mutt/1.5.23 (2014-03-12) Content-Transfer-Encoding: quoted-printable Cc: freebsd-current@FreeBSD.org X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 24 Jan 2015 01:16:35 -0000 * Garrett Cooper (yaneurabeya@gmail.com) wrote: > > Some years ago I've started a project of improving > > OptionalObsoleteFiles.inc completeness, which allows make delete-old > > / delete-old-libs / delete-old-dirs targets completelty remove files > > which are normally installed when specific src.conf WITHOUT_* knobs > > are set. > >=20 > > In other words, if a user has some WITHOUT_* set in src.conf, > > specific files are not installed by installworld, but not removed > > by remove-old, which I try to fix. > >=20 > > In yet other words, I want to make it so `make installworld > > -DWITHOUT_foo=3Dyes` and `make installworld && make delete-old > > -DWITHOUT_foo=3Dyes` result in the very same file sets. > >=20 > > Though the project seems to be useful and have real demand (added > > to IdeasPage by netchild@, though removed later by brooks@ [1]) > > and interest ([2]), the change was ignored back then and now the > > patch is completely rotten. I can redo it, but I need a reviewer. > > Here's a first small part of the patch: > >=20 > > https://reviews.freebsd.org/D1600 > >=20 > > The WIP branch with other changes is [3] > >=20 > > Also there is a question of delete-old-dirs removing directories > > which are created by mtree run by installworld unconditionally. > > This seems to be incorrect - either directories should be installed > > conditionally or not removed by delete-old-dirs. My patch will > > address this issue as well, by not remiving unconditionally installed > > dirs. > >=20 > > [1] https://wiki.freebsd.org/action/diff/IdeasPage?action=3Ddiff&rev1= =3D260&rev2=3D261 > > [2] https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D168341#c6 > > [3] https://github.com/AMDmi3/freebsd/compare/obsolete-files >=20 > Hi Dmitry, > Seems like we=E2=80=99ve duplicated work a bit. Have you looked at ^/p= rojects/building-blocks yet ? Hm, seems so, partly. How do you gather missing entries? My way is pretty dumb, I just do bunch of installworlds + delete-old's and add diff to the file, you probably do it more cleverly. Will committing my changes interfere with your work? If so, it may be better to direct them to your branch instead. Especially if you are more aware of knob combinations and their effects. --=20 Dmitry Marakasov . 55B5 0596 FF1E 8D84 5F56 9510 D35A 80DD F9D2 F77D amdmi3@amdmi3.ru ..: jabber: amdmi3@jabber.ru http://www.amdmi3.ru