Integrated in plugins_git-plugin #146
[FIXED JENKINS-9843] PreBuildMergeOptions can now export for the API properly.

Andrew Bayer : b3ddce61d33dfd4d2ac95dcdfbde5ba63c983f23
Files :

• src/main/java/hudson/plugins/git/opt/PreBuildMergeOptions.java

Kudos for the quick fix!

I just wanted to mention that the git.hpi from http://ci.jenkins-ci.org/job/plugins_git-plugin/146/ behaves strangely in our environment:

hudson.util.IOException2: Unable to read /var/hudson/jobs/git-somejob/builds/2011-06-02_08-52-17/build.xml
at hudson.XmlFile.unmarshal(XmlFile.java:152)
at hudson.model.Run.reload(Run.java:281)
at hudson.model.Run.<init>(Run.java:270)
at hudson.model.AbstractBuild.<init>(AbstractBuild.java:158)
at hudson.model.Build.<init>(Build.java:100)
at hudson.model.FreeStyleBuild.<init>(FreeStyleBuild.java:41)
at sun.reflect.GeneratedConstructorAccessor19.newInstance(Unknown Source)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:27)
at java.lang.reflect.Constructor.newInstance(Constructor.java:513)
at hudson.model.AbstractProject.loadBuild(AbstractProject.java:935)
at hudson.model.AbstractProject$1.create(AbstractProject.java:253)
at hudson.model.AbstractProject$1.create(AbstractProject.java:251)
at hudson.model.RunMap.load(RunMap.java:221)
at hudson.model.AbstractProject.onLoad(AbstractProject.java:251)
at hudson.model.Project.onLoad(Project.java:88)
at hudson.model.Items.load(Items.java:114)
at hudson.model.Hudson$13.run(Hudson.java:2354)
...

Caused by: com.thoughtworks.xstream.converters.ConversionException: Cannot deserialize object with new readObject()/writeObject() methods
---- Debugging information ----
class : hudson.model.FreeStyleBuild
required-type : org.eclipse.jgit.lib.ObjectId
path : /build/actions/hudson.plugins.git.util.BuildData/buildsByBranchName/entry/hudson.plugins.git.util.Build/revision/sha1
line number : 19

This might not be related to your fix, but I'm throwing it in just in case. Thanks.

I'm guessing this is due to the JGit upgrade - I'll see if someone can look into it.

I've just committed 0007bab4d3ca27f20d7d69ca870e8d57b4e3ebfa and b1db233fa2b32f01c46b5a561fa3bbc16f0ae68b to help with this.

Also see https://wiki.jenkins-ci.org/display/JENKINS/Hint+on+retaining+backward+compatibility

Created pull request with potential fix: https://github.com/jenkinsci/git-plugin/pull/33

The conversion exception with the ObjectId class should now be fixed by this commit

Another pull request related to this issue (fixed NPE): https://github.com/jenkinsci/git-plugin/pull/35

I haven't tried with vjuranek's patch, but prior to that, working around the NPE by filling in mergeRemote/mergeTarget (which incidentally are never retained in the UI), I then get the attached stack trace, so looks like there are more depths to plumb here?

[Winstone 2011/06/21 16:06:09] - Untrapped Error in Servlet
javax.servlet.ServletException: org.kohsuke.stapler.export.NotExportableException: class org.eclipse.jgit.transport.RemoteConfig doesn't have @ExportedBean
at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:603)