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

Checkstyle input truncated

XMLWordPrintable

      When importing the attached sample.xml input first item in presented in the right way, while the second message is truncated after ")".

      IMO the XML is valid and is usable by other tools - so I suspect something wrong with the plugin

        1. jenkins-warning-nr-xml-trunc.png
          jenkins-warning-nr-xml-trunc.png
          81 kB
        2. sample.xml
          0.6 kB

          [JENKINS-63388] Checkstyle input truncated

          Ulli Hafner added a comment -

          From the Screenshot it looks like you are using the Cppcheck parser and not the CheckStyle parser. How did you configure the plugin?

          Ulli Hafner added a comment - From the Screenshot it looks like you are using the Cppcheck parser and not the CheckStyle parser. How did you configure the plugin?

          Konrad Weihmann added a comment -

          Okay agree, the screenshot might be misleading and yes the input is not coming from checkstyle tool, but is using the checkstyle format.
          To answer your question: Checkstyle + pattern = "*/.xml" in this particular case

          Konrad Weihmann added a comment - Okay agree, the screenshot might be misleading and yes the input is not coming from checkstyle tool, but is using the checkstyle format. To answer your question: Checkstyle + pattern = "* / .xml" in this particular case

          Arnab added a comment -

          Hello drulli I've just tested the xml file with "CheckStyle" parser and it works as expected.

          Also, the xml format is of "CheckStyle" format. Since, the tests are passing, I don't think that there is any need of modifying "CheckStyle" Parser let alone "Cppcheck" parser.

          What do you think?

          I'll push the PR soon with the test cases

          Arnab added a comment - Hello drulli I've just tested the xml file with "CheckStyle" parser and it works as expected. Also, the xml format is of "CheckStyle" format. Since, the tests are passing, I don't think that there is any need of modifying "CheckStyle" Parser let alone "Cppcheck" parser. What do you think? I'll push the PR soon with the test cases

          Ulli Hafner added a comment -

          If the parser works as expected, then we can close this issue. If you already have a test, then we can push only the test.

          Ulli Hafner added a comment - If the parser works as expected, then we can close this issue. If you already have a test, then we can push only the test.

          Arnab added a comment -

          Yes that make sense! I already have the test ready to push

          Arnab added a comment - Yes that make sense! I already have the test ready to push

            code_arnab Arnab
            kweihmann Konrad Weihmann
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: