Only populate $diff if we actually have any diffs.

Fixes release always failing due to "changes in the package.xml file" error.
horde · Jan 2, 2018 · bcd2f6e · bcd2f6e
1 parent 37f892d
commit bcd2f6e
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/lib/Components/Component/Source.php b/lib/Components/Component/Source.php
@@ -198,10 +198,13 @@ public function updatePackage($action, $options)
         case 'diff':
             $diff = '';
             foreach ($this->_wrappers as $wrapper) {
-                $path = $wrapper->getLocalPath($this->_directory);
-                $diff .= '--- a/' . $path . "\n"
-                    . '--- b/' . $path . "\n"
-                    . $wrapper->diff();
+                $diff_tmp = $wrapper->diff();
+                if (!empty($diff_tmp)) {
+                    $path = $wrapper->getLocalPath($this->_directory);
+                    $diff .= '--- a/' . $path . "\n"
+                        . '--- b/' . $path . "\n"
+                        . $wrapper->diff();
+                }
             }
             return $diff;
         default: