Differential D24166

bsd.lib.mk: Do not include bsd.incs.mk for INTERNALLIB
ClosedPublic
Actions

Authored by manu on Mar 23 2020, 1:48 PM.

Tags

None

Referenced Files

	F151691672: D24166.id69816.diff
	Fri, Apr 10, 1:31 AM

	F151658126: D24166.id69798.diff
	Thu, Apr 9, 8:06 PM

	Unknown Object (File)
	Thu, Apr 9, 12:50 AM

	Unknown Object (File)
	Mon, Apr 6, 8:59 AM

	Unknown Object (File)
	Sun, Mar 29, 12:39 AM

	Unknown Object (File)
	Wed, Mar 25, 6:05 AM

	Unknown Object (File)
	Wed, Mar 18, 11:10 PM

	Unknown Object (File)
	Wed, Mar 18, 6:43 AM

Subscribers

Details

Reviewers

emaste
pizzamig
bapt

Group Reviewers

Commits

rS359267: bsd.lib.mk: Do not include bsd.incs.mk for INTERNALLIB

Summary

If we're building an internal lib do not bother including bsd.incs.mk so we
will not install the headers.
This also "solves" a problem with pkgbase where a libXXX-development package
is created and due to how packages are created we add a dependency to a
libXXX package that doesn't exists.

Reported-by: pizzamig
Signed-off-by: Emmanuel Vadot <manu@freebsd.org>

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 30062
Build 27873: arc lint + arc unit

Event Timeline

manu created this revision.Mar 23 2020, 1:48 PM

Herald added a subscriber: bdrewery. · View Herald TranscriptMar 23 2020, 1:48 PM

Harbormaster completed remote builds in B30062: Diff 69798.Mar 23 2020, 1:48 PM

LGTM, might be worth a comment though

This revision is now accepted and ready to land.Mar 23 2020, 1:50 PM

manu mentioned this in D24162: libifconfig: Don't install the header.Mar 23 2020, 1:51 PM

manu mentioned this in D24163: libpmcstat: Don't install the header.

I do prefer than one over the other approach, don't forget the ObsoleteFiles.inc that goes with it now ;)

Add ObsoleteFiles.inc

This revision now requires review to proceed.Mar 23 2020, 2:11 PM

Harbormaster completed remote builds in B30063: Diff 69800.Mar 23 2020, 2:11 PM

Add comment about why we do not include bsd.incs.mk

Harbormaster completed remote builds in B30064: Diff 69801.Mar 23 2020, 2:13 PM

bapt accepted this revision.Mar 23 2020, 2:16 PM

This revision is now accepted and ready to land.Mar 23 2020, 2:16 PM

Closed by commit rS359267: bsd.lib.mk: Do not include bsd.incs.mk for INTERNALLIB (authored by manu). · Explain WhyMar 24 2020, 1:09 AM

This revision was automatically updated to reflect the committed changes.

manu added a commit: rS359267: bsd.lib.mk: Do not include bsd.incs.mk for INTERNALLIB.

Herald added a subscriber: imp. · View Herald TranscriptMar 24 2020, 1:09 AM

Revision Contents
Changeset List

Path

Size

share/

mk/

2 lines

Diff 69798

share/mk/bsd.lib.mk

Loading...