fix(schedule): omit launchd log paths when schedule-log is configured

When schedule-log is set in the profile configuration, resticprofile
handles logging internally. Previously, the launchd plist always
included StandardOutPath/StandardErrorPath entries pointing to
<job-name>.log files, which remained empty when schedule-log was
configured. This change omits those plist fields (via their omitempty
tags) when schedule-log is set, avoiding the creation of empty log
files.

Author: wyattjoh

schedule/config.go

@@ -24,6 +24,7 @@ type Config struct {
 	Flags              map[string]string // flags added to the command line
 	AfterNetworkOnline bool
 	SystemdDropInFiles []string
+	Log                string
 	HideWindow         bool
 	removeOnly         bool
 }

schedule/handler_darwin.go

@@ -183,9 +183,14 @@ func (h *HandlerLaunchd) Scheduled(profileName string) ([]Config, error) {
 
 func (h *HandlerLaunchd) getLaunchdJob(job *Config, schedules []*calendar.Event) *darwin.LaunchdJob {
 	name := getJobName(job.ProfileName, job.CommandName)
-	// we always set the log file in the job settings as a default
-	// if changed in the configuration via schedule-log the standard output will be empty anyway
-	logfile := name + ".log"
+	// Only set the log file when schedule-log is not configured.
+	// When schedule-log is set, resticprofile handles logging internally,
+	// so we omit StandardOutPath/StandardErrorPath from the plist to avoid
+	// creating empty log files (the plist fields have omitempty tags).
+	var logfile string
+	if job.Log == "" {
+		logfile = name + ".log"
+	}
 
 	// Format schedule env, adding PATH if not yet provided by the schedule config
 	env := util.NewDefaultEnvironment(job.Environment...)

schedule_jobs.go

@@ -239,5 +239,6 @@ func scheduleToConfig(sched *config.Schedule) *schedule.Config {
 		AfterNetworkOnline: sched.AfterNetworkOnline.IsTrue(),
 		SystemdDropInFiles: sched.SystemdDropInFiles,
 		HideWindow:         sched.HideWindow.IsTrue(),
+		Log:                sched.Log,
 	}
 }