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

Add ability to specify valgrind command line options to plugin configuration

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Minor Minor
    • valgrind-plugin
    • None

      It would be nice to have the ability to specify valgrind options in a dialog for the plugin (suppression files, etc).

          [JENKINS-15138] Add ability to specify valgrind command line options to plugin configuration

          David Ritter created issue -

          Code changed in jenkins
          User: Johannes Ohlemacher
          Path:
          src/main/java/org/jenkinsci/plugins/valgrind/ValgrindBuilder.java
          src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindBuilder/config.jelly
          http://jenkins-ci.org/commit/valgrind-plugin/17bef8c049d3eb525dd66c6436d326964a2a7888
          Log:
          added support for arbitrary valgrind options (JENKINS-15139)
          added support for program options (JENKINS-15138)

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Johannes Ohlemacher Path: src/main/java/org/jenkinsci/plugins/valgrind/ValgrindBuilder.java src/main/resources/org/jenkinsci/plugins/valgrind/ValgrindBuilder/config.jelly http://jenkins-ci.org/commit/valgrind-plugin/17bef8c049d3eb525dd66c6436d326964a2a7888 Log: added support for arbitrary valgrind options ( JENKINS-15139 ) added support for program options ( JENKINS-15138 )

          fixed with release 0.12

          Johannes Ohlemacher added a comment - fixed with release 0.12
          Johannes Ohlemacher made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          Tomasz Ducin added a comment -

          Is there any possibility to specify executable options within this plugin?

          So far I can specify valgrind options, valgrind argument (the binary itself), the executable arguments - and how about executable options?

          Tomasz Ducin added a comment - Is there any possibility to specify executable options within this plugin? So far I can specify valgrind options, valgrind argument (the binary itself), the executable arguments - and how about executable options?

          @Tomasz
          what do you mean with "executable options"? the executable arguments are command line options passed to the executable under test.

          Johannes Ohlemacher added a comment - @Tomasz what do you mean with "executable options"? the executable arguments are command line options passed to the executable under test.

          Tomasz Ducin added a comment -

          @Johannes

          Thanks for reply. I've got a C++ executable program that takes "-c" option which points configuration file path. The binary is executed in such way:

          ./service -c config_file.cfg

          executable argument would be:

          ./service arg1

          When I try to use executable arguments, the plugin executes following command:

          valgrind [...] service "-c config_file.cfg"

          and I'd like to run:

          valgrind [...] service -c config_file.cfg

          (without ")

          Tomasz Ducin added a comment - @Johannes Thanks for reply. I've got a C++ executable program that takes "-c" option which points configuration file path. The binary is executed in such way: ./service -c config_file.cfg executable argument would be: ./service arg1 When I try to use executable arguments, the plugin executes following command: valgrind [...] service "-c config_file.cfg" and I'd like to run: valgrind [...] service -c config_file.cfg (without ")

          @Tomasz
          Thats just a bug, see JENKINS-16123. Its already fixed and will be part of the next release. Unfortunately you have to wait for the release until end of february, sorry

          Johannes Ohlemacher added a comment - @Tomasz Thats just a bug, see JENKINS-16123 . Its already fixed and will be part of the next release. Unfortunately you have to wait for the release until end of february, sorry
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 145858 ] New: JNJira + In-Review [ 191661 ]

            existence Johannes Ohlemacher
            dritter29 David Ritter
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: