Resolve indir to absolute path at parsing time

conf/parse.go

@@ -9,6 +9,7 @@ package conf
 import (
 	"fmt"
 	"path"
+	"path/filepath"
 	"runtime"
 	"strings"
 )
@@ -247,6 +248,10 @@ Loop:
 			dir = strings.Replace(
 				dir, confVarName, p.config.variables[confVarName], -1,
 			)
+			dir, err := filepath.Abs(dir)
+			if err != nil {
+				p.errorf("%s", err)
+			}
 			block.InDir = dir
 		case itemDaemon:
 			options := p.collectValues(itemBareString)

conf/parse_test.go

@@ -1,12 +1,21 @@
 package conf
 
 import (
+	"path/filepath"
 	"syscall"
 	"testing"
 
 	"github.com/google/go-cmp/cmp"
 )
 
+func mustAbs(s string) string {
+	f, err := filepath.Abs(s)
+	if err != nil {
+		panic(err)
+	}
+	return f
+}
+
 var parseTests = []struct {
 	path     string
 	input    string
@@ -265,7 +274,7 @@ var parseTests = []struct {
 		"{ indir: foo\n }",
 		&Config{
 			Blocks: []Block{
-				{InDir: "foo"},
+				{InDir: mustAbs("foo")},
 			},
 		},
 	},
@@ -283,7 +292,7 @@ var parseTests = []struct {
 		"{ indir: @confdir/foo\n }",
 		&Config{
 			Blocks: []Block{
-				{InDir: "path/to/foo"},
+				{InDir: mustAbs("path/to/foo")},
 			},
 			variables: map[string]string{
 				"@confdir": "path/to",
@@ -292,14 +301,18 @@ var parseTests = []struct {
 	},
 }
 
+var parseCmpOptions = []cmp.Option{
+	cmp.AllowUnexported(Config{}),
+}
+
 func TestParse(t *testing.T) {
 	for i, tt := range parseTests {
 		ret, err := Parse(tt.path, tt.input)
 		if err != nil {
 			t.Fatalf("%q - %s", tt.input, err)
 		}
 
-		if diff := cmp.Diff(ret, tt.expected, cmp.AllowUnexported(Config{})); diff != "" {
+		if diff := cmp.Diff(ret, tt.expected, parseCmpOptions...); diff != "" {
 			t.Errorf("%d %s", i, diff)
 		}
 	}