Differential D39636

config.mk: Add MK_VIMAGE knob
ClosedPublic
Actions

Authored by stevek on Apr 18 2023, 3:28 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Mar 31, 7:21 PM

	Unknown Object (File)
	Tue, Mar 31, 11:31 AM

	Unknown Object (File)
	Mon, Mar 30, 10:49 PM

	Unknown Object (File)
	Sat, Mar 21, 8:55 AM

	Unknown Object (File)
	Mon, Mar 9, 10:31 AM

	Unknown Object (File)
	Sun, Mar 8, 11:44 PM

	Unknown Object (File)
	Fri, Mar 6, 7:28 PM

	Unknown Object (File)
	Fri, Mar 6, 7:40 AM

View All 93 Files

Subscribers

Details

Reviewers

Commits

rG22ca6db50f4e: config.mk: Add MK_VIMAGE knob

Summary

Default to VIMAGE as yes.
Add VIMAGE to __DEFAULT_DEPENDENT_OPTIONS (to define VIMAGE_SUPPORT)

Only output VIMAGE to opt_global.h when VIMAGE support is wanted.

Obtained from: Juniper Networks, Inc.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

stevek created this revision.Apr 18 2023, 3:28 AM

Herald added subscribers: emaste, bdrewery. · View Herald TranscriptApr 18 2023, 3:28 AM

stevek requested review of this revision.Apr 18 2023, 3:28 AM

Harbormaster completed remote builds in B50983: Diff 120530.Apr 18 2023, 3:28 AM

stevek edited the summary of this revision. (Show Details)Apr 18 2023, 3:28 AM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 9 2024, 5:08 PM

Closed by commit rG22ca6db50f4e: config.mk: Add MK_VIMAGE knob (authored by stevek). · Explain Why

This revision was automatically updated to reflect the committed changes.

stevek added a commit: rG22ca6db50f4e: config.mk: Add MK_VIMAGE knob.

What is the value of MK_VIMAGE_SUPPORT over MK_VIMAGE? We now have an unused MK_VIMAGE. What will it control?

This commit is missing an addition of tools/build/options/WITHOUT_VIMAGE and WITHOUT_VIMAGE_SUPPORT and breaks GitHub CI because makeman spots the missing files.

imp added inline comments.Apr 22 2024, 8:53 PM

sys/conf/kern.opts.mk
187	This can't possibly work. There's no opt_vimage.h, so this just throws errors.

brooks added a reverting change: rG68cbb072fd1c: Revert "config.mk: Add MK_VIMAGE knob".Apr 25 2024, 9:44 PM

In D39636#1023799, @brooks wrote:

What is the value of MK_VIMAGE_SUPPORT over MK_VIMAGE? We now have an unused MK_VIMAGE. What will it control?

This commit is missing an addition of tools/build/options/WITHOUT_VIMAGE and WITHOUT_VIMAGE_SUPPORT and breaks GitHub CI because makeman spots the missing files.

Good point. I think just having a MK_VIMAGE would be sufficient to control the VIMAGE define in the opt_global.h

Revision Contents
Changeset List

Path

Size

share/

mk/

2 lines

sys/

conf/

2 lines

4 lines

Diff 136784

share/mk/src.opts.mk

Loading...

sys/conf/config.mk

Loading...

sys/conf/kern.opts.mk

Loading...