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

New jre6 requirement : String.isEmpty

    XMLWordPrintable

Details

    Description

      Hi,
      A selenium build was automatically launched and failed with the following exception :
      FATAL: java.lang.String.isEmpty()Z
      java.lang.NoSuchMethodError: java.lang.String.isEmpty()Z
      at hudson.plugins.seleniumhq.SeleniumhqBuilder.getOthers(SeleniumhqBuilder.java:290)
      at hudson.plugins.seleniumhq.SeleniumhqBuilder.perform(SeleniumhqBuilder.java:239)
      at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:19)
      at hudson.model.AbstractBuild$AbstractRunner.perform(AbstractBuild.java:601)
      at hudson.model.Build$RunnerImpl.build(Build.java:174)
      at hudson.model.Build$RunnerImpl.doRun(Build.java:138)
      at hudson.model.AbstractBuild$AbstractRunner.run(AbstractBuild.java:416)
      at hudson.model.Run.run(Run.java:1248)
      at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:46)
      at hudson.model.ResourceController.execute(ResourceController.java:88)
      at hudson.model.Executor.run(Executor.java:129)

      The method String.isEmpty does not exist in java 5, the build fails.
      The workaround is to use length() == 0 instead of isEmpty()

      I'll checkout the lastest code asap and post a patch

      Cheers

      Attachments

        Issue Links

          is related to

          Patch - JENKINS-6996 In some browsers selenium crashes when the "other" field is left empty

          • Major - Major loss of function.
          • Closed

          Activity

            Descending order - Click to sort in ascending order
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: evernat
            Path:
            src/main/java/hudson/plugins/seleniumhq/SeleniumhqBuilder.java
            http://jenkins-ci.org/commit/seleniumhq-plugin/e998f16c35c0650d1a0e56dbe404aac1a485fd9c
            Log:
            fix https://issues.jenkins-ci.org/browse/JENKINS-7246 caused by patch in https://issues.jenkins-ci.org/browse/JENKINS-6996
            (do not use "this.getOther().isEmpty()", to be compatible with jdk 1.5)

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: evernat Path: src/main/java/hudson/plugins/seleniumhq/SeleniumhqBuilder.java http://jenkins-ci.org/commit/seleniumhq-plugin/e998f16c35c0650d1a0e56dbe404aac1a485fd9c Log: fix https://issues.jenkins-ci.org/browse/JENKINS-7246 caused by patch in https://issues.jenkins-ci.org/browse/JENKINS-6996 (do not use "this.getOther().isEmpty()", to be compatible with jdk 1.5)
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: evernat
            Path:
            src/main/java/hudson/plugins/seleniumhq/SeleniumhqBuilder.java
            http://jenkins-ci.org/commit/seleniumhq-plugin/14026298112906d9bb670aa32872bd805fcf8dfa
            Log:
            fix https://issues.jenkins-ci.org/browse/JENKINS-7246 caused by patch in https://issues.jenkins-ci.org/browse/JENKINS-6996
            (do not use "this.getOther().isEmpty()", to be compatible with jdk 1.5)

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: evernat Path: src/main/java/hudson/plugins/seleniumhq/SeleniumhqBuilder.java http://jenkins-ci.org/commit/seleniumhq-plugin/14026298112906d9bb670aa32872bd805fcf8dfa Log: fix https://issues.jenkins-ci.org/browse/JENKINS-7246 caused by patch in https://issues.jenkins-ci.org/browse/JENKINS-6996 (do not use "this.getOther().isEmpty()", to be compatible with jdk 1.5)
            evernat evernat added a comment - - edited

            fix is committed in SeleniumhqBuilder.java, revision 40360 (and 40361)

            the plugin still needs to be released

            evernat evernat added a comment - - edited fix is committed in SeleniumhqBuilder.java, revision 40360 (and 40361) the plugin still needs to be released
            evernat evernat added a comment -

            I confirm the issue in current source code and the proposed workaround.

            evernat evernat added a comment - I confirm the issue in current source code and the proposed workaround.

            People

              pascal_martin pascal_martin
              loic_lacombe loic_lacombe
              Votes:
              2 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: