Skip to content

Commit

Permalink
Global basedirs flag uses wrong default value
Browse files Browse the repository at this point in the history
The default value of the global `basedirs` flag (1) used the
`config.AppConfig.Snowblocks.Paths` array instead of
`config.AppConfig.Snowblocks.BaseDirs` that resulted in invalid paths
being processed by task runners and the configuration validators.

References:
  (1) https://github.com/arcticicestudio/snowsaw/blob/145a4c36caf960fc9a43b16c105df997e819a04d/cmd/snowsaw/snowsaw.go#L74

Epic: GH-33
Fixes GH-77
  • Loading branch information
arcticicestudio committed Jul 15, 2019
1 parent 006ae99 commit ed6226d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmd/snowsaw/snowsaw.go
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ func init() {
rootCmd.PersistentFlags().StringVar(&explicitConfigFilePath, "config", "", "path to the configuration file")
rootCmd.PersistentFlags().BoolVar(&debug, "debug", false, "enable debug information output")
rootCmd.PersistentFlags().StringSliceVarP(
&snowblockBaseDirs, "basedirs", "b", config.AppConfig.Snowblocks.Paths,
&snowblockBaseDirs, "basedirs", "b", config.AppConfig.Snowblocks.BaseDirs,
"comma-separated paths to snowblock base directories")

// Set the app version information for the automatically generated `version` flag.
Expand Down

0 comments on commit ed6226d

Please sign in to comment.