Date: Mon, 4 Nov 2019 11:00:52 +0700 From: Alexey Dokuchaev <danfe@nsu.ru> To: John Baldwin <jhb@FreeBSD.org> Cc: freebsd-current@FreeBSD.org, jkim@FreeBSD.org Subject: Re: make delete-old: missing some files? Message-ID: <20191104040052.GA62001@regency.nsu.ru> In-Reply-To: <20191027124036.GA87210@regency.nsu.ru> References: <20191018170519.GA15695@regency.nsu.ru> <b204c470-1035-31bd-fcaa-f6dc1bb94de2@FreeBSD.org> <20191023034208.GA97072@regency.nsu.ru> <f589d031-4cf1-8c9e-1214-84807087250c@FreeBSD.org> <20191027124036.GA87210@regency.nsu.ru>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Oct 27, 2019 at 07:40:36PM +0700, Alexey Dokuchaev wrote: > On Thu, Oct 24, 2019 at 11:07:16AM -0700, John Baldwin wrote: > > On 10/22/19 8:42 PM, Alexey Dokuchaev wrote: > > > On Tue, Oct 22, 2019 at 04:34:53PM -0700, John Baldwin wrote: > > >> ... > > >> These are from the OpenSSL 1.1.1 commit. However, they are tagged as > > >> OLD_LIBS and check-old-libs and delete-old-libs should be automatically > > >> deleting these? Does 'make check-old' report these files as > > >> old libraries? > > > > > > I've manually placed one of those back on the filesystem and `make > > > check-old' reported it (twice!) under libraries. But after r353907 it > > > get cleaned up properly with `make delete-old'. > > > > Hmm, then 'make delete-old-libs' should already delete them without needing > > r353907. The issue with r353907 is if someone doesn't delete the > > actual libraries via 'make delete-old-libs' but then tries to debug an > > application that was using the old openssl and crashed, we'd no longer > > have debug symbols if the crash was in one of those libraries. That > > matters less for OpenSSL engines, but matters more for something like > > libutil, etc. hence why we delete debug symbols as part of delete-old-libs > > instead of delete-old. > > > > If 'make delete-old-libs' deletes these files already, then we should > > probably revert r353907. > > I've reverted r353907, and `make delete-old-libs' deleted those files. So what was the final decision, did you guys sort it out? I've seen that r353907 was MFC'ed while it was still being discussed, hence my question: https://svnweb.freebsd.org/base?view=revision&revision=354150. > Also, I've noticed that `lib/libdtrace.so.2' is listed twice in the > tools/build/mk/OptionalObsoleteFiles.inc, which is probably another bug. What about DTrace-related issues (this and the previous one)? ./danfe
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20191104040052.GA62001>