RFR: 8325877: Split up NativeCompilation.gmk [v2]
Magnus Ihse Bursie
ihse at openjdk.org
Wed Feb 14 15:44:27 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.
Magnus Ihse Bursie has updated the pull request incrementally with two additional commits since the last revision:
- Remove debug code (and fix one more space)
- Restore mistakenly deleted space
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/17849/files
- new: https://git.openjdk.org/jdk/pull/17849/files/9124f2a3..439ebf20
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=17849&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=17849&range=00-01
Stats: 4 lines in 1 file changed: 0 ins; 0 del; 4 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