• Icon: Task Task
    • Resolution: Fixed
    • Icon: Minor Minor
    • msbuild-plugin
    • None

      msbuild:1.28 breaks binary compatibility since a new parameter has been added to the constructor of the class MsBuildBuilder.

      Suggested actions
      1. Adding the previous constructor but marked as deprecated.
      2. Add a new test to check compatibility with previous configurations (configRoundTrip)

          [JENKINS-48194] Binary compatibility broken in MsBuildBuilder

          Code changed in jenkins
          User: Francisco Javier Fernandez Gonzalez
          Path:
          src/main/java/hudson/plugins/msbuild/MsBuildBuilder.java
          src/test/java/hudson/plugins/msbuild/MsBuildBuilderTest.java
          src/test/resources/hudson/plugins/msbuild/MsBuildBuilderTest/newMsBuildBuilderTest.zip
          http://jenkins-ci.org/commit/msbuild-plugin/278ee8c5d9bd46d6b1d651f6d7e048e7eaf66eba
          Log:
          JENKINS-48194 Reverting binary incompatibility

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Francisco Javier Fernandez Gonzalez Path: src/main/java/hudson/plugins/msbuild/MsBuildBuilder.java src/test/java/hudson/plugins/msbuild/MsBuildBuilderTest.java src/test/resources/hudson/plugins/msbuild/MsBuildBuilderTest/newMsBuildBuilderTest.zip http://jenkins-ci.org/commit/msbuild-plugin/278ee8c5d9bd46d6b1d651f6d7e048e7eaf66eba Log: JENKINS-48194 Reverting binary incompatibility

          Code changed in jenkins
          User: Francisco Javier Fernandez Gonzalez
          Path:
          src/test/resources/hudson/plugins/msbuild/MsBuildBuilderTest/configRoundtrip.zip
          src/test/resources/hudson/plugins/msbuild/MsBuildBuilderTest/newMsBuildBuilderTest.zip
          http://jenkins-ci.org/commit/msbuild-plugin/e8e6341e97af1648002c3b1246e6e5c3009bcb27
          Log:
          [JENKINS-48194 Lighter configuration for LocalData

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Francisco Javier Fernandez Gonzalez Path: src/test/resources/hudson/plugins/msbuild/MsBuildBuilderTest/configRoundtrip.zip src/test/resources/hudson/plugins/msbuild/MsBuildBuilderTest/newMsBuildBuilderTest.zip http://jenkins-ci.org/commit/msbuild-plugin/e8e6341e97af1648002c3b1246e6e5c3009bcb27 Log: [ JENKINS-48194 Lighter configuration for LocalData

          Code changed in jenkins
          User: Francisco Javier Fernandez Gonzalez
          Path:
          src/test/java/hudson/plugins/msbuild/MsBuildBuilderTest.java
          http://jenkins-ci.org/commit/msbuild-plugin/f566e248dd59bfcbf3125f7eeda064da16b15c6b
          Log:
          JENKINS-48194 dummy commit

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Francisco Javier Fernandez Gonzalez Path: src/test/java/hudson/plugins/msbuild/MsBuildBuilderTest.java http://jenkins-ci.org/commit/msbuild-plugin/f566e248dd59bfcbf3125f7eeda064da16b15c6b Log: JENKINS-48194 dummy commit

          Code changed in jenkins
          User: Marshall777
          Path:
          src/main/java/hudson/plugins/msbuild/MsBuildBuilder.java
          src/test/java/hudson/plugins/msbuild/MsBuildBuilderTest.java
          src/test/resources/hudson/plugins/msbuild/MsBuildBuilderTest/configRoundtrip.zip
          http://jenkins-ci.org/commit/msbuild-plugin/29e860368519926069225fc25dec0603a6c57e81
          Log:
          Merge pull request #35 from fcojfernandez/JENKINS-48194

          JENKINS-48194 Binary compatibility broken in MsBuildBuilder

          Compare: https://github.com/jenkinsci/msbuild-plugin/compare/a830f386f51b...29e860368519

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Marshall777 Path: src/main/java/hudson/plugins/msbuild/MsBuildBuilder.java src/test/java/hudson/plugins/msbuild/MsBuildBuilderTest.java src/test/resources/hudson/plugins/msbuild/MsBuildBuilderTest/configRoundtrip.zip http://jenkins-ci.org/commit/msbuild-plugin/29e860368519926069225fc25dec0603a6c57e81 Log: Merge pull request #35 from fcojfernandez/ JENKINS-48194 JENKINS-48194 Binary compatibility broken in MsBuildBuilder Compare: https://github.com/jenkinsci/msbuild-plugin/compare/a830f386f51b...29e860368519

          Released in 1.29

          Lionel Cabasson added a comment - Released in 1.29

            fcojfernandez Francisco Fernández
            fcojfernandez Francisco Fernández
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: