RFR: JDK-8244093 Move all IDE support into coherent structure in make directory
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Mon May 18 14:50:45 UTC 2020
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 build-dev
mailing list