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

Erik Joelsson erik.joelsson at oracle.com
Tue May 19 13:27:03 UTC 2020

I like the new structure. Looks good.


On 2020-05-19 04:33, Magnus Ihse Bursie wrote:
> How derpy of me. I forgot to post the link to the updated webrev!
> http://cr.openjdk.java.net/~ihse/JDK-8244093-move-ide-support/webrev.02/
> /Magnus
> On 2020-05-18 16:50, Magnus Ihse Bursie wrote:
>> Hi all,
>> Sorry for the long time to follow up on this patch.
>> From the reviewers' feedback, I have now restored all files to 
>> make/langtools, except for those in the netbeans and intellij 
>> directories. This also meant that in the make/ide/idea/langtools 
>> directory there were now only an "intellij" directory. This seemed 
>> awkward, so I moved everything up one level, effectively scrapping 
>> the "intellij" directory. Paths has been changed accordingly in 
>> make/langtools and make/ide/*/langtools.
>> I also removed make/ide/netbeans/jdk. No-one seemed to be countering 
>> Jan's claim that it's unused. Instead, we got testimony on how 
>> Netbeans can be setup without any specific support from the build 
>> system. I think this excellent information should be properly 
>> documented. I've created JDK-8245210 for this, and I hope I can 
>> enroll the help of Jan and Brad to get this correctly documented.
>> No other changes, outside the langtools files and 
>> make/ide/netbeans/jdk has been made, compared to the previous version.
>> /Magnus
>> On 2020-04-29 13:06, Jan Lahoda wrote:
>>> I am not sure if anyone is still using make/jdk/netbeans. Apache 
>>> NetBeans does not (should not) need these config files, it supports 
>>> OpenJDK modules out of the box (with some tweaks/dependencies on 
>>> make/langtools/** to speed up langtools build).
>>> As Maurizio, there may be some need to move the "fast" langtools 
>>> build more carefully. I'll try to take a look later, unless some 
>>> else wants to. Although, a little independently, I wonder somewhat 
>>> if there's an opportunity to further speed up the ordinary make 
>>> build in incremental environment to reduce the need for a "fast" 
>>> langtools build. E.g. by enhancing the current Depend javac plugin, 
>>> and possibly optionally disabling the interim langtools build. This 
>>> could improve incremental build behavior for other modules (like 
>>> java.base or java.desktop) as well.
>>> Jan
>>> On 29. 04. 20 12: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