rtld-elf: Use ld-elf.so.1(1) rather than rtld(1) as the manpage title
AcceptedPublic
Actions

Authored by jrtc27 on Jul 27 2023, 10:13 PM.

Details

Reviewers

kib
brooks
jhb

Summary

This flips which is canonical to match the name of the linker file
itself and leaves rtld(1), like ld.so(1), as a convenient alias. This
uniformity will also allow us to more sensibly provide a specialised
ld-elf32.so.1(1) in a subsequent commit.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 52877
Build 49768: arc lint + arc unit

Event Timeline

jrtc27 created this revision.Jul 27 2023, 10:13 PM

Herald added subscribers: bdragon, imp. · View Herald TranscriptJul 27 2023, 10:13 PM

jrtc27 requested review of this revision.Jul 27 2023, 10:13 PM

Harbormaster completed remote builds in B52873: Diff 125261.Jul 27 2023, 10:13 PM

jrtc27 added a child revision: D41217: rtld-elf: Provide a separate specialised manpage for ld-elf32.so.1(1).Jul 27 2023, 10:15 PM

Move bump of date to here from D41217

Harbormaster completed remote builds in B52877: Diff 125265.Jul 27 2023, 10:18 PM

jrtc27 mentioned this in D41217: rtld-elf: Provide a separate specialised manpage for ld-elf32.so.1(1).Jul 27 2023, 10:18 PM

brooks accepted this revision.Jul 27 2023, 11:21 PM

This revision is now accepted and ready to land.Jul 27 2023, 11:21 PM

kib accepted this revision.Jul 28 2023, 1:11 AM

kib added inline comments.

libexec/rtld-elf/Makefile
55	I believe ld.so.1 is not relevant for long time. IFAIU, it was the name of the a.out Sun-style dynamic linker. Perhaps while changing the man page name, it is also worth to drop the unneeded alias?

jhb added inline comments.Jul 28 2023, 4:49 PM

libexec/rtld-elf/Makefile
55	I believe ld.so.1 is not relevant for long time. IFAIU, it was the name of the a.out Sun-style dynamic linker. Perhaps while changing the man page name, it is also worth to drop the unneeded alias? It's probably fine to drop, yes. Linux keeps the alias, but it was also for a.out binaries. The ELF rtld on Linux is ld-linux.so.