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

Double quotes in commit message results in merge failure (Praqma case 12871)

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Merge fails when commit message has "" double quotes.

      Work around:

      Modify commits messages and push again.

        Attachments

          Activity

          Hide
          bue Bue Petersen added a comment -

          This problem could also indirectly be reported as JENKINS-27309

          Show
          bue Bue Petersen added a comment - This problem could also indirectly be reported as JENKINS-27309
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Bue Petersen
          Path:
          src/main/java/org/jenkinsci/plugins/pretestedintegration/scm/git/AccumulatedCommitStrategy.java
          src/main/java/org/jenkinsci/plugins/pretestedintegration/scm/git/SquashCommitStrategy.java
          http://jenkins-ci.org/commit/pretested-integration-plugin/bba993008f51195ebf913d760d405d7b28aa5ad6
          Log:
          JENKINS-27690,JENKINS-27662,JENKINS-27697: checkstyle fixes

          Fixed some checkstyle warnings to pass thressholds.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Bue Petersen Path: src/main/java/org/jenkinsci/plugins/pretestedintegration/scm/git/AccumulatedCommitStrategy.java src/main/java/org/jenkinsci/plugins/pretestedintegration/scm/git/SquashCommitStrategy.java http://jenkins-ci.org/commit/pretested-integration-plugin/bba993008f51195ebf913d760d405d7b28aa5ad6 Log: JENKINS-27690 , JENKINS-27662 , JENKINS-27697 : checkstyle fixes Fixed some checkstyle warnings to pass thressholds.
          Hide
          bue Bue Petersen added a comment -

          We have written some tests that shows that this should be a problem, and that the git merge may have failed because of some other merge problem.

          That merge problem isn't really reported in a good way, and the error reporting from the plugin hides the real cause of merge problems in some cases. Thus we have created JENKINS-27690 to improve logging.

          This issues will then be resolved, by now including tests that integrate commits with double quotes in one of the commit messages.

          Show
          bue Bue Petersen added a comment - We have written some tests that shows that this should be a problem, and that the git merge may have failed because of some other merge problem. That merge problem isn't really reported in a good way, and the error reporting from the plugin hides the real cause of merge problems in some cases. Thus we have created JENKINS-27690 to improve logging. This issues will then be resolved, by now including tests that integrate commits with double quotes in one of the commit messages.

            People

            • Assignee:
              bue Bue Petersen
              Reporter:
              bue Bue Petersen
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: