Page MenuHomeFreeBSD

OBJS_SRCS_FILTER
ClosedPublic

Authored by sjg on Sep 3 2017, 3:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 5, 6:02 AM
Unknown Object (File)
Dec 22 2023, 5:41 AM
Unknown Object (File)
Dec 20 2023, 4:56 AM
Unknown Object (File)
Nov 5 2023, 4:53 PM
Unknown Object (File)
Oct 4 2023, 3:50 PM
Unknown Object (File)
Aug 13 2023, 6:45 AM
Unknown Object (File)
Aug 1 2023, 2:22 PM
Unknown Object (File)
Aug 1 2023, 2:19 PM
Subscribers

Details

Summary

Allow makefiles to adjust OBJS_SRCS_FILTER to control setting of OBJS from SRCS
default R matches precedent, but makefile can add T to avoid subdirs in .OBJDIR

Test Plan

Buildworld ok
plus lib/libbearssl which uses subdirs in SRCS but not in .OBJDIR

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I'm curious what you need this for?

Also you may need to add it to DEPENDOBJS which is just which objects get -M flags at compilation and which ones are expected to have a _.depend.foo.o_ file to be read in. Could save some stats if it makes sense to filter them out.

share/mk/bsd.dep.mk:DEPENDOBJS+=        ${DEPENDSRCS:R:S,$,.o,}

Based on:

share/mk/bsd.dep.mk:DEPENDSRCS= ${SRCS:M*.[cSC]} ${SRCS:M*.cxx} ${SRCS:M*.cpp} ${SRCS:M*.cc}
This revision is now accepted and ready to land.Sep 15 2017, 2:52 AM

I'm curious what you need this for?

New library libbearssl does a reachover build of BearSSL, want to be able to represent the subdirs in SRCS but have no need of subdirs in .OBJDIR

Also you may need to add it to DEPENDOBJS which is just which objects get -M flags at compilation and which ones are expected to have a _.depend.foo.o_ file to be read in. Could save some stats if it makes sense to filter them out.

Thanks will take a look

In D12218#256751, @sjg wrote:

Also you may need to add it to DEPENDOBJS which is just which objects get -M flags at compilation and which ones are expected to have a _.depend.foo.o_ file to be read in. Could save some stats if it makes sense to filter them out.

I took a look at bsd.dep.mk most of it explicitly excludes SRCS with subdirs, and in another case it replaces / with _, so not sure there's anything to touch at all yet.

sjg edited the summary of this revision. (Show Details)
sjg edited the test plan for this revision. (Show Details)

Add bsd.dep.mk

This revision now requires review to proceed.Sep 15 2017, 6:03 AM
This revision is now accepted and ready to land.Sep 15 2017, 3:11 PM
This revision was automatically updated to reflect the committed changes.