Merge pull request #7 from foomo/fix/absolute-target

fix: absolute paths
foomo · Dec 20, 2024 · 4aeb3a3 · 4aeb3a3
2 parents 8b6e2b0 + 48e6edc
commit 4aeb3a3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/Makefile b/Makefile
@@ -24,7 +24,7 @@ export PATH := bin:$(PATH)
 
 .PHONY: brew
 ## Install project binaries
-brew:
+brew: install
 	@ownbrew install
 
 .PHONY: doc

diff --git a/pkg/ownbrew/ownbrew.go b/pkg/ownbrew/ownbrew.go
@@ -204,11 +204,10 @@ func (o *Ownbrew) symlink(source, target string) error {
 		return err
 	}
 
-	prefix, err := filepath.Rel(filepath.Base(target), "")
-	if err != nil {
-		return err
+	var prefix string
+	if value, err := filepath.Rel(filepath.Base(target), source); err == nil {
+		prefix = strings.TrimSuffix(value, ".")
 	}
-	prefix = strings.TrimSuffix(prefix, ".")
 
 	o.l.Debug("symlink:", prefix+source, target)
 	return os.Symlink(prefix+source, target)