[JENKINS-21520] transient NoClassDefFoundError: javax/servlet/ServletException

[ndeloof]

No description provided.

[ndeloof]

Not sure why having a reference to Functions do drag some unexpected classes on slave then breaks ClassLoading, anyway inlining this on-line functions seems a reasonable workaround
@reviewbybees

[ghost]

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

[MarkEWaite]

Doesn't the reference also need to be removed from src/main/java/hudson/plugins/git/GitTool.java?

Shouldn't it also be removed from src/test/java/org/jenkinsci/plugins/gitclient/WarnTempDirValueTest.java to reduce the risk that someone will see its use in a test and introduce it back into the main code?

[ndeloof]

makes sense indeed

[jenkinsadmin]

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

[stephenc]

🐝