RFR: JDK-8244093 Move all IDE support into coherent structure in make directory

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Wed Apr 29 10:57:17 UTC 2020

On 2020-04-29 12:47, Maurizio Cimadamore wrote:
> Didn't review in detail, but some pieces seems suspicious - e.g.
> moving "/make/langtools/build.properties" to 
> "///make/ide/idea/langtools/build.properties//
> Note that langtools's build.properties is used by the langtools ANT 
> build which is independent from make - and also mostly independent 
> from the IDE. It just provides an ANT target to create the necessary 
> IDE files. So, if you want you can move the IDE templates in the new 
> folder, but the build script and properties should remain where they 
> are, I think.
So who/what is using the ant build? I thought it was just the IDEA 
project that generated ant targets. Is it run explicitly using ant by 
langtool developers? I know for sure it's not used when we build the 
product in our CI.


> Maurizio
> On 29/04/2020 11:36, Magnus Ihse Bursie wrote:
>> The IDE support in OpenJDK unfortunately leaves a lot to be desired. 
>> There have been a garden variety of attempt to support a specific IDE 
>> for a specific part of the code base, cluttered all over the code base.
>> This patch is a first attempt go get one ring, eh..., structure, to 
>> rule them all.
>> I have moved all IDE project creators into the following structure:
>> make/ide/<ide>/<part of the code>
>> where <part of the code> is one of currently "hotspot", "langtools" 
>> or "jdk", and <ide> is one of "vscode", "idea", "netbeans" or 
>> "vistualstudio".
>> This will not magically improve IDE support, but will at least make 
>> it clearer what we have and what we are missing.
>> Ownership of the IDE support is notoriously vague. I've cc:ed a bunch 
>> of people who has shown interest and/or submitted fixes to some of 
>> the IDE projects according to the hg history. I'd appreciate it if 
>> anyone who is interested in a particular case for IDE support can 
>> verify that it still works. I've tried my best to make sure all 
>> targets can run without errors, but I cannot verify that the IDE 
>> environment themselves are correct.
>> If you know about an IDE project that is no longer relevant, and 
>> should be removed instead of shuffled around, please let me know!
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8244093
>> WebRev: 
>> http://cr.openjdk.java.net/~ihse/JDK-8244093-move-ide-support/webrev.01
>> /Magnus

More information about the ide-support-dev mailing list