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

clone workspace scm only saves workspace.tar.gz, not workspace.zip

XMLWordPrintable

      When selecting which format to save the file as, it seems it doesn't matter what you choose (Gzipped tar or Zipped), it always saves it as "workspace.tar.gz".

      This causes issues when the next job tries to build but can't find the "workspace.zip" file.

      Steps:
      In the publisher, pick Zipped
      Run the build
      Go to $JENKINS_HOME/jobs/$BUILD/lastSuccessful/
      Notice it is saved as .tar.gz, not .zip

        1. file_name_fix.patch
          1 kB

          [JENKINS-16695] clone workspace scm only saves workspace.tar.gz, not workspace.zip

          Pedro Rodriguez added a comment - - edited

          I found the issue. I've attached a patch for it.

          Pedro Rodriguez added a comment - - edited I found the issue. I've attached a patch for it.

          Martin Scherer added a comment -

          I tried your patch, but the file name is still "tar.gz". I try to use the clone as SCM on a different node, than it has been created.

          In the test cases the method string sometimes contains "zip" so we should compare with ignore case?

          Martin Scherer added a comment - I tried your patch, but the file name is still "tar.gz". I try to use the clone as SCM on a different node, than it has been created. In the test cases the method string sometimes contains "zip" so we should compare with ignore case?

          Pedro Rodriguez added a comment -

          There is a pull request for this, so ignore the patch.

          https://github.com/jenkinsci/jenkins-clone-workspace-scm-plugin/pull/7

          Though it is the same change, so the patch should work. I did not test multiple nodes, but I don't see how that would matter.

          Pedro Rodriguez added a comment - There is a pull request for this, so ignore the patch. https://github.com/jenkinsci/jenkins-clone-workspace-scm-plugin/pull/7 Though it is the same change, so the patch should work. I did not test multiple nodes, but I don't see how that would matter.

          evernat added a comment -

          This is merged and released.

          evernat added a comment - This is merged and released.

            abayer Andrew Bayer
            nacht Pedro Rodriguez
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: