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

Fix java.lang.StackOverflowError #93

Closed
wants to merge 1 commit into from
Closed

Fix java.lang.StackOverflowError #93

wants to merge 1 commit into from

Conversation

dee-kryvenko
Copy link

After rebuild (https://wiki.jenkins-ci.org/display/JENKINS/Rebuild+Plugin) in the upstream list may be the same project, so it's causes infinite recursion

After rebuild (https://wiki.jenkins-ci.org/display/JENKINS/Rebuild+Plugin) in the upstream list may be the same project, so it's causes infinite recursion
@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.73% when pulling 7647e8d on llibicpep:patch-1 into fc6c99e on Diabol:master.

@buildhive
Copy link

Diabol » delivery-pipeline-plugin #417 SUCCESS
This pull request looks good
(what's this?)

@patbos
Copy link
Contributor

patbos commented Feb 1, 2015

Thanks for the pull request. Looks good. Please create an issue (https://issues.jenkins-ci.org/) on component delivery-pipeline-plugin. Please add a description on how to reproduce the issue if possible.

@dee-kryvenko
Copy link
Author

@patbos
Copy link
Contributor

patbos commented Mar 15, 2015

Sorry for the late feedback. Can you please write a unittest for this?

@tommysdk
Copy link
Contributor

What's the status of this issue? @llibicpep are you able to provide a reproducing unit test? Also please make sure to provide the JIRA ticket id in the commit message.

@dee-kryvenko
Copy link
Author

Sorry for a delay.
I was trying to reproduce this one again with no luck. It seems like a 3rd party plugin caused this behaviour originally and probably it has been fixed on their side.
Closing this issue for now, will reopen in case of luck with reproducing.

@tommysdk
Copy link
Contributor

Thank you for the update!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants