Page MenuHomeFreeBSD

loader: install help files only once
ClosedPublic

Authored by emaste on May 9 2023, 12:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 14, 5:30 PM
Unknown Object (File)
Mar 7 2024, 10:30 PM
Unknown Object (File)
Mar 7 2024, 10:30 PM
Unknown Object (File)
Mar 7 2024, 10:30 PM
Unknown Object (File)
Mar 7 2024, 10:30 PM
Unknown Object (File)
Mar 7 2024, 10:17 PM
Unknown Object (File)
Dec 22 2023, 11:43 PM
Unknown Object (File)
Dec 12 2023, 12:50 PM
Subscribers

Details

Summary

PR: 271178

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Happy to defer to an alternative approach if someone wants to come up with a cleaner way.

This is simple, and makes sense to me. I think you should be consistent about which interpreter variant is the "yes" one; for efi and userboot it is "simp" but for i386 it is "4th".

If I understand things correctly, we only build one variant for kboot, ubldr, and ofw loader, so they should be unaffected.

This is simple, and makes sense to me. I think you should be consistent about which interpreter variant is the "yes" one; for efi and userboot it is "simp" but for i386 it is "4th".

Oh yes, it should be simp for i386 as well. The "yes" one should be the one that is always installed e.g. SUBDIR.yes+= loader_simp

If I understand things correctly, we only build one variant for kboot, ubldr, and ofw loader, so they should be unaffected.

Correct.

install i386 help files via loader_simp

Id prefer we do all of them in simp.
I'm not sure I like this. It's kinda fugly. But i have no better idea off the top of my head, but I'll mull this over...

I'm not sure I like this. It's kinda fugly.

I don't disagree - as I said I'm happy to defer to something nicer.

With this change applied if_wg.h (PR271266) is the only file appearing twice.

This revision is now accepted and ready to land.May 9 2023, 11:59 PM
This revision was automatically updated to reflect the committed changes.