]> ruderich.org/simon Gitweb - safcm/safcm.git/commitdiff
remote: treat paths as slash separated in triggerPaths()
authorSimon Ruderich <simon@ruderich.org>
Sun, 23 May 2021 10:33:06 +0000 (12:33 +0200)
committerSimon Ruderich <simon@ruderich.org>
Sun, 23 May 2021 10:33:06 +0000 (12:33 +0200)
Remotes are only supported on UNIX systems which use slashes as path
separators so this had no direct effect. Change it to make the code more
obvious.

This was forgotten in afb7e8e (safcm: use only slash separated paths for
the configuration, 2021-05-13).

remote/sync/triggers.go

index d201835e7e7271b9729da033a409c53b004edce9..74d45c3175b1b4aa2fcf670f20dfc12d0d2c43dc 100644 (file)
@@ -18,7 +18,7 @@
 package sync
 
 import (
-       "path/filepath"
+       slashpath "path"
        "strings"
 
        "ruderich.org/simon/safcm"
@@ -50,7 +50,8 @@ func (s *Sync) queueTriggers(file *safcm.File) {
 // more specific triggers can override effects of less specific ones (first
 // "/" or ".", then the parents and finally path itself).
 func triggerPaths(path string) []string {
-       sep := string(filepath.Separator)
+       // Slash separated paths are used for the configuration
+       const sep = "/"
        if path == sep || path == "." {
                return []string{path}
        }
@@ -65,7 +66,7 @@ func triggerPaths(path string) []string {
 
        var res []string
        for i := 0; i < len(parts); i++ {
-               res = append(res, filepath.Join(parts[:i+1]...))
+               res = append(res, slashpath.Join(parts[:i+1]...))
        }
        return res
 }