Makefile
ClosedPublic
Actions

Authored by arichardson on Jul 8 2018, 10:08 PM.

Tags

None

Referenced Files

	F124974342: D16187.diff
	Fri, Aug 1, 11:20 PM

	Unknown Object (File)
	Thu, Jul 17, 7:31 PM

	Unknown Object (File)
	Wed, Jul 16, 7:06 AM

	Unknown Object (File)
	Fri, Jul 11, 9:03 PM

	Unknown Object (File)
	Wed, Jul 9, 10:17 PM

	Unknown Object (File)
	Wed, Jul 9, 9:52 AM

	Unknown Object (File)
	Mon, Jul 7, 8:03 AM

	Unknown Object (File)
	Mon, Jul 7, 1:12 AM

View All 71 Files

Subscribers

imp

Details

Reviewers

brooks
bdrewery
kevans

Commits

rS336335: No longer install sys/nv.h and sys/cnv.h in lib/libnv/Makefile

Summary

Use tools/build/Makefile to install the headers into ${WORLDTMP}/legacy
instead. Compared to r336026 this has the minor advantage that it avoids
unncessary header installation when building the non-bootstrap libnv.

Test Plan

Host tools still build

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

arichardson created this revision.Jul 8 2018, 10:08 PM

Harbormaster completed remote builds in B17921: Diff 45038.Jul 8 2018, 10:08 PM

A-ha! \0/ This is the solution I was searching for.

This revision is now accepted and ready to land.Jul 8 2018, 10:41 PM

@imp usually likes to see changes here.

tools/build/Makefile
52–54 ↗	(On Diff #45038)	The pattern above has been to use .PATH: ${SRCTOP}/sys/sys and then use SYSINCS+= nv.h and cnv.h. But if this works then fine.