Date: Sat, 24 Jan 2015 04:16:10 +0300 From: Dmitry Marakasov <amdmi3@amdmi3.ru> To: Garrett Cooper <yaneurabeya@gmail.com> Cc: freebsd-current@FreeBSD.org Subject: Re: OptionalObsoleteFiles.inc completeness improvement, try 2 Message-ID: <20150124011610.GJ1101@hades.panopticon> In-Reply-To: <4F0C24D1-9313-49BC-992E-5C31DAC6DADC@gmail.com> References: <20150124002956.GI1101@hades.panopticon> <4F0C24D1-9313-49BC-992E-5C31DAC6DADC@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
* 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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150124011610.GJ1101>