Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3911] Filter source roots to avoids clashes between module-info files. #3924

Merged
merged 1 commit into from
Apr 12, 2022

Conversation

jlahoda
Copy link
Contributor

@jlahoda jlahoda commented Apr 4, 2022

The source path currently contains directories for all modules, and the compiler is confused and reads wrong module-info.java. As an attempt to fix this, lets try to filter the source path to contain only the one appropriate source root, and lets see how it works.

@jlahoda jlahoda linked an issue Apr 4, 2022 that may be closed by this pull request
@mbien mbien added the kind:bug Bug report or fix label Apr 5, 2022
@mbien mbien added this to the NB14 milestone Apr 5, 2022
@mbien mbien added the Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) label Apr 5, 2022
@jtulach jtulach changed the title [3911] Filter source roots to avoids clashes between moule-info files. [3911] Filter source roots to avoids clashes between module-info files. Apr 10, 2022
@lkishalmi
Copy link
Contributor

For me it looks good. A small enough fix to be reverted even in the NB14 cycle if anything goes south.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) kind:bug Bug report or fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NPE Parsing javax.swing.DefaultListModel.java
3 participants