Page MenuHomeFreeBSD

[[ my current integration branch, that also has refactor work ]] Unify boot1 with loader
AbandonedPublic

Authored by imp on Oct 13 2017, 3:23 PM.
Tags
None
Referenced Files
F82631113: D12659.diff
Wed, May 1, 1:01 AM
F82628374: D12659.diff
Wed, May 1, 12:41 AM
Unknown Object (File)
Mon, Apr 29, 3:02 PM
Unknown Object (File)
Fri, Apr 26, 5:51 AM
Unknown Object (File)
Thu, Apr 25, 5:43 PM
Unknown Object (File)
Jan 18 2024, 8:02 PM
Unknown Object (File)
Dec 12 2023, 9:40 PM
Unknown Object (File)
Nov 27 2023, 10:42 PM

Details

Reviewers
None
Summary

Refactor boot1 to use the same I/O code as /boot/loader uses. Refactor
to use the common efi_main.c.

Submitted by: Eric McCorkle

Move orphaned man pages into new man directory from common. This helps keep cleaer that common is just for the MI files for /boot/loader programs.

Sponsored by: Netflix

Rename top level Makefile.ficl to ficl.mk.

Move common/Makefile.inc to sys/boot/loader.mk.

Makefile.inc has a specific meaning in the tree, and
common/Makefile.inc doesn't quite fit into that. Rename it to
loader.mk and it will be a place to collect common things to all
/boot/loader programs there.

Sponsored by: Netflix

tweak style

create defs.mk for common definitions

Move all the ficl common code into ficl.mk

There's a number of copies of basically identical code to enable
building forth in /boot/loader. Move it all into ficl.mk.

LOADER_foo_SUPPORTED

Explicitly inlcude SYSDIR in the include path -- need machine path too?

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 12031
Build 12343: arc lint + arc unit

Event Timeline

Is this applied against some branch? I'm getting complaints about sys/boot/ficl.mk not being there

Is this applied against some branch? I'm getting complaints about sys/boot/ficl.mk not being there

The patch is against -current as of earlier today...

In D12659#262923, @imp wrote:

Is this applied against some branch? I'm getting complaints about sys/boot/ficl.mk not being there

The patch is against -current as of earlier today...

Looking at the raw diff shows something is amiss. I'll just update this with the one change.