Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-26671

Multiple description setter build steps should append to build description

      I had expected that multiple description setter build steps yield multiple lines in the description, i.e. each steps appends to the build description.

      However, it appears that instead the last description setter build step wins.

          [JENKINS-26671] Multiple description setter build steps should append to build description

          Code changed in jenkins
          User: michael
          Path:
          .gitignore
          src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterHelper.java
          src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisher.java
          http://jenkins-ci.org/commit/description-setter-plugin/3b98efa98268f225bfb33fa2cf8d75df23dce75b
          Log:
          JENKINS-26671 Multiple description setter build steps should append
          ...

          This commit fixes JENKINS-26671: Multiple description setter build
          steps
          should append to build description]
          At the same time it also fixes JENKINS-14198: Ability to append a
          description to existing description for use with flexible publish

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: michael Path: .gitignore src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterHelper.java src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisher.java http://jenkins-ci.org/commit/description-setter-plugin/3b98efa98268f225bfb33fa2cf8d75df23dce75b Log: JENKINS-26671 Multiple description setter build steps should append ... This commit fixes JENKINS-26671 : Multiple description setter build steps should append to build description] At the same time it also fixes JENKINS-14198 : Ability to append a description to existing description for use with flexible publish

          Code changed in jenkins
          User: michael1010
          Path:
          .gitignore
          src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterHelper.java
          src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisher.java
          src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterBuilder/help-description.html
          src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterBuilder/help.html
          src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterPublisher/help-description.html
          src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterPublisher/help-regexpForFailed.html
          src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterPublisher/help.html
          http://jenkins-ci.org/commit/description-setter-plugin/284925c209ae47a429f821b777ccbc023d652ac1
          Log:
          Merge pull request #6 from michael1010/master

          JENKINS-26671 Multiple description setter build steps should append
          JENKINS-16377 help text for "description setter" seems to be incorrect
          JENKINS-17461 Expose description as an environment variable

          Compare: https://github.com/jenkinsci/description-setter-plugin/compare/965d2e9d8512...284925c209ae

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: michael1010 Path: .gitignore src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterHelper.java src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisher.java src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterBuilder/help-description.html src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterBuilder/help.html src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterPublisher/help-description.html src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterPublisher/help-regexpForFailed.html src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterPublisher/help.html http://jenkins-ci.org/commit/description-setter-plugin/284925c209ae47a429f821b777ccbc023d652ac1 Log: Merge pull request #6 from michael1010/master JENKINS-26671 Multiple description setter build steps should append JENKINS-16377 help text for "description setter" seems to be incorrect JENKINS-17461 Expose description as an environment variable Compare: https://github.com/jenkinsci/description-setter-plugin/compare/965d2e9d8512...284925c209ae

          Michael R added a comment -

          fixed in release 1.10

          Michael R added a comment - fixed in release 1.10

          This change can cause issues with matrix builds. The description set in the post-build step for each configuration will be concatenated for the matrix parent job. This could be desirable in some cases, but in cases where the same description is set in all configurations (e.g. version number), this gives ugly results.

          Samuel Delisle added a comment - This change can cause issues with matrix builds. The description set in the post-build step for each configuration will be concatenated for the matrix parent job. This could be desirable in some cases, but in cases where the same description is set in all configurations (e.g. version number), this gives ugly results.

          Ben Gottesman added a comment -

          I disagree with this request. I think if you want to append to the build description, you should have to do so explicitly by referencing the existing build description via the DESCRIPTION_SETTER_DESCRIPTION environment variable in your new description. That way, you would have both options. As it is, there is no option to replace the existing description.

          Ben Gottesman added a comment - I disagree with this request. I think if you want to append to the build description, you should have to do so explicitly by referencing the existing build description via the DESCRIPTION_SETTER_DESCRIPTION environment variable in your new description. That way, you would have both options. As it is, there is no option to replace the existing description.

            michael1010 Michael R
            harryw Harry Weppner
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved: