RFR: 8326947: Minimize MakeBase.gmk
Magnus Ihse Bursie
ihse at openjdk.org
Wed Feb 28 11:32:07 UTC 2024
This is part of a general "spring cleaning" of the build system, addressing old code that has bit-rotted, been subject to lava flow, or just had bad or smelly code that we've never gotten around to fix.
This particular patch tries to make MakeBase truly minimal; only including the core parts of the build system that all makefiles will need. This is now limited to essential functionality for named parameter functions, variable dependency, tool execution, logging and fixpath functionality. MakeBase still includes Utils.gmk and FileUtils.gmk, and thus "provides" this functionality as well. Separating these out as well will be the subject of a future patch.
-------------
Commit messages:
- Whitespace fix
- MakeBase.gmk should not include MakeIO.gmk anymore
- MakeBase.gmk should not include CopyFiles.gmk anymore
- Reorder BaseUtils.gmk to make more sense
- Move some more functionality to BaseUtils.gmk
- Create BaseUtils.gmk with the most basic stuff
- Move all file stuff from Utils.gmk to FileUtils.gmk
- Document the purpose of MakeBase
- Move SOURCE_REVISION_TRACKER to where it is used.
- Move timers to InitSupport where they are used
- ... and 3 more: https://git.openjdk.org/jdk/compare/1ab6bd43...3c86bcfe
Changes: https://git.openjdk.org/jdk/pull/18041/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=18041&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8326947
Stats: 1099 lines in 43 files changed: 607 ins; 480 del; 12 mod
Patch: https://git.openjdk.org/jdk/pull/18041.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/18041/head:pull/18041
PR: https://git.openjdk.org/jdk/pull/18041
More information about the build-dev
mailing list