RFR: 8326947: Minimize MakeBase.gmk

Magnus Ihse Bursie ihse at openjdk.org
Wed Feb 28 14:56:54 UTC 2024


On Wed, 28 Feb 2024 11:24:06 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

> 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.

The distinction was that BaseUtils.gmk are needed for MakeBase.gmk, Utils.gmk and FileUtils.gmk, but these three are not dependent on anything else. My initial goal was to stop having MakeBase.gmk include Utils.gmk as well, but it required a lot of extra includes and I did not want to do that in this PR. (Maybe it is not a good idea at all; I have not yet really decided).

I agree that the split seems a bit arbitrary. Maybe it was a bad idea. Let me ponder it for a while.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/18041#issuecomment-1969158791


More information about the build-dev mailing list