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

Not supporting spaces in log directory ( Praqma case 10230 )

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Using Jenkins with Dr Memory plugin on windows, I have this error :

      Executing C:\Program Files (x86)\Jenkins\workspace\Project\bin\vs2010\Debug\Project.V2.Feature.test_d.exe -visual_studio
      Setting log path to C:\Program Files (x86)\Jenkins\workspace\Project\"bin\vs2010\Debug\"\8
      Recieved Unable to execute drmemory.exe -logdir C:\Program Files (x86)\Jenkins\workspace\Project\"bin\vs2010\Debug\"\8 -batch -quiet – C:\Program Files (x86)\Jenkins\workspace\Project\bin\vs2010\Debug\Project.V2.Feature.test_d.exe -visual_studio: ERROR: invalid -logdir: cannot find/write C:\Program

      As Jenkins install into "Program Files" folder on windows by default, this make the plugin unusable on default Jenkins install on this platform.

        Attachments

          Activity

          Hide
          jbrejner Jens Brejner added a comment -

          Hi Alex. That really looks strange. Would you mind sending me the full log output as well as the job configuration xml.

          Show
          jbrejner Jens Brejner added a comment - Hi Alex. That really looks strange. Would you mind sending me the full log output as well as the job configuration xml.
          Hide
          asmodehn Alex V added a comment -

          Hi Jens, I just sent these to your email address

          Show
          asmodehn Alex V added a comment - Hi Jens, I just sent these to your email address
          Hide
          jbrejner Jens Brejner added a comment -

          Hi Alex.
          Thank you, I've got them. We will take a look at it, and get back to you.

          Show
          jbrejner Jens Brejner added a comment - Hi Alex. Thank you, I've got them. We will take a look at it, and get back to you.
          Show
          asmodehn Alex V added a comment - I didnt write tests for the problem... But these seemed to fix my issue : https://github.com/asmodehn/drmemory-api/commit/1d7a732fc43beb580cd90c2e1914895bd75391ad https://github.com/asmodehn/drmemory-plugin/commit/f8e95495eb6ffe0b1a668d540d180f863f7384e8
          Hide
          madsnielsen Mads Nielsen added a comment -

          Fixed in 1.0

          Show
          madsnielsen Mads Nielsen added a comment - Fixed in 1.0
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Mads Nielsen
          Path:
          src/main/java/net/praqma/jenkins/plugin/drmemory/DrMemoryBuilder.java
          http://jenkins-ci.org/commit/drmemory-plugin/c2ace3fda30c7d583c7482501e6f1c7906a62fdc
          Log:
          Merge pull request #1 from asmodehn/master

          JENKINS-20141 Fix for path issues

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Mads Nielsen Path: src/main/java/net/praqma/jenkins/plugin/drmemory/DrMemoryBuilder.java http://jenkins-ci.org/commit/drmemory-plugin/c2ace3fda30c7d583c7482501e6f1c7906a62fdc Log: Merge pull request #1 from asmodehn/master JENKINS-20141 Fix for path issues
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Mads Nielsen
          Path:
          pom.xml
          http://jenkins-ci.org/commit/drmemory-plugin/0a361d0ae071754c0f5a97b2ce528a9848b97b0c
          Log:
          Updated pom to use changes that fixes JENKINS-20141

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Mads Nielsen Path: pom.xml http://jenkins-ci.org/commit/drmemory-plugin/0a361d0ae071754c0f5a97b2ce528a9848b97b0c Log: Updated pom to use changes that fixes JENKINS-20141

            People

            • Assignee:
              praqma Praqma Support
              Reporter:
              asmodehn Alex V
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: