> We are setting one of the flags `CFLAGS_JDKLIB`, `CXXFLAGS_JDKLIB`, 
> `CFLAGS_JDKEXE` or `CXXFLAGS_JDKEXE` to `CFLAGS` or `CXXFLAGS`, respectively, 
> in basically all calls to `SetupJdkLibrary` and `SetupJdkExecutable`.
> 
> These flag variables contain a lot of duplication.
> 
> The first step towards bringing some sanity to this mess is to move the 
> setting of these variables into `SetupJdkLibrary/SetupJdkExecutable`.
> 
> In a few places these standard flags are not set, partially or fullly. This 
> is handled by the new arguments `DEFAULT_CFLAGS := false` (to disable the 
> entire setting) and `C[XX]FLAGS_FILTER_OUT` (which excludes some specific 
> flag) to `SetupJdkLibrary/SetupJdkExecutable`.

Magnus Ihse Bursie has updated the pull request incrementally with one 
additional commit since the last revision:

  Restore -MT to jpackage libs

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/18301/files
  - new: https://git.openjdk.org/jdk/pull/18301/files/feb192d9..2cbe7416

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=18301&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=18301&range=03-04

  Stats: 4 lines in 1 file changed: 0 ins; 0 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/18301.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/18301/head:pull/18301

PR: https://git.openjdk.org/jdk/pull/18301

Reply via email to