Merge pull request #279 from marmotz/EscapeTarGzExcludes

Escape targz excludes to allow wildcard
9 years ago · a1e5677192
parent 8ab3a25a72 ab4cbeafbd
commit a1e5677192
1 changed files with 5 additions and 2 deletions
--- a/Mage/Task/BuiltIn/Deployment/Strategy/TarGzTask.php
+++ b/Mage/Task/BuiltIn/Deployment/Strategy/TarGzTask.php
@ -46,8 +46,7 @@ class TarGzTask extends BaseStrategyTaskAbstract implements IsReleaseAware
        $this->checkOverrideRelease();

        $excludes = $this->getExcludes();
-        $excludesListFilePath   = $this->getConfig()->deployment('excludes_file', '');
-        ;
+        $excludesListFilePath = $this->getConfig()->deployment('excludes_file', '');

        // If we are working with releases
        $deployToDirectory = $this->getConfig()->deployment('to');
@ -65,6 +64,10 @@ class TarGzTask extends BaseStrategyTaskAbstract implements IsReleaseAware
        $remoteTarGz = basename($localTarGz);
        $excludeCmd = '';
        foreach ($excludes as $excludeFile) {
+            if (strpos($excludeFile, '*') !== false) {
+                $excludeFile = '"' . $excludeFile . '"';
+            }
+
            $excludeCmd .= ' --exclude=' . $excludeFile;
        }