shithub: hugo

Download patch

ref: de82404d35bc0673ca4fe5c400945b3fc4e9aae2
parent: a1d232f4b674dcdc7757ccc3bdae2f95f0733720
author: Anthony Fok <[email protected]>
date: Sat Dec 19 00:19:31 EST 2015

Fix hugo server "Watching for changes" path display

especially when the given `--source` path is a relative directory.

Also, when `--source` is specified, make WorkingDir an absolute path
from the very beginning, to be consistent with the case when `--source`
is not given.  Otherwise, the function name helpers.AbsPathify(), which
prepends WorkingDir to a relative path, does not really make sense.

Fixes #1721

--- a/commands/hugo.go
+++ b/commands/hugo.go
@@ -329,7 +329,8 @@
 	}
 
 	if Source != "" {
-		viper.Set("WorkingDir", Source)
+		dir, _ := filepath.Abs(Source)
+		viper.Set("WorkingDir", dir)
 	} else {
 		dir, _ := os.Getwd()
 		viper.Set("WorkingDir", dir)
--- a/commands/server.go
+++ b/commands/server.go
@@ -159,7 +159,7 @@
 	// Watch runs its own server as part of the routine
 	if serverWatch {
 		watchDirs := getDirList()
-		baseWatchDir := helpers.AbsPathify(viper.GetString("WorkingDir"))
+		baseWatchDir := viper.GetString("WorkingDir")
 		for i, dir := range watchDirs {
 			watchDirs[i], _ = helpers.GetRelativePath(dir, baseWatchDir)
 		}
--- a/helpers/path.go
+++ b/helpers/path.go
@@ -130,7 +130,7 @@
 		return filepath.Clean(inPath)
 	}
 
-	// todo consider move workingDir to argument list
+	// TODO(bep): Consider moving workingDir to argument list
 	return filepath.Clean(filepath.Join(viper.GetString("WorkingDir"), inPath))
 }