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

          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

          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

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

              Created:
              Updated:
              Resolved: