Force MK_INCLUDES for the legacy stage
ClosedPublic
Actions

Authored by emaste on Mar 25 2015, 8:17 PM.

Tags

None

Referenced Files

	F153381004: D2143.diff
	Mon, Apr 20, 8:01 PM

	F153336227: D2143.diff
	Mon, Apr 20, 1:47 PM

	Unknown Object (File)
	Sun, Apr 19, 1:34 PM

	Unknown Object (File)
	Sun, Apr 12, 10:26 AM

	Unknown Object (File)
	Fri, Apr 10, 11:40 PM

	Unknown Object (File)
	Tue, Apr 7, 7:11 PM

	Unknown Object (File)
	Tue, Mar 31, 9:22 PM

	Unknown Object (File)
	Wed, Mar 25, 8:22 AM

View All Files

Subscribers

markj

Details

Reviewers

ngie
imp

Commits

rS280631: Force MK_INCLUDES for the legacy stage

Summary

As legacy executes "make installincludes" we don't want a src.conf setting to disable it.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste updated this revision to Diff 4420.Mar 25 2015, 8:17 PM

emaste retitled this revision from to Force MK_INCLUDES for the legacy stage.

emaste updated this object.

emaste edited the test plan for this revision. (Show Details)

emaste added a reviewer: imp.

emaste added subscribers: ngie, markj.

Nice catch. Were we doing anythingwhere this made a difference before?

This revision is now accepted and ready to land.Mar 25 2015, 8:33 PM

In D2143#4, @imp wrote:

Nice catch. Were we doing anythingwhere this made a difference before?

I don't think so - I noticed it as a side effect of D2144

Closed by commit rS280631 (authored by @emaste).

Might have been because we required additional tools before to build include files. LGTM though :) (I don't think it's backportable in its current state because of the bsd.mkopt.mk refactor though)!

ngie added a reviewer: ngie.Mar 25 2015, 11:49 PM

ngie removed a subscriber: ngie.

Revision Contents
Changeset List

Path

Size

head/

Makefile.inc1

3 lines

Diff 4422

View Options

Force MK_INCLUDES for the legacy stageClosedPublicActions