console: logging to a log file is now optional; add -log flag to spec…

…ify log output

hsapp/app.go

@@ -157,17 +157,24 @@ func (a *App) Run() {
 
 	defer a.Quit() // force-close and save everything (in case of crash)
 
-	a.logFile, err = os.OpenFile(a.config.LogFilePath, os.O_CREATE|os.O_APPEND|os.O_WRONLY, logFilePerms)
-	if err != nil {
-		log.Printf("Error opening log file at %s: %v", a.config.LogFilePath, err)
-	}
+	if a.config.LoggingToFile || *a.Flags.logFile != "" {
+		var path string = a.config.LogFilePath
+		if *a.Flags.logFile != "" {
+			path = *a.Flags.logFile
+		}
 
-	defer func() {
-		err := a.logFile.Close()
+		a.logFile, err = os.OpenFile(filepath.Clean(path), os.O_CREATE|os.O_APPEND|os.O_WRONLY, logFilePerms)
 		if err != nil {
-			log.Fatal(err)
+			log.Printf("Error opening log file at %s: %v", a.config.LogFilePath, err)
 		}
-	}()
+
+		defer func() {
+			err := a.logFile.Close()
+			if err != nil {
+				log.Fatal(err)
+			}
+		}()
+	}
 
 	if err := a.setup(); err != nil {
 		log.Panic(err)

hsapp/flags.go

@@ -12,18 +12,21 @@ import (
 type Flags struct {
 	optionalConfigPath *string
 	bgColor            *uint
+	logFile            *string
 }
 
 func (a *App) parseArgs() {
 	configDescr := fmt.Sprintf("specify a custom config path. Default is: %s", hsconfig.GetConfigPath())
 	a.Flags.optionalConfigPath = flag.String("config", "", configDescr)
 	a.Flags.bgColor = flag.Uint("bgColor", hsconfig.DefaultBGColor, "custom background color")
+	a.Flags.logFile = flag.String("log", "", "path to the output log file")
 	showHelp := flag.Bool("h", false, "Show help")
 
 	flag.Usage = func() {
 		fmt.Printf("usage: %s [<flags>]\n\nFlags:\n", os.Args[0])
 		flag.PrintDefaults()
 	}
+
 	flag.Parse()
 
 	if *showHelp {

hsconfig/config.go

@@ -39,6 +39,7 @@ type Config struct {
 	ExternalListFile        string                      `json:"externalListFile"`
 	OpenMostRecentOnStartup bool                        `json:"openMostRecentOnStartup"`
 	ProjectStates           map[string]hsstate.AppState `json:"projectStates"`
+	LoggingToFile           bool                        `json:"loggingToFile"`
 	LogFilePath             string                      `json:"logFile"`
 	Locale                  hsenum.Locale               `json:"locale"`
 	BGColor                 color.RGBA                  `json:"bgColor"`
@@ -60,6 +61,7 @@ func generateDefaultConfig(path string) *Config {
 		RecentProjects:          []string{},
 		OpenMostRecentOnStartup: true,
 		ProjectStates:           make(map[string]hsstate.AppState),
+		LoggingToFile:           false,
 		LogFilePath:             filepath.Join(filepath.Dir(path), "output.txt"),
 		Locale:                  hsenum.LocaleEnglish,
 		BGColor:                 hsutil.Color(DefaultBGColor),

hswindow/hsdialog/hspreferencesdialog/preferencesdialog.go

@@ -73,6 +73,19 @@ func (p *PreferencesDialog) Build() {
 			g.Separator(),
 			g.Checkbox("Open most recent project on start-up", &p.config.OpenMostRecentOnStartup),
 			g.Separator(),
+			g.Checkbox("Save log output in a log file", &p.config.LoggingToFile),
+			g.Custom(func() {
+				if p.config.LoggingToFile {
+					g.Layout{
+						g.Label("Log file path"),
+						g.Line(
+							g.InputText("##AppPreferencesLogFilePath", &p.config.LogFilePath).Size(textboxSize).Flags(g.InputTextFlags_ReadOnly),
+							g.Button("...##AppPreferencesLogFilePathBrowse").Size(btnW, btnH).OnClick(p.onBrowseLogFilePathClicked),
+						),
+					}.Build()
+				}
+			}),
+			g.Separator(),
 			g.Custom(func() {
 				if !p.restartPrompt {
 					return
@@ -135,6 +148,15 @@ func (p *PreferencesDialog) onBrowseExternalListfileClicked() {
 	p.config.ExternalListFile = filePath
 }
 
+func (p *PreferencesDialog) onBrowseLogFilePathClicked() {
+	path, err := dialog.Directory().Browse()
+	if err != nil || path == "" {
+		return
+	}
+
+	p.config.LogFilePath = path
+}
+
 func (p *PreferencesDialog) onSaveClicked() {
 	p.onConfigChanged(p.config)
 	p.Visible = false

hswindow/hstoolwindow/hsconsole/console.go

@@ -64,11 +64,13 @@ func (c *Console) Write(p []byte) (n int, err error) {
 
 	fmt.Print(msg) // print to terminal
 
-	n, err = c.logFile.Write(p) // print to file
-	if err != nil {
-		return n, fmt.Errorf("error writing to log file: %w", err)
-	} else if n != len(p) {
-		return n, fmt.Errorf("invalid data written to log file")
+	if c.logFile != nil {
+		n, err = c.logFile.Write(p) // print to file
+		if err != nil {
+			return n, fmt.Errorf("error writing to log file: %w", err)
+		} else if n != len(p) {
+			return n, fmt.Errorf("invalid data written to log file")
+		}
 	}
 
 	return len(p), nil