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

Java compiler warnings parser can't cope with Maven timestamps in log

    • 5.0.0 (analysis-model and warnings-ng)

      We switched on Maven timestamps in our log, since it's a handy way to check build times.

      In Maven, this is done by editing conf/logging/simplelogger.properties and setting org.slf4j.simpleLogger.showDateTime=true

      This results in compiler warnings like this:

      08:52:34.395 [WARNING] /my/path/to/workspace/src/main/java/my/package/TemporalJsonSeqFiling.java:[86,76] redundant cast to long
      

      ( given that the date/time format is org.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss.SSS )

      Taking a look at the source code, it seems that JAVAC_WARNING_PATTERN does a full match of the line, which does not accept that something could exist before [WARNING] .

      We use other parsers for Checkstyle, PMD etc. and they work fine.

          [JENKINS-56737] Java compiler warnings parser can't cope with Maven timestamps in log

          cristalp created issue -
          Ulli Hafner made changes -
          Epic Link New: JENKINS-56456 [ 198054 ]
          Ulli Hafner made changes -
          Component/s New: analysis-model [ 23523 ]
          Component/s Original: analysis-model-api-plugin [ 24221 ]
          Labels New: help-wanted newbie-friendly
          Ulli Hafner made changes -
          Assignee Original: Ulli Hafner [ drulli ]
          Tobias Schaffner made changes -
          Assignee New: Tobias Schaffner [ tschaffn ]
          Ulli Hafner made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Ulli Hafner made changes -
          Rank New: Ranked higher
          Ulli Hafner made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Fixed but Unreleased [ 10203 ]
          Ulli Hafner made changes -
          Released As New: 5.0.0 (analysis-model and warnings-ng)
          Status Original: Fixed but Unreleased [ 10203 ] New: Resolved [ 5 ]

            tschaffn Tobias Schaffner
            cristalp cristalp
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: