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

Warnngs regexp should be case insensitve

    XMLWordPrintable

Details

    • Bug
    • Status: Closed (View Workflow)
    • Major
    • Resolution: Fixed
    • warnings-plugin
    • None
    • Platform: PC, OS: All

    Description

      The Regex of MSBuildParser seems to be applied non-case-insensitive. Your regex
      expects the keywords 'warning' and 'error' with lower first letter. However the
      C# 2.0 compiler of Visual Studio 2005 produces 'Warning' and 'Error'.

      The RegexParser of Hudson Warnings needs to parse regex case insensitive or you
      need to change the regex in your MSBuildParser to allow upper-cased first
      letters, to support valid VS 2005 outputs.

      I have checked your Regex:
      (.)((\d).)\s:\s*(warning|error)\s*([^:]):\s(.*) with
      http://www.fileformat.info/tool/regex.htm,

      using the following input:

      [solution] Folder\File.cs(632,11): Warning CS0219: Die Variable result ist
      zugewiesen, ihr Wert wird aber nie verwendet.

      Attachments

        Issue Links

          Activity

            Code changed in hudson
            User: : drulli
            Path:
            trunk/hudson/plugins/warnings/src/main/java/hudson/plugins/warnings/parser/MsBuildParser.java
            trunk/hudson/plugins/warnings/src/test/java/hudson/plugins/warnings/parser/MsBuildParserTest.java
            http://fisheye4.cenqua.com/changelog/hudson/?cs=12485
            Log:
            [FIXED JENKINS-2383] Made regexp case insensitive. Applied patch of sjancke.

            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/MsBuildParser.java trunk/hudson/plugins/warnings/src/test/java/hudson/plugins/warnings/parser/MsBuildParserTest.java http://fisheye4.cenqua.com/changelog/hudson/?cs=12485 Log: [FIXED JENKINS-2383] Made regexp case insensitive. Applied patch of sjancke.
            sjancke sjancke added a comment -

            Created an attachment (id=386)
            This patch contains a Test reproducing the problem and the fix for MSBuildParser.java

            sjancke sjancke added a comment - Created an attachment (id=386) This patch contains a Test reproducing the problem and the fix for MSBuildParser.java

            People

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

              Dates

                Created:
                Updated:
                Resolved: