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

Ability to provide merge comments to svn command

    Details

    • Similar Issues:

      Description

      Currently, the Subversion Merge plug-in lacks the ability to provide a commit comment. For my particular setup, Subversion has pre-commit hooks to JIRA to check whether a valid issue from the user is being used. When I attempt to create a branch using the plug-in, the job fails with a similar error message as follows (removed confidential info):

      svn: Commit failed (details follow): svn: Commit blocked by pre-commit hook (exit code 4) with output: JIRA issue Created was not found on <JIRA-server>, commit aborted. JIRA issues are case sensitive and must be entered in UPPER case. Valid issues for <user> include;
      <issue-list>

      In this case, each of the feature that makes a svn merge call should take a commit message option. In addition, I also question how the plug-in detects the user performing the merge command. Should a user name and password be supplied to svn commands that write? Perhaps this is also a requirement for a secured Subversion scenario.

      These might not be bugs, but they would be critical features for some folks to take advantage of this plug-in. Please consider providing them and let me know if you have any questions about my scenarios. Thanks a lot!

        Attachments

          Activity

          Hide
          wo_hauser Wolfgang Hauser added a comment - - edited

          Like in JENKINS-11827 I would propose following enhancements:

          • A field for an optional commit message in UI for creating a branch. Or a dialog like in "SCM Sync Configuration Plugin" for creating a branch.
          • A field for an optional commit message in configuration UI of each project for automatic merges.
          Show
          wo_hauser Wolfgang Hauser added a comment - - edited Like in JENKINS-11827 I would propose following enhancements: A field for an optional commit message in UI for creating a branch. Or a dialog like in "SCM Sync Configuration Plugin" for creating a branch. A field for an optional commit message in configuration UI of each project for automatic merges.
          Hide
          clifford_jenkins Clifford Sanders added a comment - - edited

          I implemented this in my fork and created a pull request

          Show
          clifford_jenkins Clifford Sanders added a comment - - edited I implemented this in my fork and created a pull request
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Hugues Chabot
          Path:
          src/main/java/jenkins/plugins/svnmerge/IntegratableProjectAction.java
          src/main/resources/jenkins/plugins/svnmerge/IntegratableProjectAction/index.groovy
          http://jenkins-ci.org/commit/svnmerge-plugin/f2b6945e64a526bf45797499ab7b996b327de650
          Log:
          Merge pull request #6 from clifford-github/master

          [FIXED JENKINS-11012] Added possibility to enter a commit message

          Compare: https://github.com/jenkinsci/svnmerge-plugin/compare/159b0b18db73...f2b6945e64a5

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Hugues Chabot Path: src/main/java/jenkins/plugins/svnmerge/IntegratableProjectAction.java src/main/resources/jenkins/plugins/svnmerge/IntegratableProjectAction/index.groovy http://jenkins-ci.org/commit/svnmerge-plugin/f2b6945e64a526bf45797499ab7b996b327de650 Log: Merge pull request #6 from clifford-github/master [FIXED JENKINS-11012] Added possibility to enter a commit message Compare: https://github.com/jenkinsci/svnmerge-plugin/compare/159b0b18db73...f2b6945e64a5

            People

            • Assignee:
              hugueschabot Hugues Chabot
              Reporter:
              acwwat Anthony Wat
            • Votes:
              3 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: