Integrated: 8284588: Remove GensrcCommonLangtools.gmk

[Magnus Ihse Bursie]

On Fri, 8 Apr 2022 14:57:02 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

> There's been unnecessary (near) duplication of functionality, and messy logic, in the langtools gensrc step, for a long time. (Basically since we got rid of the mercurial forest...)
> 
> This is a first attempt at cleaning this up. The file GensrcCommonLangtools.gmk is removed. This contained three pieces of functionality. 
> 
> * SetupVersionProperties was moved to GensrcCommonJdk.gmk (which was renamed to GensrcCommon.gmk). 
> * PropertiesParser was only used in one place, and moved there. (jdk.compiler/Gensrc.gmk). It was also heavily rewritten to show more clearly what it actually is doing.
> * SetupCompileProperties (from langtools) were replaced with calls to SetupCompileProperties (from GensrcCommon/"jdk"), which was modified slightly to accomodate for the differences. (Also the Java tool needed to be slightly modified to accept the same syntax as the JDK version of this tool.)
> 
> A reasonable follow-up is to research if the two SetupCompileProperties tools can be unified into one.

This pull request has now been integrated.

Changeset: 3357d9a1
Author:    Magnus Ihse Bursie <ihse at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/3357d9a168f031e1de4fa0d203f16a6f060fd062
Stats:     367 lines in 21 files changed: 124 ins; 185 del; 58 mod

8284588: Remove GensrcCommonLangtools.gmk

Reviewed-by: erikj

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

PR: https://git.openjdk.java.net/jdk/pull/8167