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

	F148233706: D24166.id69801.diff
	Mon, Mar 16, 4:50 PM

	F148232597: D24166.id69801.diff
	Mon, Mar 16, 4:40 PM

	Unknown Object (File)
	Sun, Mar 15, 9:54 AM

	Unknown Object (File)
	Thu, Mar 12, 3:56 AM

	Unknown Object (File)
	Wed, Mar 11, 11:35 PM

	Unknown Object (File)
	Mon, Mar 9, 1:26 PM

	Unknown Object (File)
	Mon, Mar 9, 12:06 PM

	Unknown Object (File)
	Sun, Mar 8, 6:26 PM

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

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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

head/

ObsoleteFiles.inc

5 lines

share/

mk/

3 lines

Diff 69816

head/ObsoleteFiles.inc

Loading...

head/share/mk/bsd.lib.mk

Loading...