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

Not supporting spaces in log directory ( Praqma case 10230 )

    XMLWordPrintable

Details

    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

          Ascending order - Click to sort in descending order
          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.

          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.
          asmodehn Alex V added a comment -

          Hi Jens, I just sent these to your email address

          asmodehn Alex V added a comment - Hi Jens, I just sent these to your email address
          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.

          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.
          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

          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
          madsnielsen Mads Nielsen added a comment -

          Fixed in 1.0

          madsnielsen Mads Nielsen added a comment - Fixed in 1.0
          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

          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
          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

          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

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

            Dates

              Created:
              Updated:
              Resolved: