Fixes acquia#2633: Make git hook symlinks relative.

src/Robo/Commands/Setup/SettingsCommand.php

@@ -217,15 +217,16 @@ public function gitHooks() {
   protected function installGitHook($hook) {
     if ($this->getConfigValue('git.hooks.' . $hook)) {
       $this->say("Installing $hook git hook...");
-      $source = $this->getConfigValue('git.hooks.' . $hook) . "/$hook";
-      $dest = $this->getConfigValue('repo.root') . "/.git/hooks/$hook";
+      $bltHook = $this->getConfigValue('git.hooks.' . $hook) . "/$hook";
+      $projectHookDirectory = $this->getConfigValue('repo.root') . "/.git/hooks";
       $fs = new Filesystem();
-      $source = rtrim($fs->makePathRelative($source, $dest), '/');
+      $pathToBltHook = rtrim($fs->makePathRelative($bltHook, $projectHookDirectory), '/');
+      $projectHook = $projectHookDirectory . "/$hook";
 
       $result = $this->taskFilesystemStack()
         ->mkdir($this->getConfigValue('repo.root') . '/.git/hooks')
-        ->remove($dest)
-        ->symlink($source, $dest)
+        ->remove($projectHook)
+        ->symlink($pathToBltHook, $projectHook)
         ->stopOnFail()
         ->setVerbosityThreshold(VerbosityThresholdInterface::VERBOSITY_VERBOSE)
         ->run();

tests/phpunit/BltProject/SetupGitHooksTest.php

@@ -16,8 +16,17 @@ class GitTasksTest extends BltProjectTestBase {
   public function testGitConfig() {
     $this->blt("blt:init:git-hooks");
     $this->assertFileExists($this->sandboxInstance . '/.git');
-    $this->assertFileExists($this->sandboxInstance . '/.git/hooks/commit-msg');
-    $this->assertFileExists($this->sandboxInstance . '/.git/hooks/pre-commit');
+    $hooksPath = $this->sandboxInstance . '/.git/hooks';
+    $hookNames = [
+      'commit-msg',
+      'pre-commit',
+    ];
+    foreach ($hookNames as $hookName) {
+      $projectHook = $hooksPath . $hookName;
+      $this->assertFileExists($projectHook);
+      $bltHook = readlink($projectHook);
+      $this->assertFileExists($bltHook);
+    }
   }
 
   /**