RFR: 8325877: Split up NativeCompilation.gmk
Magnus Ihse Bursie
ihse at openjdk.org
Wed Feb 14 15:39:29 UTC 2024
The file NativeCompilation.gmk is a beast. It is one of the largest in the build system, and it is not very well organized. This makes it hard to read, understand, debug, edit and modify, especially since IDEs have a hard time helping out with makefiles, so you get very little overview or navigation support.
This patch will split up the file into several parts. The splits are somewhat arbitrary, but tries to keep things sort of logically connected, and make the chunks somewhat approximate equal size.
I've gone to great pains to make sure I do not accidentally change anything. The order for the code in each of these files are the same as in the original NativeCompilation.gmk. I have not rearranged any code (with a few trivially exceptions, moving some assignments to allow better grouping), and instead preferred to split up functionality in several parts (as in SetupBasicVariables1-3).
Since I include the files in alphabetic order, some code will inevitable switch places, but this should either be just defines (which are trivially safe to move around), or it should be code that are independent of each other.
My intention is to follow up this shuffling with more intrusive fixes, that can e.g. reorder stuff to make for more logical grouping. But I want to do that separately.
-------------
Commit messages:
- 8325877: Split up NativeCompilation.gmk
Changes: https://git.openjdk.org/jdk/pull/17849/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=17849&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8325877
Stats: 2547 lines in 7 files changed: 1420 ins; 1103 del; 24 mod
Patch: https://git.openjdk.org/jdk/pull/17849.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/17849/head:pull/17849
PR: https://git.openjdk.org/jdk/pull/17849
More information about the build-dev
mailing list