RFR: JDK-8251193 bin/idea.sh generating wrong source folders for JVMCI modules

Severin Gehwolf sgehwolf at redhat.com
Mon Sep 7 16:00:13 UTC 2020


Hi,

On Thu, 2020-09-03 at 16:48 +0100, Maurizio Cimadamore wrote:
> Sorry for the delay,
> overall this looks like a good improvement, so I'll approve.
> 
> Is there some colleague that can help you do the push?
> 
> Let me know if you need help.

I'll sponsor the patch for Galder. Yet, sponsoring with git works
differently now.

Maurizio: I guess approving this PR would be sufficient?
https://github.com/openjdk/jdk/pull/63

Thanks,
Severin



> Thanks
> Maurizio
> 
> On 06/08/2020 15:28, Galder Zamarreno wrote:
> > Hi,
> > 
> > `bin/idea.sh` is not generating the right source folders for
> > jdk.internal.vm.ci and jdk.internal.vm.compiler modules.
> > 
> > These modules have different directory structures to the rest and so should
> > be handled exceptionally so that the IDE has the right source folder
> > definitions.
> > 
> > The fix takes these two modules, inspects all the subdirectories within
> > each module root and generates a source folder definition for each. As
> > example, for jdk.internal.vm.ci, there should be a source folder for:
> > 
> > * src/jdk.internal.vm.ci/share/classes/jdk.vm.ci.aarch64/src
> > * src/jdk.internal.vm.ci/share/classes/jdk.vm.ci.amd64/src
> > * ...
> > 
> > I've tested this in the patch on IDEA 2020.2.
> > 
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8251193
> > WebRev:
> > http://cr.openjdk.java.net/~sgehwolf/webrevs/galder/JDK-8251193/01/webrev/
> > 
> > Galder



More information about the ide-support-dev mailing list