Skip to content
Commit 70fb68ee authored by David M. Carr's avatar David M. Carr
Browse files

Don't call SCM#calcPollingingBaseline if SCM#checkout fails

In the JavaDocs for SCM#calcRevisionsFromBuild, if says:
This method is called after source code is checked out for the given build (that is, after SCM#checkout has finished successfully.)

This appears to make sense; after you have a known SCM state, you calculate the revisions.  It doesn't explicitly state it, but it
implies that it won't be called if checkout failed.  In those cases, the state may well not be known, and it doesn't make sense to
attempt to calculate the revisions.

The behavior in AbstractProject#checkout has been to always call calcPollingBaseline, regardless of whether checkout failed or
succeeded.  This is causing misleading error output for the mercurial plugin.  With this change, AbstractProject will only do the
calculations when checkout succeeds.
parent e645c176
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment