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

Set files to parse per parser

    XMLWordPrintable

Details

    Description

      Please allow to specify the files and console log to parse in the warnings plugin for each parser. Otherwise you easily end up with false positives. E.g. the doxygen parser picks up some GCC messages. I already generate a doxygen warnings log. So I could easily avoid the false positives if I could stick the doxygen parser only to this file but let the GCC parser operate on the console log.

      Attachments

        Activity

          drulli Ulli Hafner added a comment - - edited

          Can you please attach a log that produces duplicates? It will be easier to improve the regular expressions.

          drulli Ulli Hafner added a comment - - edited Can you please attach a log that produces duplicates? It will be easier to improve the regular expressions.

          I tried to create a log, but when I also enable the doxygen parser for the command line output, which was the original problem, I now get a StackOverflowError:

          [WARNINGS] Parsing warnings in console log...
          FATAL: null
          java.lang.StackOverflowError
          at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3383)
          at java.util.regex.Pattern$GroupHead.match(Pattern.java:4185)
          at java.util.regex.Pattern$Loop.match(Pattern.java:4312)
          at java.util.regex.Pattern$GroupTail.match(Pattern.java:4244)
          at java.util.regex.Pattern$Curly.match0(Pattern.java:3799)
          at java.util.regex.Pattern$Curly.match(Pattern.java:3761)
          at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362)
          at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362)
          at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362)
          at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3383)
          at java.util.regex.Pattern$GroupHead.match(Pattern.java:4185)
          at java.util.regex.Pattern$Loop.match(Pattern.java:4312)
          at java.util.regex.Pattern$GroupTail.match(Pattern.java:4244)
          at java.util.regex.Pattern$Curly.match0(Pattern.java:3799)
          at java.util.regex.Pattern$Curly.match(Pattern.java:3761)
          at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362)
          at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362)
          at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362)
          at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3383)
          at java.util.regex.Pattern$GroupHead.match(Pattern.java:4185)
          at java.util.regex.Pattern$Loop.match(Pattern.java:4312)

          languitar Johannes Wienke added a comment - I tried to create a log, but when I also enable the doxygen parser for the command line output, which was the original problem, I now get a StackOverflowError: [WARNINGS] Parsing warnings in console log... FATAL: null java.lang.StackOverflowError at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3383) at java.util.regex.Pattern$GroupHead.match(Pattern.java:4185) at java.util.regex.Pattern$Loop.match(Pattern.java:4312) at java.util.regex.Pattern$GroupTail.match(Pattern.java:4244) at java.util.regex.Pattern$Curly.match0(Pattern.java:3799) at java.util.regex.Pattern$Curly.match(Pattern.java:3761) at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362) at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362) at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362) at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3383) at java.util.regex.Pattern$GroupHead.match(Pattern.java:4185) at java.util.regex.Pattern$Loop.match(Pattern.java:4312) at java.util.regex.Pattern$GroupTail.match(Pattern.java:4244) at java.util.regex.Pattern$Curly.match0(Pattern.java:3799) at java.util.regex.Pattern$Curly.match(Pattern.java:3761) at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362) at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362) at java.util.regex.Pattern$CharProperty.match(Pattern.java:3362) at java.util.regex.Pattern$BmpCharProperty.match(Pattern.java:3383) at java.util.regex.Pattern$GroupHead.match(Pattern.java:4185) at java.util.regex.Pattern$Loop.match(Pattern.java:4312)
          bmatos Bruno Matos added a comment - - edited

          I have the same problem, and the warning output format of both GCC and Doxgyen seems to be identical, I think that regular expressions will not solve this problem.

          Output from GCC:

          [absolute PATH]/mytableview.cpp:252:9: warning: unused variable ‘oldMP’

          Output from Doxygen:

          [absolute PATH]/datasubscriber.h:5: warning: Compound DataSubscriber is not documented.

          The only 2 differences I can see:

          • GCC puts the column number, what gives 3 ':' after filename;
          • Doxygen puts a dot at the end of the line.

          But I don't know if GCC always puts the column (and the separator) and if there are any GCC warnings with a dot at the end or vice-versa.

          bmatos Bruno Matos added a comment - - edited I have the same problem, and the warning output format of both GCC and Doxgyen seems to be identical, I think that regular expressions will not solve this problem. Output from GCC: [absolute PATH]/mytableview.cpp:252:9: warning: unused variable ‘oldMP’ Output from Doxygen: [absolute PATH]/datasubscriber.h:5: warning: Compound DataSubscriber is not documented. The only 2 differences I can see: GCC puts the column number, what gives 3 ':' after filename; Doxygen puts a dot at the end of the line. But I don't know if GCC always puts the column (and the separator) and if there are any GCC warnings with a dot at the end or vice-versa.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/analysis/core/ParserResult.java
          http://jenkins-ci.org/commit/analysis-core-plugin/b665e52c8d0385a891aec642041004ff2695d874
          Log:
          JENKINS-8833: Added convenience method to add a project.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/analysis/core/ParserResult.java http://jenkins-ci.org/commit/analysis-core-plugin/b665e52c8d0385a891aec642041004ff2695d874 Log: JENKINS-8833 : Added convenience method to add a project.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/analysis/core/PluginDescriptor.java
          http://jenkins-ci.org/commit/analysis-core-plugin/771e176893563f7575508fd63ded3144ffa43ffe
          Log:
          JENKINS-8833: Removed final from validation methods to allow extension.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/analysis/core/PluginDescriptor.java http://jenkins-ci.org/commit/analysis-core-plugin/771e176893563f7575508fd63ded3144ffa43ffe Log: JENKINS-8833 : Removed final from validation methods to allow extension.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/analysis/core/ParserResult.java
          http://jenkins-ci.org/commit/analysis-core-plugin/b665e52c8d0385a891aec642041004ff2695d874
          Log:
          JENKINS-8833: Added convenience method to add a project.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/analysis/core/ParserResult.java http://jenkins-ci.org/commit/analysis-core-plugin/b665e52c8d0385a891aec642041004ff2695d874 Log: JENKINS-8833 : Added convenience method to add a project.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/analysis/core/PluginDescriptor.java
          http://jenkins-ci.org/commit/analysis-core-plugin/771e176893563f7575508fd63ded3144ffa43ffe
          Log:
          JENKINS-8833: Removed final from validation methods to allow extension.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/analysis/core/PluginDescriptor.java http://jenkins-ci.org/commit/analysis-core-plugin/771e176893563f7575508fd63ded3144ffa43ffe Log: JENKINS-8833 : Removed final from validation methods to allow extension.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          go.sh
          pom.xml
          src/main/java/hudson/plugins/warnings/ParserConfiguration.java
          src/main/java/hudson/plugins/warnings/WarningsDescriptor.java
          src/main/java/hudson/plugins/warnings/WarningsPublisher.java
          src/main/java/hudson/plugins/warnings/parser/FileWarningsParser.java
          src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java
          src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.jelly
          src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.properties
          src/main/resources/hudson/plugins/warnings/WarningsPublisher/config_de.properties
          http://jenkins-ci.org/commit/warnings-plugin/585be4161adaa8fcd9e4e491df89dad3c23f90b8
          Log:
          [FIXED JENKINS-8833] Allow configuration of parsers for each file.

          Compare: https://github.com/jenkinsci/warnings-plugin/compare/18271af...585be41

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: go.sh pom.xml src/main/java/hudson/plugins/warnings/ParserConfiguration.java src/main/java/hudson/plugins/warnings/WarningsDescriptor.java src/main/java/hudson/plugins/warnings/WarningsPublisher.java src/main/java/hudson/plugins/warnings/parser/FileWarningsParser.java src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.jelly src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.properties src/main/resources/hudson/plugins/warnings/WarningsPublisher/config_de.properties http://jenkins-ci.org/commit/warnings-plugin/585be4161adaa8fcd9e4e491df89dad3c23f90b8 Log: [FIXED JENKINS-8833] Allow configuration of parsers for each file. Compare: https://github.com/jenkinsci/warnings-plugin/compare/18271af...585be41

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          go.sh
          pom.xml
          src/main/java/hudson/plugins/warnings/ParserConfiguration.java
          src/main/java/hudson/plugins/warnings/WarningsDescriptor.java
          src/main/java/hudson/plugins/warnings/WarningsPublisher.java
          src/main/java/hudson/plugins/warnings/parser/FileWarningsParser.java
          src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java
          src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.jelly
          src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.properties
          src/main/resources/hudson/plugins/warnings/WarningsPublisher/config_de.properties
          http://jenkins-ci.org/commit/warnings-plugin/585be4161adaa8fcd9e4e491df89dad3c23f90b8
          Log:
          [FIXED JENKINS-8833] Allow configuration of parsers for each file.

          Compare: https://github.com/jenkinsci/warnings-plugin/compare/18271af...585be41

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: go.sh pom.xml src/main/java/hudson/plugins/warnings/ParserConfiguration.java src/main/java/hudson/plugins/warnings/WarningsDescriptor.java src/main/java/hudson/plugins/warnings/WarningsPublisher.java src/main/java/hudson/plugins/warnings/parser/FileWarningsParser.java src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.jelly src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.properties src/main/resources/hudson/plugins/warnings/WarningsPublisher/config_de.properties http://jenkins-ci.org/commit/warnings-plugin/585be4161adaa8fcd9e4e491df89dad3c23f90b8 Log: [FIXED JENKINS-8833] Allow configuration of parsers for each file. Compare: https://github.com/jenkinsci/warnings-plugin/compare/18271af...585be41
          dogfood dogfood added a comment -

          Integrated in plugins_warnings #139
          [FIXED JENKINS-8833] Allow configuration of parsers for each file.

          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java
          • src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.jelly
          • pom.xml
          • src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.properties
          • src/main/java/hudson/plugins/warnings/WarningsPublisher.java
          • go.sh
          • src/main/java/hudson/plugins/warnings/parser/FileWarningsParser.java
          • src/main/resources/hudson/plugins/warnings/WarningsPublisher/config_de.properties
          • src/main/java/hudson/plugins/warnings/WarningsDescriptor.java
          • src/main/java/hudson/plugins/warnings/ParserConfiguration.java
          dogfood dogfood added a comment - Integrated in plugins_warnings #139 [FIXED JENKINS-8833] Allow configuration of parsers for each file. Ulli Hafner : Files : src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.jelly pom.xml src/main/resources/hudson/plugins/warnings/WarningsPublisher/config.properties src/main/java/hudson/plugins/warnings/WarningsPublisher.java go.sh src/main/java/hudson/plugins/warnings/parser/FileWarningsParser.java src/main/resources/hudson/plugins/warnings/WarningsPublisher/config_de.properties src/main/java/hudson/plugins/warnings/WarningsDescriptor.java src/main/java/hudson/plugins/warnings/ParserConfiguration.java

          People

            drulli Ulli Hafner
            languitar Johannes Wienke
            Votes:
            2 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: