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

msbuild compiler warning not detected / extracted

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Minor
    • Resolution: Fixed
    • warnings-plugin
    • None
    • Jenkins 2.7.1, Warnings Plug-in 4.56, Master is an Ubuntu 14.04 system, Slave is a Windows 10 system

    Description

      The console out of the job http://ci.ros2.org/job/ci_windows/1651/consoleFull shows the following compiler warning:

      (ClCompile target) ->
      cl : Command line warning D9002: ignoring unknown option '-std=c++11'

      but the result says: "MSBuild Warnings: 0 warnings".

      I am happy to provide more information if necessary and/or try a from-source build to check if it makes any difference.

      Attachments

        Activity

          drulli Ulli Hafner added a comment -

          This warning totally looks different, here is an example from my tests: https://github.com/jenkinsci/warnings-plugin/blob/master/src/test/resources/hudson/plugins/warnings/parser/msbuild.txt

          Is this the only warning that is not parsed?

          drulli Ulli Hafner added a comment - This warning totally looks different, here is an example from my tests: https://github.com/jenkinsci/warnings-plugin/blob/master/src/test/resources/hudson/plugins/warnings/parser/msbuild.txt Is this the only warning that is not parsed?
          dthomas Dirk Thomas added a comment -

          Yes, that is the only warning not being recognized (the same warning appears twice in the console log).

          In contrast to the cases in your test which are all file specific these ones are warning for the command line arguments. Therefore it doesn't start with a filename / row / column but just with `Command line warning ...`.

          dthomas Dirk Thomas added a comment - Yes, that is the only warning not being recognized (the same warning appears twice in the console log). In contrast to the cases in your test which are all file specific these ones are warning for the command line arguments. Therefore it doesn't start with a filename / row / column but just with `Command line warning ...`.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/warnings/parser/MsBuildParser.java
          src/test/java/hudson/plugins/warnings/parser/MsBuildParserTest.java
          src/test/resources/hudson/plugins/warnings/parser/issue38215.txt
          http://jenkins-ci.org/commit/warnings-plugin/4834ebc22697bf682b0024ba0aa1c03d4f07a542
          Log:
          [FIXED JENKINS-38215] Detect MSBuild command line warnings.

          Compare: https://github.com/jenkinsci/warnings-plugin/compare/01a54cc79a4a...4834ebc22697

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/warnings/parser/MsBuildParser.java src/test/java/hudson/plugins/warnings/parser/MsBuildParserTest.java src/test/resources/hudson/plugins/warnings/parser/issue38215.txt http://jenkins-ci.org/commit/warnings-plugin/4834ebc22697bf682b0024ba0aa1c03d4f07a542 Log: [FIXED JENKINS-38215] Detect MSBuild command line warnings. Compare: https://github.com/jenkinsci/warnings-plugin/compare/01a54cc79a4a...4834ebc22697
          dthomas Dirk Thomas added a comment -

          Thank you very much for addressing this. I am looking forward for the next release to use it.

          dthomas Dirk Thomas added a comment - Thank you very much for addressing this. I am looking forward for the next release to use it.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/warnings/parser/MsBuildParser.java
          http://jenkins-ci.org/commit/warnings-plugin/5e3f481071b86ba45b666413978ea9f53bdc1790
          Log:
          Improved fix for JENKINS-38215.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/warnings/parser/MsBuildParser.java http://jenkins-ci.org/commit/warnings-plugin/5e3f481071b86ba45b666413978ea9f53bdc1790 Log: Improved fix for JENKINS-38215 .

          People

            drulli Ulli Hafner
            dthomas Dirk Thomas
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: