From owner-freebsd-current@FreeBSD.ORG Sat Jan 24 01:23:25 2015 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id E4B68C71 for ; Sat, 24 Jan 2015 01:23:24 +0000 (UTC) Received: from mail-pa0-x22f.google.com (mail-pa0-x22f.google.com [IPv6:2607:f8b0:400e:c03::22f]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id A9B758 for ; Sat, 24 Jan 2015 01:23:24 +0000 (UTC) Received: by mail-pa0-f47.google.com with SMTP id lj1so408431pab.6 for ; Fri, 23 Jan 2015 17:23:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=content-type:mime-version:subject:from:in-reply-to:date:cc :message-id:references:to; bh=3wjl8FS0r9XNMeqVvd35CxGxNx5I77qYUodksB1JnDs=; b=ZSzRao7uYmL4wV6bSMSeKVYN86rNIJbVKMxr0ObdSPsElkLyo+24dFlJcEfBf1BlMh GKoFcOUJreSunKKWvlLw48zlRHzt1vDgajk2fVQmGjpNBu669hPM/uV+x6BwUAMXg6kq w/lw0DVfIbB/8wyVPalnYHHLdL6YNwJxdulzN2VTRnU3fCkrnKxzr3kr1G3NdEFjNwJm SSCakYLCkBWRyVvgjzb0i3sZIgZgA7XlRkXT7exJZ25HNg5cv9C9OQYr68cmgfDp1LT1 mbCwLbdgEevGCOOGNEjL27MBPcilGyZ1d826rGBBpf6HKNMh+d93Q2hmeJv2RcNxkcDh dRMg== X-Received: by 10.68.203.195 with SMTP id ks3mr16068488pbc.104.1422062604331; Fri, 23 Jan 2015 17:23:24 -0800 (PST) Received: from [192.168.20.5] (c-98-247-240-204.hsd1.wa.comcast.net. [98.247.240.204]) by mx.google.com with ESMTPSA id ej7sm3126226pac.21.2015.01.23.17.23.23 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 23 Jan 2015 17:23:23 -0800 (PST) Content-Type: multipart/signed; boundary="Apple-Mail=_C22E1B49-1F7B-49E6-B4C3-C8054815AC7C"; protocol="application/pgp-signature"; micalg=pgp-sha512 Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\)) Subject: Re: OptionalObsoleteFiles.inc completeness improvement, try 2 From: Garrett Cooper In-Reply-To: <8162C656-E851-492D-840A-27A222E3A7CC@gmail.com> Date: Fri, 23 Jan 2015 17:23:22 -0800 Message-Id: <8D1D9451-B8B1-4121-8F89-E03951DB65BF@gmail.com> References: <20150124002956.GI1101@hades.panopticon> <4F0C24D1-9313-49BC-992E-5C31DAC6DADC@gmail.com> <20150124011610.GJ1101@hades.panopticon> <8162C656-E851-492D-840A-27A222E3A7CC@gmail.com> To: Dmitry Marakasov X-Mailer: Apple Mail (2.1878.6) 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:23:25 -0000 --Apple-Mail=_C22E1B49-1F7B-49E6-B4C3-C8054815AC7C Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=windows-1252 On Jan 23, 2015, at 17:20, Garrett Cooper wrote: > On Jan 23, 2015, at 17:16, Dmitry Marakasov wrote: >=20 >> * Garrett Cooper (yaneurabeya@gmail.com) wrote: >>=20 >>>> 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&re= v2=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=92ve duplicated work a bit. Have you looked at = ^/projects/building-blocks yet ? >>=20 >> 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 > I wrote this script to track what files get installed by directories:=20= > = https://svnweb.freebsd.org/base/projects/building-blocks/tools/add-optiona= l-obsolete-files-entries.sh?revision=3D275238&view=3Dmarkup . It=92s not = perfect (doesn=92t work for =93kitchen sink=94 directories like etc/ and = share/=85), but it=92s a reasonable starting point for grabbing all of = the files. >=20 > I=92m going to hold off on the rc.d deletions/rewrites for dependent = services after I do more targeted review/testing, as it might screw up = boot order in unexpected ways with services and programs that get run = out of order, but I=92m reasonably confident that the contents of the = branch work. >=20 > I was going to start cherry-picking changes from the branch this = weekend. Forgot to respond on one important point: reviewing and committing the = higher-level changes (make remove-old) shouldn=92t be a problem, and = I=92ll be sure to commit the overlapping changes with = OptionalObsoleteFiles.inc first to reduce the diff between our branches. Thanks! --Apple-Mail=_C22E1B49-1F7B-49E6-B4C3-C8054815AC7C Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=signature.asc Content-Type: application/pgp-signature; name=signature.asc Content-Description: Message signed with OpenPGP using GPGMail -----BEGIN PGP SIGNATURE----- Comment: GPGTools - https://gpgtools.org iQEcBAEBCgAGBQJUwvQKAAoJEMZr5QU6S73ecRIIAIDbkwf6RXRp0ag8Fy4rkLhF 67JD+JX2fETcQM3Igorq8VY0BF4rILIFdrDxxRpf8h+LzOqeGMw5Dv041HUl3PV1 5LhjBUTGbSsZuVk8w9iY+6WTN2w4dvHGjfoSCNtxc81BMirehmqR+/mEVi/TY6Xc jFK/Ru1xphzD21uwExRonhBPSXawQ48Ti+gl6uCaPLlDsiFN8X0v+k2JxxJkM5Q+ Tn4CpXEtNvdcoR9jR1Xr5U3hzJC3UUrA8rKj9BfsLZET6Y4i+UwWpfADdTjhOOEK pHafYWVHQmp2xUytF5Qx5zJD+dualnjuJiihVHTReHmRPh2yYcsfgrWtUtTMmlo= =djr9 -----END PGP SIGNATURE----- --Apple-Mail=_C22E1B49-1F7B-49E6-B4C3-C8054815AC7C--