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
Detect proxy issues and offer fixes. #4043
Conversation
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.
I have not noticed a problem.
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.
Generally it looks good. I made a few comments though.
extide/gradle/src/org/netbeans/modules/gradle/problems/ChangeOrRemovePropertyResolver.java
Outdated
Show resolved
Hide resolved
extide/gradle/src/org/netbeans/modules/gradle/problems/PropertiesEditor.java
Show resolved
Hide resolved
extide/gradle/src/org/netbeans/modules/gradle/problems/PropertiesEditor.java
Show resolved
Hide resolved
* @return files accessor. | ||
* @since 2.24 | ||
*/ | ||
public GradleFiles getFiles() { |
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.
Could the naming of the method be consistent and use getGradleFiles instead of getFiles?
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.
getFiles() -> getGradleFiles()
Addressed in e1b89f8 |
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.
Thank you!
Proxy settings is one of the very unpleasant parts of Gradle: normal tools tend to honour environment variables, but Gradle knows better: only respects its own settings, either in the project, or in the
~/.gradle/gradle.properties
.This PR processes
GradleReport
s produced after project (re)load. It detects some (not all!) typical situation - the number of matches may expand:IDE settings are used to 'autodetect' the valid proxy settings, I used a well-known root DNS
8.8.8.8
that everyone should have route to.Gradle project offer fix that will alter the .properties file (project or user) that defines the proxies. If a proxy is not set up and the fix introduces one, it is always written into user properties file (~/.gradle/gradle.properties).