Page MenuHomeFreeBSD
Differential D8642

usr.bin/Makefile: Make dtc depend on MK_FDT
AbandonedPublic
Actions

Authored by rakuco on Nov 25 2016, 9:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 3, 8:17 PM
Unknown Object (File)
Mon, Mar 25, 10:23 AM
Unknown Object (File)
Mon, Mar 25, 10:02 AM
Unknown Object (File)
Mon, Mar 25, 9:47 AM
Unknown Object (File)
Mon, Mar 25, 9:33 AM
Unknown Object (File)
Mon, Mar 25, 9:14 AM
Unknown Object (File)
Mon, Mar 25, 8:57 AM
Unknown Object (File)
Mar 8 2024, 7:17 AM
View All 29 Files
Subscribers
None

Details

Reviewers
emaste
theraven
imp
bdrewery
Summary

The code was only checking for MK_GPL_DTC, which means setting both WITHOUT_FDT and WITHOUT_GPL_DTC would result in dtc being built nonetheless.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 6062

Event Timeline

rakuco updated this revision to Diff 22517.Nov 25 2016, 9:29 PM
rakuco retitled this revision from to usr.bin/Makefile: Make dtc depend on MK_FDT.
rakuco updated this object.
rakuco edited the test plan for this revision. (Show Details)
rakuco added reviewers: theraven, imp, bdrewery.
rakuco set the repository for this revision to rS FreeBSD src repository - subversion.
rakuco added a reviewer: emaste.Nov 26 2016, 12:28 PM
theraven edited edge metadata.Nov 26 2016, 1:11 PM

Is this bad? Licensing aside, dtc is useful as a cross-build and cross-development tool. We use it extensively on x86 servers that make no use of FDT.

rakuco added a comment.Nov 26 2016, 1:47 PM

It's not bad per se; I thought this tool was only useful with the rest of the FDT code. I'm happy to drop this patch if that's not the case.

imp requested changes to this revision.Nov 27 2016, 6:26 PM
imp edited edge metadata.

FDT and DTC are different things. Please don't conflate them.

DTC is to build the compiler.
FDT is to enable support in the boot loader.

Though related, this change shouldn't be made.

This revision now requires changes to proceed.Nov 27 2016, 6:26 PM
rakuco abandoned this revision.Nov 27 2016, 8:40 PM

Thanks for the clarification, I'm closing the review.

Revision Contents
Changeset List

PathSize
usr.bin/
2 lines

Diff 22517

View Options

usr.bin/Makefile

Loading...