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

Publish Over FTP 1.8 not working with spaces in filename/path of source files

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Critical
    • Resolution: Fixed
    • Labels:
      None
    • Environment:
      Jenkins server is running Windows 7. Jenkins build clients on Windows 7 and Mac OSX10.8 both have the same problem.
    • Similar Issues:

      Description

      "Transfer Set"/"Source Files" does not support spaces.

      I want to transfer something like
      Project/Output/Install Product.exe
      Or possibly:
      Project Output/*.exe

      Quoting the path does not help.

        Attachments

          Activity

          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: bap2000
          Path:
          src/main/java/jenkins/plugins/publish_over_ftp/BapFtpTransfer.java
          src/main/java/jenkins/plugins/publish_over_ftp/descriptor/BapFtpTransferDescriptor.java
          src/main/java/jenkins/plugins/publish_over_ftp/options/FtpOverrideTransferDefaults.java
          src/main/resources/jenkins/plugins/publish_over_ftp/BapFtpTransfer/config.jelly
          src/main/resources/jenkins/plugins/publish_over_ftp/BapFtpTransfer/help-patternSeparator.html
          src/main/resources/jenkins/plugins/publish_over_ftp/options/FtpOverrideTransferDefaults/config.jelly
          src/test/java/jenkins/plugins/publish_over_ftp/jenkins/CurrentConfigurationTest.java
          src/test/java/jenkins/plugins/publish_over_ftp/jenkins/IntegrationTest.java
          src/test/java/jenkins/plugins/publish_over_ftp/jenkins/LegacyConfigurationTest.java
          http://jenkins-ci.org/commit/publish-over-ftp-plugin/a972ce7c0c4d6695e7a80aa5de96f3e833e04d9a
          Log:
          [FIXED JENKINS-16681] Don't let Ant separate the patterns - and allow the separator regex to be configured to enable patterns with spaces in them


          You received this message because you are subscribed to the Google Groups "Jenkins Commits" group.
          To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscribe@googlegroups.com.
          For more options, visit https://groups.google.com/groups/opt_out.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: bap2000 Path: src/main/java/jenkins/plugins/publish_over_ftp/BapFtpTransfer.java src/main/java/jenkins/plugins/publish_over_ftp/descriptor/BapFtpTransferDescriptor.java src/main/java/jenkins/plugins/publish_over_ftp/options/FtpOverrideTransferDefaults.java src/main/resources/jenkins/plugins/publish_over_ftp/BapFtpTransfer/config.jelly src/main/resources/jenkins/plugins/publish_over_ftp/BapFtpTransfer/help-patternSeparator.html src/main/resources/jenkins/plugins/publish_over_ftp/options/FtpOverrideTransferDefaults/config.jelly src/test/java/jenkins/plugins/publish_over_ftp/jenkins/CurrentConfigurationTest.java src/test/java/jenkins/plugins/publish_over_ftp/jenkins/IntegrationTest.java src/test/java/jenkins/plugins/publish_over_ftp/jenkins/LegacyConfigurationTest.java http://jenkins-ci.org/commit/publish-over-ftp-plugin/a972ce7c0c4d6695e7a80aa5de96f3e833e04d9a Log: [FIXED JENKINS-16681] Don't let Ant separate the patterns - and allow the separator regex to be configured to enable patterns with spaces in them – You received this message because you are subscribed to the Google Groups "Jenkins Commits" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscribe@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out .
          Hide
          ventuz Alex Klein added a comment -

          Okay, it should have been obvious but I didn't find it right away: it now works, but one has to change the separator pattern in the advanced options from "[, ]" to "[,]". Otherwise the whitespace is still interpreted as a possibility to split the string into multiple parts. Perhaps the default pattern should be changed?

          Show
          ventuz Alex Klein added a comment - Okay, it should have been obvious but I didn't find it right away: it now works, but one has to change the separator pattern in the advanced options from " [, ] " to " [,] ". Otherwise the whitespace is still interpreted as a possibility to split the string into multiple parts. Perhaps the default pattern should be changed?

            People

            • Assignee:
              bap bap
              Reporter:
              fatpete Peter Fornwall
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: