diff --git a/os.go b/os.go index f1366321..b48724ee 100644 --- a/os.go +++ b/os.go @@ -16,6 +16,7 @@ package afero import ( "os" + "path/filepath" "time" ) @@ -105,7 +106,11 @@ func (OsFs) LstatIfPossible(name string) (os.FileInfo, bool, error) { } func (OsFs) SymlinkIfPossible(oldname, newname string) error { - return os.Symlink(oldname, newname) + relpath, err := filepath.Rel(filepath.Dir(newname), oldname) + if err != nil { + return &os.LinkError{Op: "symlink", Old: oldname, New: newname, Err: err} + } + return os.Symlink(relpath, newname) } func (OsFs) ReadlinkIfPossible(name string) (string, error) { diff --git a/symlink_test.go b/symlink_test.go index 77dd742f..004d8f43 100644 --- a/symlink_test.go +++ b/symlink_test.go @@ -132,7 +132,7 @@ func TestReadlinkIfPossible(t *testing.T) { } testRead := func(r LinkReader, name string, output *string) { - _, err := r.ReadlinkIfPossible(name) + str, err := r.ReadlinkIfPossible(name) if (err != nil) && (output == nil) { t.Fatalf("Error reading link, expected success, got error: %v", err) } else if (err == nil) && (output != nil) { @@ -140,6 +140,7 @@ func TestReadlinkIfPossible(t *testing.T) { } else if err != nil && err.Error() != *output && !strings.HasSuffix(err.Error(), *output) { t.Fatalf("Error reading link, expected error '%v', instead received '%v'", *output, err) } + t.Logf("str: %v", str) } notSupported := ErrNoReadlink.Error()