Fix crash with "config", "check" and "benchmark"

The crash was introduced by commit 00d04774 in PR #1652
where access to NoTimes was attempted even though --noTimes
was not defined for the config, check and benchmark commands.

Special thanks to @bep for the heads-up!

See also #1624 - CLI UX: Flags shouldn't be global

diff --git a/commands/hugo.go b/commands/hugo.go
index f250d4eff0a41046311db74eb6670550eca37ddb..3d26d8265aedcd0bccc282fc512e8c64a63c865e 100644 (file)
--- a/commands/hugo.go
+++ b/commands/hugo.go
@@ -310,9 +310,10 @@ func InitializeConfig(subCmdVs ...*cobra.Command) error {
                if cmdV.Flags().Lookup("ignoreCache").Changed {
                        viper.Set("IgnoreCache", IgnoreCache)
                }
-               if cmdV.Flags().Lookup("noTimes").Changed {
-                       viper.Set("NoTimes", NoTimes)
-               }
+       }
+
+       if hugoCmdV.Flags().Lookup("noTimes").Changed {
+               viper.Set("NoTimes", NoTimes)
        }
 
        if BaseURL != "" {

diff --git a/commands/server.go b/commands/server.go
index adcf6dff2dc81e790edbb0ba1067e4bb054f7ffc..1ca39278b67bb3f937ca753cce28aed64db6d447 100644 (file)
--- a/commands/server.go
+++ b/commands/server.go
@@ -148,6 +148,10 @@ func server(cmd *cobra.Command, args []string) error {
                viper.Set("PublishDir", "/")
        }
 
+       if serverCmd.Flags().Lookup("noTimes").Changed {
+               viper.Set("NoTimes", NoTimes)
+       }
+
        if err := build(serverWatch); err != nil {
                return err
        }