diff options
author | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2022-05-09 10:05:19 +0200 |
---|---|---|
committer | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2022-05-09 14:18:40 +0200 |
commit | 51f08b0b6aae175299b4e73d521343a4386a9cf7 (patch) | |
tree | 6966bc277671a1f62e9363a2a16fa7a2d6ccacd1 /commands/hugo.go | |
parent | 860c51c314e1f2b06b1424a3b277a2db96fc1f04 (diff) |
Revise the use of htime.Since/htime.Now
We cannot (also, it doesn't add any value) use that when the `clock` is set,
* To measure time (before that global is set)
* To compare file timestamps re cache eviction
Fixes #9868
Diffstat (limited to 'commands/hugo.go')
-rw-r--r-- | commands/hugo.go | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/commands/hugo.go b/commands/hugo.go index 43ec7e5c7..c13fdce06 100644 --- a/commands/hugo.go +++ b/commands/hugo.go @@ -681,7 +681,11 @@ func (c *commandeer) firstPathSpec() *helpers.PathSpec { } func (c *commandeer) timeTrack(start time.Time, name string) { - elapsed := htime.Since(start) + // Note the use of time.Since here and time.Now in the callers. + // We have a htime.Sinnce, but that may be adjusted to the future, + // and that does not make sense here, esp. when used before the + // global Clock is initialized. + elapsed := time.Since(start) c.logger.Printf("%s in %v ms", name, int(1000*elapsed.Seconds())) } @@ -792,7 +796,7 @@ func (c *commandeer) fullRebuild(changeType string) { time.Sleep(2 * time.Second) }() - defer c.timeTrack(htime.Now(), "Rebuilt") + defer c.timeTrack(time.Now(), "Rebuilt") c.commandeerHugoState = newCommandeerHugoState() err := c.loadConfig() @@ -1137,7 +1141,7 @@ func (c *commandeer) handleEvents(watcher *watcher.Batcher, c.changeDetector.PrepareNew() func() { - defer c.timeTrack(htime.Now(), "Total") + defer c.timeTrack(time.Now(), "Total") if err := c.rebuildSites(dynamicEvents); err != nil { c.handleBuildErr(err, "Rebuild failed") } |