Merge pull request #20 from juanjol/make-git-dependency-optional

Issue #19: Added functionality to check if there is git installed and…

src/BaseCommand.php

@@ -177,11 +177,15 @@ protected function assertArtifactContentIsClean() {
       $this->getSymlinks(),
       $this->getConfiguration()->getInclude(),
     ));
-    $files_changed = trim($this->runCommand(sprintf("git status -s %s", implode(' ', $artifact_content)))->getOutput());
-    if (strlen($files_changed > 0)) {
-      $list = implode("\n", array_map(function($item) { $parts = explode(' ', $item); return $parts[1]; }, explode("\n", $files_changed)));
-      throw new \Exception("There are changes in the repository (changed and/or untracked files), please run this artifact generation script with folder tree clean. Files changed:\n$list");
+
+    if ($this->gitCommandExist() && $this->isGitRepository()) {
+      $files_changed = trim($this->runCommand(sprintf("git status -s %s", implode(' ', $artifact_content)))->getOutput());
+      if (strlen($files_changed > 0)) {
+        $list = implode("\n", array_map(function($item) { $parts = explode(' ', $item); return $parts[1]; }, explode("\n", $files_changed)));
+        throw new \Exception("There are changes in the repository (changed and/or untracked files), please run this artifact generation script with folder tree clean. Files changed:\n$list");
+      }
     }
+
   }
 
   /**
@@ -236,4 +240,23 @@ protected function getSymlinks() {
     return ['docroot', 'web', 'public_html'];
   }
 
+  /**
+   * Check if git command exists
+   *
+   * @return bool
+   */
+  protected function gitCommandExist() {
+    return $this->runCommand('which git')->isSuccessful();
+  }
+
+  /**
+   * Check if we are in a git repository
+   *
+   * @return bool
+   */
+  protected function isGitRepository() {
+    $command_output = $this->runCommand('git rev-parse --is-inside-work-tree || true')->getOutput();
+    return $command_output === 'true';
+  }
+
 }

src/DrupalArtifactBuilderCreate.php

@@ -38,7 +38,6 @@ protected function execute(InputInterface $input, OutputInterface $output) : int
   protected function generateArtifact() {
     // Create the folder with the artifact.
     $this->createArtifactFolder();
-    $this->assertArtifactContentIsClean();
 
     $this->log('Cleaning previous artifact');
     $this->log('##########################');

src/DrupalArtifactBuilderGit.php

@@ -32,6 +32,10 @@ protected function configure() {
   protected function initialize(InputInterface $input, OutputInterface $output) {
     parent::initialize($input, $output);
 
+    if (!$this->gitCommandExist()) {
+      throw new \RuntimeException("Git command not found. Git must be installed and available in the PATH variable to generate an artifact.");
+    }
+
     // Branch setup.
     if ($input->hasOption('repository') && !empty($input->getOption('repository'))) {
       $this->config->setRepository($input->getOption('repository'));