Page MenuHomeFreeBSD
Differential D32479

sys/conf: move libfdt path inclusion to kern.pre.mk
Needs ReviewPublic
Actions

Authored by ehem_freebsd_m5p.com on Oct 13 2021, 2:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 5, 6:32 AM
Unknown Object (File)
Tue, Oct 22, 10:21 AM
Unknown Object (File)
Tue, Oct 22, 10:21 AM
Unknown Object (File)
Tue, Oct 22, 10:20 AM
Unknown Object (File)
Tue, Oct 22, 10:10 AM
Unknown Object (File)
Oct 5 2024, 11:35 AM
Unknown Object (File)
Sep 28 2024, 8:08 PM
Unknown Object (File)
Sep 24 2024, 4:46 AM
View All 33 Files
Subscribers
andrew
emaste
imp
Contributor Reviews (src)
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

As all architectures now include libfdt, it should be in the common
Makefile, rather than duplicated in every architecture.

Switch to per-library include lines, the better to make future
adjustments smaller and simpler.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 42296
Build 39184: arc lint + arc unit

Event Timeline

ehem_freebsd_m5p.com created this revision.Oct 13 2021, 2:53 PM
Herald added subscribers: Contributor Reviews (src), emaste, andrew, imp. · View Herald TranscriptOct 13 2021, 2:53 PM
ehem_freebsd_m5p.com requested review of this revision.Oct 13 2021, 2:53 PM
Harbormaster completed remote builds in B42104: Diff 96811.Oct 13 2021, 2:53 PM
ehem_freebsd_m5p.com updated this revision to Diff 97271.Oct 22 2021, 5:08 AM

Updating in light of commit 8e53cd709943.

Harbormaster completed remote builds in B42296: Diff 97271.Oct 22 2021, 5:08 AM
imp added inline comments.Oct 22 2021, 5:34 AM
sys/conf/Makefile.amd64
34

This line can be removed here and elsewhere.

ehem_freebsd_m5p.com added inline comments.Oct 23 2021, 2:47 AM
sys/conf/Makefile.amd64
34

True. I thought it better to leave the empty line behind to keep the Makefile.<arch> files more consistent with each other (in case a new include needs to be added). Perhaps leave as commented lines?

I don't have strong feelings on this though, feel free to simply nuke the lines on commit.

Revision Contents
Changeset List

PathSize
sys/
conf/
2 lines
3 lines
2 lines
2 lines
2 lines
2 lines
2 lines
4 lines

Diff 97271

View Options

sys/conf/Makefile.amd64

Loading...
View Options

sys/conf/Makefile.arm

Loading...
View Options

sys/conf/Makefile.arm64

Loading...
View Options

sys/conf/Makefile.i386

Loading...
View Options

sys/conf/Makefile.mips

Loading...
View Options

sys/conf/Makefile.powerpc

Loading...
View Options

sys/conf/Makefile.riscv

Loading...
View Options

sys/conf/kern.pre.mk

Loading...