• Icon: New Feature New Feature
    • Resolution: Unresolved
    • Icon: Minor Minor
    • valgrind-plugin
    • None

      I would like the plugin to support Helgrind (in addition to Memcheck).

      With very small changes to the code, I already managed to run Helgrind from the plugin (without the option of any configuration or command line arguments) and process the XML results (currently only for one error kind).

      If this is generally something you would like to include in the plugin, I'd be happy to contribute.

          [JENKINS-25776] Support for Helgrind

          Adriaan Schmidt added a comment - Addressed in this pull request: https://github.com/jenkinsci/valgrind-plugin/pull/7

          Code changed in jenkins
          User: Johannes Ohlemacher
          Path:
          src/main/java/org/jenkinsci/plugins/valgrind/ValgrindBuildAction.java
          src/main/java/org/jenkinsci/plugins/valgrind/ValgrindBuilder.java
          src/main/java/org/jenkinsci/plugins/valgrind/ValgrindDetail.java
          src/main/java/org/jenkinsci/plugins/valgrind/ValgrindErrorDetail.java
          src/main/java/org/jenkinsci/plugins/valgrind/ValgrindResult.java
          src/main/java/org/jenkinsci/plugins/valgrind/ValgrindThreadDetail.java
          src/main/java/org/jenkinsci/plugins/valgrind/model/ValgrindErrorKind.java
          src/main/java/org/jenkinsci/plugins/valgrind/model/ValgrindProcess.java
          src/main/java/org/jenkinsci/plugins/valgrind/model/ValgrindReport.java
          src/main/java/org/jenkinsci/plugins/valgrind/model/ValgrindThread.java
          src/main/java/org/jenkinsci/plugins/valgrind/parser/ValgrindSaxParser.java
          src/main/java/org/jenkinsci/plugins/valgrind/util/ValgrindErrorList.java
          src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindBuilder/Helgrind.jelly
          src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindBuilder/Memcheck.jelly
          src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindBuilder/config.jelly
          src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindDetail/index.jelly
          src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindErrorDetail/index.jelly
          src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindProcessDetails/index.jelly
          src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindResult/errorlist.jelly
          src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindResult/index.jelly
          src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindResult/threadlist.jelly
          src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindThreadDetail/index.jelly
          http://jenkins-ci.org/commit/valgrind-plugin/75592fbf4ad1e9382e465441a8c2306a51ca9f82
          Log:
          Merge pull request #7 from adriaan42/JENKINS-25776

          Helgrind Support

          Compare: https://github.com/jenkinsci/valgrind-plugin/compare/27c6d08eb408...75592fbf4ad1

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Johannes Ohlemacher Path: src/main/java/org/jenkinsci/plugins/valgrind/ValgrindBuildAction.java src/main/java/org/jenkinsci/plugins/valgrind/ValgrindBuilder.java src/main/java/org/jenkinsci/plugins/valgrind/ValgrindDetail.java src/main/java/org/jenkinsci/plugins/valgrind/ValgrindErrorDetail.java src/main/java/org/jenkinsci/plugins/valgrind/ValgrindResult.java src/main/java/org/jenkinsci/plugins/valgrind/ValgrindThreadDetail.java src/main/java/org/jenkinsci/plugins/valgrind/model/ValgrindErrorKind.java src/main/java/org/jenkinsci/plugins/valgrind/model/ValgrindProcess.java src/main/java/org/jenkinsci/plugins/valgrind/model/ValgrindReport.java src/main/java/org/jenkinsci/plugins/valgrind/model/ValgrindThread.java src/main/java/org/jenkinsci/plugins/valgrind/parser/ValgrindSaxParser.java src/main/java/org/jenkinsci/plugins/valgrind/util/ValgrindErrorList.java src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindBuilder/Helgrind.jelly src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindBuilder/Memcheck.jelly src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindBuilder/config.jelly src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindDetail/index.jelly src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindErrorDetail/index.jelly src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindProcessDetails/index.jelly src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindResult/errorlist.jelly src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindResult/index.jelly src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindResult/threadlist.jelly src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindThreadDetail/index.jelly http://jenkins-ci.org/commit/valgrind-plugin/75592fbf4ad1e9382e465441a8c2306a51ca9f82 Log: Merge pull request #7 from adriaan42/ JENKINS-25776 Helgrind Support Compare: https://github.com/jenkinsci/valgrind-plugin/compare/27c6d08eb408...75592fbf4ad1

          Code changed in jenkins
          User: Johannes Ohlemacher
          Path:
          src/main/java/org/jenkinsci/plugins/valgrind/graph/ValgrindGraph.java
          http://jenkins-ci.org/commit/valgrind-plugin/4697c31a660603516b4f8eb546e78d045da6b321
          Log:
          Merge pull request #8 from adriaan42/JENKINS-25776

          Fix layout of trend graph

          Compare: https://github.com/jenkinsci/valgrind-plugin/compare/75592fbf4ad1...4697c31a6606

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Johannes Ohlemacher Path: src/main/java/org/jenkinsci/plugins/valgrind/graph/ValgrindGraph.java http://jenkins-ci.org/commit/valgrind-plugin/4697c31a660603516b4f8eb546e78d045da6b321 Log: Merge pull request #8 from adriaan42/ JENKINS-25776 Fix layout of trend graph Compare: https://github.com/jenkinsci/valgrind-plugin/compare/75592fbf4ad1...4697c31a6606

            existence Johannes Ohlemacher
            adriaan42 Adriaan Schmidt
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: