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

Accept ecj warnings

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: warnings-plugin
    • Labels:
      None
    • Environment:
      Platform: All, OS: All
    • Similar Issues:

      Description

      Could you please support ecj (the eclipse compiler) warnigns? They are like:

      [javac] Compiling 34 source files to
      D:\Desenvolvimento\hudson\data\jobs\jfg\workspace\build\build\warning
      [javac] ----------
      [javac] 1. WARNING in
      D:\Desenvolvimento\hudson\data\jobs\jfg\workspace\src\jfg\AttributeException.java (at
      line 3)
      [javac] public class AttributeException extends RuntimeException
      [javac] ^^^^^^^^^^^^^^^^^^
      [javac] The serializable class AttributeException does not declare a static
      final serialVersionUID field of type long
      [javac] ----------
      [javac] ----------
      [javac] 2. WARNING in
      D:\Desenvolvimento\hudson\data\jobs\jfg\workspace\src\jfg\gui\swt\SWTTextBuilder.java
      (at line 85)
      [javac] protected void addValidation(Text text, Object type)
      [javac] ^^^^
      [javac] The parameter text is never read
      [javac] ----------
      [javac] 3. WARNING in
      D:\Desenvolvimento\hudson\data\jobs\jfg\workspace\src\jfg\gui\swt\SWTTextBuilder.java
      (at line 85)
      [javac] protected void addValidation(Text text, Object type)
      [javac] ^^^^
      [javac] The parameter type is never read
      [javac] ----------
      [javac] 4. WARNING in
      D:\Desenvolvimento\hudson\data\jobs\jfg\workspace\src\jfg\gui\swt\SWTTextBuilder.java
      (at line 89)
      [javac] protected String convertToString(Text text, Object value, Object type)
      [javac] ^^^^
      [javac] The parameter text is never read
      [javac] ----------
      [javac] 4 problems (4 warnings)

      Thanks

        Attachments

        1. output.txt
          3 kB
        2. output.txt
          0.5 kB

          Activity

          Hide
          drulli Ulli Hafner added a comment -

          Which version of the Eclipse compiler are you using? (javac indicates SUN
          compiler and not the eclipse compiler?). Do you use the eclipse compiler with
          ant (PDE build) or with maven?

          Could you please attach the log messages as a text file, too. I'm not sure if
          the line breaks are conserved in the textfield (and these are important for the
          parser.).

          Show
          drulli Ulli Hafner added a comment - Which version of the Eclipse compiler are you using? (javac indicates SUN compiler and not the eclipse compiler?). Do you use the eclipse compiler with ant (PDE build) or with maven? Could you please attach the log messages as a text file, too. I'm not sure if the line breaks are conserved in the textfield (and these are important for the parser.).
          Hide
          pescuma pescuma added a comment -

          I'm using it with ant. javac can run "any" compiler. To use eclipse compiler
          with it you have to:
          1. Donwload http://download.eclipse.org/eclipse/downloads/drops/R-3.4-
          200806172000/download.php?dropFile=ecj-3.4.jar and put it in ANT_HOME/lib
          2. In ant xml, use the javac line like
          <javac srcdir="${src}" destdir="${build}"
          compiler="org.eclipse.jdt.core.JDTCompilerAdapter" source="1.5" target="1.5">
          <classpath>
          <fileset dir="${libs.src}">
          <include name="*.jar" />
          </fileset>
          </classpath>
          <compilerarg line="-warn:+unused -warn:+hiding" />
          </javac>

          To see a warning, create a class with an unused variable inside some method.

          Anyway, I'll attach an example output

          Show
          pescuma pescuma added a comment - I'm using it with ant. javac can run "any" compiler. To use eclipse compiler with it you have to: 1. Donwload http://download.eclipse.org/eclipse/downloads/drops/R-3.4- 200806172000/download.php?dropFile=ecj-3.4.jar and put it in ANT_HOME/lib 2. In ant xml, use the javac line like <javac srcdir="${src}" destdir="${build}" compiler="org.eclipse.jdt.core.JDTCompilerAdapter" source="1.5" target="1.5"> <classpath> <fileset dir="${libs.src}"> <include name="*.jar" /> </fileset> </classpath> <compilerarg line="-warn:+unused -warn:+hiding" /> </javac> To see a warning, create a class with an unused variable inside some method. Anyway, I'll attach an example output
          Hide
          pescuma pescuma added a comment -

          Created an attachment (id=369)
          ant output

          Show
          pescuma pescuma added a comment - Created an attachment (id=369) ant output
          Hide
          pescuma pescuma added a comment -

          One more thing: this output was created in a windows machine

          Show
          pescuma pescuma added a comment - One more thing: this output was created in a windows machine
          Hide
          pescuma pescuma added a comment -

          Created an attachment (id=370)
          ant output with a lot of warnings

          Show
          pescuma pescuma added a comment - Created an attachment (id=370) ant output with a lot of warnings
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : drulli
          Path:
          trunk/hudson/plugins/warnings/src/main/java/hudson/plugins/warnings/parser/AntEclipseParser.java
          trunk/hudson/plugins/warnings/src/main/java/hudson/plugins/warnings/parser/HpiCompileParser.java
          trunk/hudson/plugins/warnings/src/main/java/hudson/plugins/warnings/parser/MavenParser.java
          trunk/hudson/plugins/warnings/src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java
          trunk/hudson/plugins/warnings/src/main/java/hudson/plugins/warnings/parser/RegexpParser.java
          trunk/hudson/plugins/warnings/src/test/java/hudson/plugins/warnings/parser/AntEclipseParserTest.java
          trunk/hudson/plugins/warnings/src/test/java/hudson/plugins/warnings/parser/AntJavacParserTest.java
          trunk/hudson/plugins/warnings/src/test/java/hudson/plugins/warnings/parser/ParserRegistryTest.java
          trunk/hudson/plugins/warnings/src/test/resources/hudson/plugins/warnings/parser/all.txt
          trunk/hudson/plugins/warnings/src/test/resources/hudson/plugins/warnings/parser/eclipse.txt
          http://fisheye4.cenqua.com/changelog/hudson/?cs=12112
          Log:
          [FIXED JENKINS-2308] Added parser for Eclipse compiler warnings.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : drulli Path: trunk/hudson/plugins/warnings/src/main/java/hudson/plugins/warnings/parser/AntEclipseParser.java trunk/hudson/plugins/warnings/src/main/java/hudson/plugins/warnings/parser/HpiCompileParser.java trunk/hudson/plugins/warnings/src/main/java/hudson/plugins/warnings/parser/MavenParser.java trunk/hudson/plugins/warnings/src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java trunk/hudson/plugins/warnings/src/main/java/hudson/plugins/warnings/parser/RegexpParser.java trunk/hudson/plugins/warnings/src/test/java/hudson/plugins/warnings/parser/AntEclipseParserTest.java trunk/hudson/plugins/warnings/src/test/java/hudson/plugins/warnings/parser/AntJavacParserTest.java trunk/hudson/plugins/warnings/src/test/java/hudson/plugins/warnings/parser/ParserRegistryTest.java trunk/hudson/plugins/warnings/src/test/resources/hudson/plugins/warnings/parser/all.txt trunk/hudson/plugins/warnings/src/test/resources/hudson/plugins/warnings/parser/eclipse.txt http://fisheye4.cenqua.com/changelog/hudson/?cs=12112 Log: [FIXED JENKINS-2308] Added parser for Eclipse compiler warnings.
          Hide
          pescuma pescuma added a comment -

          Thanks, it works

          Show
          pescuma pescuma added a comment - Thanks, it works

            People

            Assignee:
            drulli Ulli Hafner
            Reporter:
            pescuma pescuma
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: