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
Centralize internal maven artifact version queries. #6751
Conversation
@@ -17,7 +17,6 @@ | |||
|
|||
javac.source=1.8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
modules which use nb-javac can't change their source level at the moment since they prepend the bootclasspath and this is only allowed on java 8.
Since target isn't (and can't) be set either once boot cp is modified, no API checks are active. Going to file an issue with more details.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
filled as issue so that we don't forget #6817
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks sane to me, but I think the API break is unnecessary (yeah, I know, that it is friend-only). IMHO the class rename is not necessary and the fields could be retained.
java/maven/src/org/netbeans/modules/maven/options/NbArtifactVersionPreferences.java
Outdated
Show resolved
Hide resolved
java/maven/src/org/netbeans/modules/maven/options/MavenVersionSettings.java
Show resolved
Hide resolved
java/maven/src/org/netbeans/modules/maven/options/NbArtifactVersionPreferences.java
Outdated
Show resolved
Hide resolved
- put all hardcoded fallback versions in one place - use index for updates if available
cc86fe1
to
92080ad
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks sane to me. Thank you.
also fixes https://issues.apache.org/jira/browse/NETBEANS-6491 as side effect
note: its all internal, and only used in two modules for now