removed the -e flag

Author: fabpot

CHANGELOG

@@ -1,10 +1,11 @@
 1.1.0
 -----
 
+ * remove the -e flag (use the MAILTO environment variable instead)
  * added support for MAILTO as the preferred way to configure croncape
  * disabled the timeout by default
  * added a way to disable the timeout
- * removed the -c flag
+ * removed the -c flag (pass the command directly)
  * fixed environment variables handling
 
 1.0.0

README.md

@@ -34,11 +34,6 @@ That's it!
 Note that the `MAILTO` environment variable can also be defined globally in
 `/etc/crontab`; it supports multiple recipients by separating them with a comma.
 
-Alternatively, you can also the `-e` flag to define emails:
-
-    0 6 * * * croncape -e "[email protected]" ls -lsa
-    0 6 * * * croncape -e "[email protected],[email protected]" ls -lsa
-
 Besides sending emails, croncape can also kill the run command after a given
 timeout, via the `-t` flag (disabled by default):
 
@@ -59,7 +54,8 @@ Croncape is very similar to [cronwrap][2], with some differences:
  * Tries to use `sendmail` or `mail` depending on availability (cronwrap only
    works with `sendmail`);
 
- * Reads the email from the standard crontab `MAILTO` environment variable.
+ * Reads the email from the standard crontab `MAILTO` environment variable
+   instead of a `-e` flag.
 
 For a simpler alternative, have a look at [cronic][3].
 

main.go

@@ -39,9 +39,10 @@ func main() {
 		log.Fatalln(err)
 	}
 
-	req := request{}
+	req := request{
+		emails: os.Getenv("MAILTO"),
+	}
 
-	flag.StringVar(&req.emails, "e", "", `Emails to send reports when the command fails or exceeds timeout, like '-e "[email protected],[email protected]"'`)
 	flag.DurationVar(&req.timeout, "t", 0, `Timeout for the command, like "-t 2h", "-t 2m", or "-t 30s". After the timeout, the command is killed, disabled by default`)
 	flag.StringVar(&req.transport, "p", "auto", `Transport to use, like "-p auto", "-p mail", "-p sendmail"`)
 	flag.BoolVar(&req.verbose, "v", false, "Enable sending emails even if command is successful")
@@ -53,10 +54,6 @@ func main() {
 		os.Exit(1)
 	}
 
-	if req.emails == "" && os.Getenv("MAILTO") != "" {
-		req.emails = os.Getenv("MAILTO")
-	}
-
 	r := execCmd(wd, req)
 
 	if r.killed || r.code != 0 || r.request.verbose {