check/delete-old: Auto-generate lib32 entries for libraries.

[jhb]

Summary

• Add usr/lib32/libfoo.so.N for lib/libfoo.so.N.

• Add usr/lib32/foo for usr/lib/foo.

• Treat casper libraries special since they are installed to /usr/lib32 instead of /usr/lib32/casper and thus map usr/lib/casper/foo to usr/lib32/foo.

Note that OLD_DIRS entries are not duplicated, only OLD_FILES and
OLD_LIBS.

[jhb]

Remaining references to lib32 in *.inc. Most of these are the MK_LIB32 bits:

> grep lib32 ObsoleteFiles.inc tools/build/mk/OptionalObsoleteFiles.inc 
ObsoleteFiles.inc:# 20200601: OpenSSL 32-bit compat engines moved to /usr/lib32/engines
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/capi.so
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/padlock.so
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/lib4758cca.so
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/libaep.so
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/libatalla.so
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/libcapi.so
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/libchil.so
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/libcswift.so
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/libgost.so
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/libnuron.so
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/libsureware.so
ObsoleteFiles.inc:OLD_LIBS+=usr/lib32/libubsec.so
ObsoleteFiles.inc:OLD_DIRS+=usr/lib32/private
ObsoleteFiles.inc:OLD_FILES+=usr/lib/debug/usr/lib32/i18n
ObsoleteFiles.inc:OLD_FILES+=usr/lib/debug/usr/lib32/private
tools/build/mk/OptionalObsoleteFiles.inc:OLD_DIRS+=usr/lib32/dtrace
tools/build/mk/OptionalObsoleteFiles.inc:OLD_FILES+=etc/mtree/BSD.lib32.dist
tools/build/mk/OptionalObsoleteFiles.inc:.  if exists(${DESTDIR}/usr/lib32)
tools/build/mk/OptionalObsoleteFiles.inc:LIB32_DIRS!=find ${DESTDIR}/usr/lib32 -type d \
tools/build/mk/OptionalObsoleteFiles.inc:LIB32_FILES!=find ${DESTDIR}/usr/lib32 \! -type d \
tools/build/mk/OptionalObsoleteFiles.inc:LIB32_LIBS!=find ${DESTDIR}/usr/lib32 \! -type d \
tools/build/mk/OptionalObsoleteFiles.inc:.    if exists(${DESTDIR}/usr/lib/debug/usr/lib32)
tools/build/mk/OptionalObsoleteFiles.inc:DEBUG_LIB32_DIRS!=find ${DESTDIR}/usr/lib/debug/usr/lib32 -type d \
tools/build/mk/OptionalObsoleteFiles.inc:DEBUG_LIB32_FILES!=find ${DESTDIR}/usr/lib/debug/usr/lib32 \! -type d \
tools/build/mk/OptionalObsoleteFiles.inc:DEBUG_LIB32_LIBS!=find ${DESTDIR}/usr/lib/debug/usr/lib32 \! -type d \

Makefile.inc1

3225

So this is how I tested this was by starting with the stock version of this and generating before/after output and comparing those. I won't include the list-* targets in the commit, but kind of useful for them to exist in the history perhaps. Alternatively, it might reduce some duplication if the check/delete-old-* targets invoked list-old-* to generate the list of files.

[emaste]

What do you think about implementing check-old-* and delete-old-* by invoking list-old-*? Perhaps we could have only one instance of the :Mlib/*:S,^lib,usr/lib32,,

Makefile

157 ↗

(On Diff #99669)

Makefile defines existing check-old-{files,libs} as:

# check-old-files     - List obsolete files.
# check-old-libs      - List obsolete libraries.

We should add new targets to Makefile's comment, and update the description for those two.

ObsoleteFiles.inc

9

Should we add a note in this large comment about the automatically generated entries? (including already-existing handling for /usr/lib/debug)

[emaste]

Makefile.inc1

3225

Yes I was looking at this / entering my comments before I saw this new comment.

I think leaving them in and invoking them from check-old-* or delete-old-* is fine, or you could have delete-old-* invoke check-old-*

[jhb]

So I've found an annoying edge case. :( When a library moves from /usr/lib to /lib, we add a rule to remove /usr/lib32/libfoo.so.1 even though it is valid with this logic. (libbe.so.1 is an example of this)

I'm not quite sure how to resolve that edge case cleanly.

[jhb]

• Rebase on change to add list-old-* targets.

[emaste]

In D33324#753463, @jhb wrote:

I'm not quite sure how to resolve that edge case cleanly.

Ah, this is annoying. The best idea I've got is to introduce a new var (MOVED_LIBS or OLD_MOVED_LIBS, say) and specify libs that moved there, and then avoid the lib32 special case for them.

[jhb]

In D33324#753492, @emaste wrote:

In D33324#753463, @jhb wrote:

I'm not quite sure how to resolve that edge case cleanly.

Ah, this is annoying. The best idea I've got is to introduce a new var (MOVED_LIBS or OLD_MOVED_LIBS, say) and specify libs that moved there, and then avoid the lib32 special case for them.

So I've implemented MOVED_LIBS and added it as two earlier commits in this series (stack) and this now seems to work reliably (and finds several stale things on my desktop that aren't currently listed).

[jhb]

Ping.