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