loader/efi: build with -Wall
ClosedPublic
Actions

Authored by vexeduxr on Sep 8 2025, 9:02 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Nov 28, 12:44 PM

	Unknown Object (File)
	Wed, Nov 26, 9:36 PM

	Unknown Object (File)
	Mon, Nov 24, 2:21 AM

	Unknown Object (File)
	Sat, Nov 22, 11:31 AM

	Unknown Object (File)
	Sun, Nov 16, 1:26 PM

	Unknown Object (File)
	Sat, Nov 8, 1:38 AM

	Unknown Object (File)
	Nov 4 2025, 2:15 AM

	Unknown Object (File)
	Nov 2 2025, 7:23 AM

View All 31 Files

Subscribers

bcran

dab

Details

Reviewers

imp
kevans
tsoome
jhb
manu

Commits

rGd81b84d6de65: loader/efi: build with -Wall

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 66905
Build 63788: arc lint + arc unit

Event Timeline

vexeduxr created this revision.Sep 8 2025, 9:02 PM

Herald added a reviewer: manu. · View Herald TranscriptSep 8 2025, 9:02 PM

Herald added subscribers: dab, bcran. · View Herald Transcript

vexeduxr requested review of this revision.Sep 8 2025, 9:02 PM

Harbormaster completed remote builds in B66905: Diff 161743.Sep 8 2025, 9:03 PM

Most errors generated by this are trivial to fix. I just opened reviews for the ones that weren't.

TBH, I would like to see whole loader being built with -Wall (and -Wextra;)

This revision is now accepted and ready to land.Sep 9 2025, 6:49 AM

The BIOS loader uses -Wall, but I'm not sure about the rest.

Closed by commit rGd81b84d6de65: loader/efi: build with -Wall (authored by vexeduxr). · Explain WhySep 9 2025, 5:30 PM

This revision was automatically updated to reflect the committed changes.

vexeduxr added a commit: rGd81b84d6de65: loader/efi: build with -Wall.

Revision Contents
Changeset List

Path

Size

stand/

efi/

loader/

Makefile

1 line

Diff 161743

View Options

loader/efi: build with -WallClosedPublicActions