Page MenuHomeFreeBSD

Kill MFILES and find things automatically.
ClosedPublic

Authored by imp on Apr 19 2015, 4:30 AM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 27 2024, 6:58 AM
Unknown Object (File)
Jan 4 2024, 3:31 PM
Unknown Object (File)
Dec 28 2023, 2:45 AM
Unknown Object (File)
Dec 20 2023, 1:04 AM
Unknown Object (File)
Nov 12 2023, 2:25 PM
Unknown Object (File)
Nov 10 2023, 2:15 PM
Unknown Object (File)
Nov 8 2023, 12:55 AM
Unknown Object (File)
Nov 4 2023, 4:20 PM
Subscribers

Details

Summary

Kill MFILES and find things automatically. It turned out to be only
lightly used. Find the proper .m file when we depend on *_if.[ch]
in the srcs line, with seat-belts for false positive matches.

Diff Detail

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

Event Timeline

imp retitled this revision from to Kill MFILES and find things automatically..
imp updated this object.
imp edited the test plan for this revision. (Show Details)
jhb edited edge metadata.

Running lots of finds is perhaps not ideal. Not sure if you want to make find avoid recursing into 'compile' for people doing old-style builds still.

This revision is now accepted and ready to land.Apr 20 2015, 3:09 PM
imp edited edge metadata.

Better way of finding the .m files, though it introduces implicit
rules for some files.

This revision now requires review to proceed.Apr 28 2015, 9:35 PM
imp edited edge metadata.
  • Kill MFILES and find things automatically. It turned out to be only

arc loves to mess with me by doing all commits, not just the most
recent queue entry. Fix that.

sys/conf/kmod.mk
360 ↗(On Diff #5075)

This will still do a find for each module (probably N as each time the Makefile is processed for each of clean, obj, depend, install each module Makefile has to run find).

Why not let .PATH.m be static? This is less hardcoding than the current MFILES (e.g. sys/kern being in the path picks up most of them). I still think that simply hiding MFILES in this file and updating it when new foo_if.m's are added isn't that onerous. Having a static .PATH.m would have that simplicity while still being somewhat automagical.

This revision was automatically updated to reflect the committed changes.