Fix bug in previous commit; fixes #438

cmd/minify/fs.go

@@ -0,0 +1,21 @@
+package main
+
+import (
+	"io/fs"
+	"os"
+	"path/filepath"
+)
+
+func NewFS() fs.FS {
+	return dirFS("")
+}
+
+type dirFS string
+
+func (dir dirFS) Open(name string) (fs.File, error) {
+	return os.Open(filepath.Join(string(dir), name))
+}
+
+func (dir dirFS) Stat(name string) (fs.FileInfo, error) {
+	return os.Stat(filepath.Join(string(dir), name))
+}

cmd/minify/main.go

@@ -378,7 +378,7 @@ func run() int {
 		tasks = append(tasks, task)
 		roots = append(roots, "")
 	} else {
-		fsys := os.DirFS("").(fs.StatFS)
+		fsys := NewFS()
 		tasks, roots, err = createTasks(fsys, inputs, output)
 		if err != nil {
 			Error.Println(err)
@@ -565,7 +565,7 @@ func createTasks(fsys fs.FS, inputs []string, output string) ([]Task, []string,
 	tasks := []Task{}
 	roots := []string{}
 	for _, input := range inputs {
-		root := filepath.Clean(filepath.Dir(input)) // + string(os.PathSeparator) + ".")
+		root := filepath.Clean(filepath.Dir(input))
 		input = filepath.Clean(input)
 
 		var err error

cmd/minify/main_test.go

@@ -19,55 +19,25 @@ func TestCreateTasks(t *testing.T) {
 		tasks         map[string]string
 	}{
 		// root file
-		{"a.js", "", map[string]string{
-			"a.js": "a.js",
-		}},
-		{"a.js", ".", map[string]string{
-			"a.js": "a.js",
-		}},
-		{"a.js", "./", map[string]string{
-			"a.js": "a.js",
-		}},
-		{"a.js", "out", map[string]string{
-			"a.js": "out",
-		}},
-		{"a.js", "out/", map[string]string{
-			"a.js": "out/a.js",
-		}},
+		{"a.js", "", map[string]string{"a.js": "a.js"}},
+		{"a.js", ".", map[string]string{"a.js": "a.js"}},
+		{"a.js", "./", map[string]string{"a.js": "a.js"}},
+		{"a.js", "out", map[string]string{"a.js": "out"}},
+		{"a.js", "out/", map[string]string{"a.js": "out/a.js"}},
 
 		// nested file
-		{"dir/b.js", "", map[string]string{
-			"dir/b.js": "b.js",
-		}},
-		{"dir/b.js", ".", map[string]string{
-			"dir/b.js": "b.js",
-		}},
-		{"dir/b.js", "./", map[string]string{
-			"dir/b.js": "b.js",
-		}},
-		{"dir/b.js", "out", map[string]string{
-			"dir/b.js": "out",
-		}},
-		{"dir/b.js", "out/", map[string]string{
-			"dir/b.js": "out/b.js",
-		}},
+		{"dir/b.js", "", map[string]string{"dir/b.js": "b.js"}},
+		{"dir/b.js", ".", map[string]string{"dir/b.js": "b.js"}},
+		{"dir/b.js", "./", map[string]string{"dir/b.js": "b.js"}},
+		{"dir/b.js", "out", map[string]string{"dir/b.js": "out"}},
+		{"dir/b.js", "out/", map[string]string{"dir/b.js": "out/b.js"}},
 
 		// directory
-		{"dir", "", map[string]string{
-			"dir/b.js": "dir/b.js",
-		}},
-		{"dir", ".", map[string]string{
-			"dir/b.js": "dir/b.js",
-		}},
-		{"dir", "./", map[string]string{
-			"dir/b.js": "dir/b.js",
-		}},
-		{"dir", "out/", map[string]string{
-			"dir/b.js": "out/dir/b.js",
-		}},
-		{"dir/", "out/", map[string]string{
-			"dir/b.js": "out/b.js",
-		}},
+		{"dir", "", map[string]string{"dir/b.js": "dir/b.js"}},
+		{"dir", ".", map[string]string{"dir/b.js": "dir/b.js"}},
+		{"dir", "./", map[string]string{"dir/b.js": "dir/b.js"}},
+		{"dir", "out/", map[string]string{"dir/b.js": "out/dir/b.js"}},
+		{"dir/", "out/", map[string]string{"dir/b.js": "out/b.js"}},
 	}
 
 	recursive = true