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

failure when parsing the bzr log

    Details

    • Similar Issues:

      Description

      I'm seeing alot of failures in hudson due to a bug in the bzr plugin. Here's output from the build job. I think the problem may be
      related to lines 33 and 38 of hudson.plugins.bazaar.BazaarChangeLogParser (for version 1.6). The variable len is set to the length of the string, but then s = s.trim() is called. Perhaps that is causing the index out of range error??

      $ /usr/bin/bzr revision-info -d /var/lib/hudson/jobs/Production-head/workspace
      cskinfill@movenetworks.com-20100512134220-9n81y022i7ckl2wq
      FATAL: String index out of range: 41
      java.lang.StringIndexOutOfBoundsException: String index out of range: 41
      at java.lang.String.substring(String.java:1934)
      at hudson.plugins.bazaar.BazaarChangeLogParser.parse(BazaarChangeLogParser.java:74)
      at hudson.plugins.bazaar.BazaarChangeLogParser.parse(BazaarChangeLogParser.java:19)
      at hudson.model.AbstractBuild.calcChangeSet(AbstractBuild.java:652)
      at hudson.model.AbstractBuild.access$600(AbstractBuild.java:86)
      at hudson.model.AbstractBuild$AbstractRunner.checkout(AbstractBuild.java:484)
      at hudson.model.AbstractBuild$AbstractRunner.run(AbstractBuild.java:411)
      at hudson.model.Run.run(Run.java:1243)
      at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:46)
      at hudson.model.ResourceController.execute(ResourceController.java:88)
      at hudson.model.Executor.run(Executor.java:122)

        Attachments

          Activity

          Hide
          zigarn Alexandre Garnier added a comment -

          Introduced by my patch from JENKINS-5421.
          Caused by commiter names with trailing spaces and without email between <>.
          Will propose a patch to solve this.

          Show
          zigarn Alexandre Garnier added a comment - Introduced by my patch from JENKINS-5421 . Caused by commiter names with trailing spaces and without email between <>. Will propose a patch to solve this.
          Hide
          zigarn Alexandre Garnier added a comment -

          Correction for this bug.
          Just reevaluate 'len' after the trim.

          Show
          zigarn Alexandre Garnier added a comment - Correction for this bug. Just reevaluate 'len' after the trim.
          Hide
          zigarn Alexandre Garnier added a comment -
          Show
          zigarn Alexandre Garnier added a comment - github pull request : https://github.com/HudsonLabs/plugin-bazaar/pull/2

            People

            • Assignee:
              zigarn Alexandre Garnier
              Reporter:
              cskinfill cskinfill
            • Votes:
              1 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: